Re: [PATCH v3 0/5] Add support for ethernet tethering

2013-04-28 Thread Patrik Flykt
On Mon, 2013-04-29 at 08:40 +0300, Patrik Flykt wrote: > PreferredTechnologies it might should be only TetheringTechnologies Since it is named 'Preferred', maybe it should actually be 'Tethered' ? Patrik ___ connman mailing list connman@connma

Re: [PATCH v3 0/5] Add support for ethernet tethering

2013-04-28 Thread Patrik Flykt
On Fri, 2013-04-26 at 07:51 -0700, Marcel Holtmann wrote: > actually I was not fully convinced with the main.conf setting name > yet. > It is now AllowedTetheringTechnologies and I was looking how we named > the others. So essentially to follow in sync with > PreferredTechnologies it might should

Re: [PATCH v3 0/5] Add support for ethernet tethering

2013-04-26 Thread Marcel Holtmann
Hi Patrik, >> v3: >> - Changed the name and semantics of the main.conf variable. >> Now it is a list of technologies that are allowed to >> tether. Patches 1, 2 and 5 >> - Reworked the rtnl patch 4, get rid of extra variable >> - Combined patches 5 and 6 together > > Applied, thanks! actually

Re: [PATCH v3 0/5] Add support for ethernet tethering

2013-04-26 Thread Patrik Flykt
On Thu, 2013-04-25 at 14:47 +0300, Jukka Rissanen wrote: > Hi, > > v3: > - Changed the name and semantics of the main.conf variable. > Now it is a list of technologies that are allowed to > tether. Patches 1, 2 and 5 > - Reworked the rtnl patch 4, get rid of extra variable > - Combined patches

[PATCH v3 0/5] Add support for ethernet tethering

2013-04-25 Thread Jukka Rissanen
Hi, v3: - Changed the name and semantics of the main.conf variable. Now it is a list of technologies that are allowed to tether. Patches 1, 2 and 5 - Reworked the rtnl patch 4, get rid of extra variable - Combined patches 5 and 6 together v2: - Removed patch 1, the proper fix for the issue in