[PATCH v8] fff-hoods: Use channel provided by gateway

Tim Niemeyer tim at tn-x.org
Do Nov 2 20:44:26 CET 2017


Hi

Und.. Applied.

Am Montag, den 30.10.2017, 19:16 +0100 schrieb Christian Dresel:
> Hi
> 
> danke für deine Arbeit und den Stress mit mir im IRC ;) Hab das Ding nun
> endlich begriffen und auch getestet. Sieht soweit gut aus. Nach diesem
> Patch sollte am besten auch gleich "[PATCH v5] fff-hoods: Introduce
> sector files for custom settings" mit applied werden.
> 
> Kleine Anmerkung noch inline, vielleicht kann man das einfach beim
> applien anpassen
> 
> Reviewed-by: Christian Dresel <fff at chrisi01.de>
> 
> On 26.10.2017 08:57, Adrian Schmutzler wrote:
Leider ist auch in diesem Patch wieder die Commit-Message falsch. Commit
message (nicht wirklich vorhanden) und Kommentare für die Reviewer (wie
z.B. die Changes) sind beide in der Commit-Message vermischt.

Ein paar Patches lang werde ich das noch nachbearbeiten, aber irgendwann
werde ich sowas einfach nicht mehr applien!

> > ALTERNATIVE PATCH: This prevents setup if a channel
> > is missing (json defective)
> > 
> > I personally however still prefer v7 with default channel
> > 
> > Changes in v2:
> > - rebase due to conflict with 802.11s
> > 
> > Changes in v3:
> > - Fix wrong variable name introduced in v2
> > 
> > Changes in v4:
> > - Fixed comment
> > - Fixed channel comparison for wXsta
> > 
> > Changes in v5:
> > - Make radio in wifiAddPhyCond local
> > 
> > Changes in v6:
> > - Rebase
> > 
> > Changes in v7:
> > - Check if channel set and default otherwise
> > - Remove obsolete channel variables at beginning of script
> > 
> > Changes in v8:
> > - Exit if channel missing
> > 
> > Signed-off-by: Adrian Schmutzler <freifunk at adrianschmutzler.de>
> > Signed-off-by: Fabian Bläse <fabian at blaese.de>
> > ---
> >  .../fff/fff-hoods/files/usr/sbin/configurehood     | 27 ++++++++++++++--------
> >  .../fff-wireless/files/lib/functions/fff/wireless  | 17 +++++++-------
> >  2 files changed, 25 insertions(+), 19 deletions(-)
> > 
> > diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> > index 88f6f53..b30e942 100755
> > --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> > +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
[..]
> >  			#and here we add the station
> > -			if ! wifiAddSta "$radio" "config.franken.freifunk.net" "configSta"; then
> > +			if ! wifiAddSta "$radio" "config.franken.freifunk.net" "configSta" "$staiface" ; then
> 
> das Leerzeichen vor dem ; könnte man noch eben entfernen ;)

Stimme dir da sehr zu. Ich finde das optisch auch unglaublich
unappetitlich.. ;)

Leider sind in den entsprechenden Files noch mehr davon da drin, so dass
ich das beim Applien jetzt nicht hier an der einen Stelle nachbessern
möchte.

Ich fänd es sehr gut, wenn du das anpasst und ein Patch schickst.

Tim
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 473 bytes
Beschreibung: This is a digitally signed message part
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20171102/fc6c8abf/attachment.sig>


Mehr Informationen über die Mailingliste franken-dev