Skip to contentSkip to conversation

Added notes to the mosh package help to clarify run time issues.

PATCH New Gonçalo Salazar 2 participants 6 messages May 29, 2017

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 Added notes to the mosh package help to clarify run time issues.

Added notes to the mosh package help to clarify some
issues related with it to ensure mosh will work properly after adding it. This includes adding a proper LOCALE on the host and an extra flag when using mosh with dropbear.

---
 package/mosh/Config.in | 4 ++++
 1 file changed, 4 insertions(+)
package/mosh/Config.in +4 -0
index e0ff2cf..9f09b4f 100644
@@ -25,3 +25,7 @@ config BR2_PACKAGE_MOSH
 	  of user keystrokes.
 
 	  https://mosh.mit.edu
+
+          Mosh requires a host LOCALE matching the client provided LOCALE.
+          When using mosh with dropbear, a version with the --no-ssh-pty option on
+          the client is required.
Signature · 1 line
2.7.4
#1 lore raw

PATCH 1/1 Added notes to the mosh package help to clarify run time issues.

Hello,

Quoted 4 lines · Gon?alo Salazar

Added notes to the mosh package help to clarify some
issues related with it to ensure mosh will work properly after adding it. This includes adding a proper LOCALE on the host and an extra flag when using mosh with dropbear.

The commit title should start with the name of the package, i.e:

mosh: add notes to clarify runtime issues

Quoted 1 line

https://mosh.mit.edu

The notes should go before the upstream URL, and the indentation should be one tab + two spaces.

package/mosh/Config.in config BR2_PACKAGE_MOSH · line 29

Quoted 2 lines
package/mosh/Config.in +2 -0
+
+          Mosh requires a host LOCALE matching the client provided LOCALE.

I'm sorry, but I don't understand this sentence.

package/mosh/Config.in config BR2_PACKAGE_MOSH · line 31

Quoted 2 lines
package/mosh/Config.in +2 -0
+          When using mosh with dropbear, a version with the --no-ssh-pty option on
+          the client is required.

And also this one.

So for me, those explanations do not clarify anything :-/

Best regards,

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
#2 lore raw

PATCH 1/1 Added notes to the mosh package help to clarify run time issues.

Hello,

Thank you for the inputs, I realize that the notes are not really clear. I'm trying to convey the idea of runtime dependencies that exist if you are trying to include mosh in a host you will be accessing later.

I'll write the notes.

Should I resubmit the patch after clarifying the notes?

Best regards,
Gon?alo

2017-05-29 21:39 GMT+01:00 Thomas Petazzoni
<[email protected]>:

Quoted 36 lines

Hello,

Quoted 4 lines · Gon?alo Salazar

Added notes to the mosh package help to clarify some
issues related with it to ensure mosh will work properly after adding it. This includes adding a proper LOCALE on the host and an extra flag when using mosh with dropbear.

The commit title should start with the name of the package, i.e:

mosh: add notes to clarify runtime issues

Quoted 1 line

https://mosh.mit.edu

The notes should go before the upstream URL, and the indentation should be one tab + two spaces.

package/mosh/Config.in config BR2_PACKAGE_MOSH · line 29

Quoted 2 lines
package/mosh/Config.in +2 -0
+
+          Mosh requires a host LOCALE matching the client provided LOCALE.

I'm sorry, but I don't understand this sentence.

package/mosh/Config.in config BR2_PACKAGE_MOSH · line 31

Quoted 2 lines
package/mosh/Config.in +2 -0
+          When using mosh with dropbear, a version with the --no-ssh-pty option on
+          the client is required.

And also this one.

So for me, those explanations do not clarify anything :-/

Best regards,

Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

#3 lore raw

PATCH 1/1 Added notes to the mosh package help to clarify run time issues.

Hello,

Quoted 3 lines · Gon?alo Salazar

Thank you for the inputs, I realize that the notes are not really clear. I'm trying to convey the idea of runtime dependencies that exist if you are trying to include mosh in a host you will be accessing later.

Beware that in the context of Buildroot, "host" is often used to indicate the machine where the Buildroot build is done, as opposed to the "target", where the stuff built by Buildroot is executed.

Hence, it should be made extra clear that you mean "host" in the networking sense, and not the build sense.

Quoted 1 line

Should I resubmit the patch after clarifying the notes?

Yes, please.

Thanks!

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
#4 lore raw

PATCH 1/1 Added notes to the mosh package help to clarify run time issues.

Hello,

Just to ensure that the patch is submitted correctly this time, is this clearer?

Mosh requires that the remote machine has a locale setting matching the one on the client.

When using mosh with dropbear on a remote machine, the client needs mosh with a version above 1.3.0 and the --no-ssh-pty option needs to be passed when starting the connection.

Thanks,
Gon?alo Salazar

2017-05-29 22:15 GMT+01:00 Thomas Petazzoni
<[email protected]>:

Quoted 26 lines

Hello,

Quoted 3 lines · Gon?alo Salazar

Thank you for the inputs, I realize that the notes are not really clear. I'm trying to convey the idea of runtime dependencies that exist if you are trying to include mosh in a host you will be accessing later.

Beware that in the context of Buildroot, "host" is often used to indicate the machine where the Buildroot build is done, as opposed to the "target", where the stuff built by Buildroot is executed.

Hence, it should be made extra clear that you mean "host" in the networking sense, and not the build sense.

Quoted 1 line

Should I resubmit the patch after clarifying the notes?

Yes, please.

Thanks!

Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

#5 lore raw

PATCH 1/1 Added notes to the mosh package help to clarify run time issues.

Hello,

Quoted 8 lines · Gon?alo Salazar

Just to ensure that the patch is submitted correctly this time, is this clearer?

Mosh requires that the remote machine has a locale setting matching the one on the client.

When using mosh with dropbear on a remote machine, the client needs mosh with a version above 1.3.0 and the --no-ssh-pty option needs to be passed when starting the connection.

Yes, much better!

Thanks!

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com