Update connecting-to-the-internet-via-usb.md #306

ghpille · 2016-12-09T22:34:32Z

I've never used Gnome, and never any GUI for my network connections. I couldn't find how to establish the connection, so I found out myself, but I wanted to register what I learned. This page seemed the most appropriate place.

dlech

Thanks for the pull request. There are a few things I would like to fix before we merge it.

Also see #39. It is basically the same thing as what is being added here, but was never finished. It is quite a bit out of date, but there may be some useful info there.

dlech · 2016-12-09T23:48:20Z

docs/tutorials/connecting-to-the-internet-via-usb.md

+    3.  Set default Linux network settings: "Wireless and Networks" > "All Network Connections" > "Wired" >
+        "IPv4" > "Change" > "Load Linux Defaults".  The EV3 should get address 10.42.0.3, gateway 10.42.0.1.
+
+    4.  Verify if the EV3 is recognized by your Linux PC:  "lsusb" should show


use single backtick instead of quotes for lsusb

dlech · 2016-12-09T23:49:14Z

docs/tutorials/connecting-to-the-internet-via-usb.md

@@ -135,6 +135,75 @@ If you are using an older version, please upgrade.

        {% include /util/screenshot.html source="/images/brickman/wired-status-online-connect-automatically-selected.png" %}

+
+*   {: tab="Debian_commandline"}{% include /style/icon.html type="info" %}


There doesn't seem to be anything Debian-specifc here. Maybe we could just call this "linux_command_line"?

You're probably right.

dlech · 2016-12-09T23:50:16Z

docs/tutorials/connecting-to-the-internet-via-usb.md

+
+*   {: tab="Debian_commandline"}{% include /style/icon.html type="info" %}
+    These instructions were written using Debian look-alikes for those wanting to know what happens under the hood.
+    The commands need to be executed by or as root.


"by or as" seems redundant. One or the other is sufficient.

dlech · 2016-12-09T23:52:48Z

docs/tutorials/connecting-to-the-internet-via-usb.md

+
+    2.  Verify that USB tethering is disabled: "Wireless and Networks" > "Tethering.
+
+    3.  Set default Linux network settings: "Wireless and Networks" > "All Network Connections" > "Wired" >


To be consistent with the rest of the website, use italics for name of things printed on the screen.

*Wireless and Networks > All Network Connections > Wired*

Also, a screenshot here would be nice. We may have some existing ones that can be reused.

Do you mean screenshots of how to navigate through ev3dev's menus?

dlech · 2016-12-09T23:53:12Z

docs/tutorials/connecting-to-the-internet-via-usb.md

+
+    4.  Verify if the EV3 is recognized by your Linux PC:  "lsusb" should show
+
+        `Bus 001 Device 002: ID 0525:a4aa Netchip Technology, Inc. Linux-USB CDC Composite Gadge (Ethernet and ACM)`


indent 4 extra spaces instead of using single backtick

With #307, we'll be able to use code fences, so let's replace this with a fence and not remove the fences in other places. Fences are much more readable than indentation.

Fences are much more readable than indentation.

I disagree.

Although it really doesn't make a difference in normal text, in lists I find it very hard to identify code formatting when it's space-indented. I think that code fences make it much easier to quickly identify the code in an indented list.

In lists, it is still indented 4 extra spaces from the list text, so it looks exactly the same as normal text to me.

And regardless of any opinions on which is "better", the rest of the website uses indents, so I would like to stay consistent with that.

Additionally, indented code is a pain to edit, especially in lists.

OK, I've changed my mind (c.f. #308).

In this pull request though, I would like to have a blank line before the start of each code fence.

In lists, it is still indented 4 extra spaces from the list text, so it looks exactly the same as normal text to me.

It's extremely difficult to differentiate between sub-bullets, indented additional lines, and code blocks; they all are indented in the same way. Using indentation for code blocks is an unnatural use of whitespace, and as a result it conflicts with all the other uses of whitespace.

And regardless of any opinions on which is "better", the rest of the website uses indents, so I would like to stay consistent with that.

argumentum ad antiquitatem. "It has always been done this way" isn't a reason to continue doing it; switching to code fences is trivial and something that I would happily put in the work for now that we have that ability.

How so?

Because you must maintain the indentation. Every line must be prepended with four spaces, so not only do you need to add the spacing before every line, but you also need to mess with indentation when copy-pasting code. This is especially difficult in GitHub's online editor.

Oh, it looks like GitHub has decided not to auto-load comments on review threads 😆 Sounds good.

dlech · 2016-12-09T23:54:52Z

docs/tutorials/connecting-to-the-internet-via-usb.md

+        `# ifconfig enp0s2f1u5 10.42.0.1 netmask 255.255.255.0  up`
+
+    8.  Verify if you can ping the EV3:
+        ```


indent 4 extra spaces instead of using code fence

dlech · 2016-12-09T23:55:38Z

docs/tutorials/connecting-to-the-internet-via-usb.md

+        connection which you want to make available to your EV3, there is more that needs to be done.
+
+    10.  Allow your system to forward IP traffic (given eth0 carries your internet connection):
+        ```


dlech · 2016-12-09T23:55:43Z

docs/tutorials/connecting-to-the-internet-via-usb.md

+        # echo 1 > /proc/sys/net/ipv4/ip_forward
+        ```
+        (to make this permanent, add "net.ipv4.ip_forward=1" to /etc/sysctl.conf)
+        ```


dlech · 2016-12-09T23:57:24Z

docs/tutorials/connecting-to-the-internet-via-usb.md

+
+    11.  Assign a name-server (eg. google-public-dns-a) to your EV3 (you need to be logged in on your EV3 as root):
+
+        `# echo "nameserver 8.8.8.8" > /etc/resolv.conf`


We use ConnMan on the EV3 for network management. It will write over any changes made manually to /etc/resolv.conf. Instead a step should be added after step 3 that explains how to set the DNS using Brickman.

I did some reading about connman (does it sound like con-man?), tried in vain to get it to do what I wanted, and disabled it. Now, ev3dev nicely configures the network the way I tell it to in /etc/network/interfaces and /etc/resolv.conf.

dlech · 2016-12-09T23:59:52Z

docs/tutorials/connecting-to-the-internet-via-usb.md

+    12.  Verify if you can ping that nameserver:
+
+         `# ping 8.8.8.8`
+


Shouldn't there be a step where you press the Connect button for the network connection on the EV3? When you do this, it will say "Online" on the screen, which verifies that DNS resolution is working.

Quite right, after selection of the "Load Linux Defaults", but possibly because of the manual interventions, the state remains "Connected", even though I can "apt-get upgrade".

ghpille · 2016-12-10T14:44:40Z

Hallo David,
I was quite angry this morning, just after having issued my pull request, when I came across #39. I hate to waste my time, not so much finding a solution myself, that's always fun, but getting to grips with Github and its PITA editor. Then, scanning the pull requests, I found the same information was entered two years ago.
You'll need to convince me to put any more effort into documenting my encounters with EV3.
Kind regards,
Gerard

Forgot an elementary part of this exercise: loading the g_cdc module.

dlech · 2016-12-11T19:46:05Z

I am sorry that this makes you angry. I hope that you are willing to complete this work so that other people will not have to be frustrated like you are now.

ghpille · 2016-12-12T10:27:51Z

Can you tell me what went wrong with #39 so that I don't make the same mistake? Can I update the documentation offline and avoid the Github-interface? Is it possible to see what it will look like? The Github online editor's "preview" looks nothing like the real page.

ghpille · 2016-12-12T13:39:43Z

Just found the answers to my offline and preview questions in the README.md. What a strange name for that document. Why is it called "README.md"?

WasabiFan · 2016-12-12T14:12:21Z

What a strange name for that document. Why is it called "README.md"?

Because that is the convention across GitHub and the broader software-development world. The expectation is that you will "read it" (hence "read me") to get important information on the project you are looking at. GitHub extends that basic idea to render the README as the home page of your repository. I would recommend getting into the habit of reading the opening paragraph of every README that you see on GitHub, because it'll probably answer most of your questions about the project.

ghpille · 2016-12-12T14:19:26Z

Sorry about that, WasabiFan, this is probably not the right place, but I was only making fun of myself.

dlech · 2016-12-12T16:17:57Z

Can you tell me what went wrong with #39 so that I don't make the same mistake?

Nothing went wrong. The submitter just did not make time to address the review comments. You have already done this (thank you), so there is nothing to worry about.

WasabiFan · 2016-12-20T04:42:10Z

@dlech can you take another pass at this now that your review has been addressed?

dlech · 2016-12-20T06:01:04Z

docs/tutorials/connecting-to-the-internet-via-usb.md

+
+    1.  Booting with a standard ev3dev image, lsusb will show
+
+        `Bus 002 Device 008: ID 0694:0005 Lego Group`


This should be using code fence. e.g.

``` Bus 002 Device 008: ID 0694:0005 Lego Group ```

dlech · 2016-12-20T06:03:30Z

docs/tutorials/connecting-to-the-internet-via-usb.md

+
+    10. Assign the gateway address to the EV3 network device:
+
+        `# ifconfig enp0s2f1u5 10.42.0.1 netmask 255.255.255.0  up`


dlech · 2016-12-20T06:03:38Z

docs/tutorials/connecting-to-the-internet-via-usb.md

+    13. Allow your system to forward IP traffic (given eth0 carries your
+        internet connection):
+
+        `# echo 1 > /proc/sys/net/ipv4/ip_forward`


dlech · 2016-12-20T06:04:36Z

docs/tutorials/connecting-to-the-internet-via-usb.md

+    14. Assign a name-server (eg. google-public-dns-a) to your EV3 (you need
+        to be logged in on your EV3 as root):
+
+        `# echo "nameserver 8.8.8.8" > /etc/resolv.conf`


As I mentioned in my previous review, ev3dev uses ConnMan for network management. It will write over /etc/resolv.conf, so I don't thing manually editing it is a good idea. Instead, I think it would be better to set the nameserver using Bricman at the same time we are setting the manual configuration above.

dlech · 2016-12-20T06:05:41Z

docs/tutorials/connecting-to-the-internet-via-usb.md

+
+    15. Verify if you can ping that nameserver:
+
+        `# ping 8.8.8.8`


Again, as in my previous review, I mentioned that it is possible to see the connection status using Brickman. It would save the steps of having to log in to the EV3 in order to complete this tutorial.

I see. In that case we can drop steps 1 to 15.

dlech · 2016-12-20T06:11:14Z

docs/tutorials/connecting-to-the-internet-via-usb.md

+
+    4.  Connect the USB cable to your computer and its mini-B plug to the EV3 (slot marked "PC").
+
+    5.  Verify that USB tethering is disabled: *"Wireless and Networks" > "Tethering*.


Prefer no quotes on *Wireless and Networks > Tethering*

If you insist on quotes, you are missing one at the end of Tethering.

This could also be more clear that we are now using the EV3, not the host computer.

dlech · 2016-12-20T06:11:36Z

docs/tutorials/connecting-to-the-internet-via-usb.md

+
+    5.  Verify that USB tethering is disabled: *"Wireless and Networks" > "Tethering*.
+
+    6.  Set default Linux network settings: *"Wireless and Networks" > "All Network Connections" > "Wired" >


again, prefer no quotes here

dlech · 2016-12-20T06:13:30Z

docs/tutorials/connecting-to-the-internet-via-usb.md

+    The commands need to be executed by root.
+    {:class="alert alert-info"}
+
+    1.  Booting with a standard ev3dev image, lsusb will show


This is not clear if lsusb should be run on the EV3 or the host computer.

I think you should give some credit to the people interested in this subject: they would not expect to see the EV3 when lsusb would be run on the EV3. Moreover, before step 12 it is not possible to enter an OS command on the EV3. But I'll try to add this to the introduction, along with Debian version etc.

dlech · 2016-12-20T06:15:50Z

docs/tutorials/connecting-to-the-internet-via-usb.md

+
+        I'll be using the EV3 as a USB Gadget, as follows.
+
+    2.  When you've copied the ev3dev's image to the SD-card, mount the second partition (EV3DEV_ROOTFS).


Why are we doing this? ev3dev already sets up a gadget driver by default. Does it not work for you?

dlech · 2016-12-20T06:25:17Z

docs/tutorials/connecting-to-the-internet-via-usb.md

@@ -135,6 +135,99 @@ If you are using an older version, please upgrade.

        {% include /util/screenshot.html source="/images/brickman/wired-status-online-connect-automatically-selected.png" %}

+
+*   {: tab="linux_command_line"}{% include /style/icon.html type="info" %}
+    These instructions were written using Debian look-alikes for those wanting to know what happens under the hood.


Please mention the exact debian version that was used for the tutorial. We should also mention that is should work for most Linux distros that don't otherwise have a network manager program (or something like that).

WasabiFan · 2016-12-28T19:00:52Z

@ghpille Are you going to be able to finish this? I really want to make sure that we get these changes merged this time!

ghpille · 2016-12-29T09:34:38Z

@WasabiFan I certainly hope so, but currently facing a part of my roof taken away by a storm and my girlfriends heating system breaking down while it's winter down here. Priorities, priorities ...

WasabiFan · 2016-12-30T02:05:37Z

currently facing a part of my roof taken away by a storm

Aw, I hate when that happens 😉

No problem, I just wanted to make sure this wasn't forgotten about. There's no rush on this work.

egnrse · 2017-08-04T05:55:04Z

@ghpille why aren't you finishing this??

ghpille · 2017-08-09T13:03:32Z

@enitsche Are you interested in my private life? It took me a couple of weeks to replace my girlfriends heating system, the replacment weighing some 280kg. The same time I had to replace a 200l boiler. Then my dog died, took me a week to find a new one, but this last one asks for more attention then the previous one. I've been on holiday for 3 weeks to southern France, spent an entire weekend making plum jam, had a small car accident but it took me some time to repair the damage. I don't keep a diary, there are large hiatuses.

Dlech made it clear that the steps I added to the tutorial, are unnecessary. In that case, why would I finish it?

Update connecting-to-the-internet-via-usb.md

f25bb78

dlech requested changes Dec 10, 2016

View reviewed changes

WasabiFan added the documentation content label Dec 10, 2016

Update connecting-to-the-internet-via-usb.md

af44168

Forgot an elementary part of this exercise: loading the g_cdc module.

beautification

4f563e2

WasabiFan requested a review from dlech December 20, 2016 06:25

dlech reviewed Dec 20, 2016

View reviewed changes

dlech force-pushed the master branch from a126dfd to 536a64c Compare June 14, 2018 18:33

		@@ -135,6 +135,75 @@ If you are using an older version, please upgrade.

		{% include /util/screenshot.html source="/images/brickman/wired-status-online-connect-automatically-selected.png" %}


		* {: tab="Debian_commandline"}{% include /style/icon.html type="info" %}


		2. Verify that USB tethering is disabled: "Wireless and Networks" > "Tethering.

		3. Set default Linux network settings: "Wireless and Networks" > "All Network Connections" > "Wired" >


		4. Verify if the EV3 is recognized by your Linux PC: "lsusb" should show

		`Bus 001 Device 002: ID 0525:a4aa Netchip Technology, Inc. Linux-USB CDC Composite Gadge (Ethernet and ACM)`


		11. Assign a name-server (eg. google-public-dns-a) to your EV3 (you need to be logged in on your EV3 as root):

		`# echo "nameserver 8.8.8.8" > /etc/resolv.conf`


		1. Booting with a standard ev3dev image, lsusb will show

		`Bus 002 Device 008: ID 0694:0005 Lego Group`


		10. Assign the gateway address to the EV3 network device:

		`# ifconfig enp0s2f1u5 10.42.0.1 netmask 255.255.255.0 up`


		4. Connect the USB cable to your computer and its mini-B plug to the EV3 (slot marked "PC").

		5. Verify that USB tethering is disabled: "Wireless and Networks" > "Tethering.


		5. Verify that USB tethering is disabled: "Wireless and Networks" > "Tethering.

		6. Set default Linux network settings: *"Wireless and Networks" > "All Network Connections" > "Wired" >


		I'll be using the EV3 as a USB Gadget, as follows.

		2. When you've copied the ev3dev's image to the SD-card, mount the second partition (EV3DEV_ROOTFS).

Update connecting-to-the-internet-via-usb.md #306

Are you sure you want to change the base?

Update connecting-to-the-internet-via-usb.md #306

Conversation

ghpille commented Dec 9, 2016

dlech left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

ghpille commented Dec 10, 2016

dlech commented Dec 11, 2016

ghpille commented Dec 12, 2016

ghpille commented Dec 12, 2016

WasabiFan commented Dec 12, 2016

ghpille commented Dec 12, 2016

dlech commented Dec 12, 2016

WasabiFan commented Dec 20, 2016

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

WasabiFan commented Dec 28, 2016

ghpille commented Dec 29, 2016

WasabiFan commented Dec 30, 2016

egnrse commented Aug 4, 2017

ghpille commented Aug 9, 2017