search

FreifunkMD / site-ffmd

Freifunk Magdeburg specific Gluon configuration
Creative Commons Zero v1.0 Universal
2 stars 12 forks source link

GLUON_REGION ist nicht definiert #95

Closed LeSpocky closed 6 years ago

LeSpocky commented 6 years ago

Die Beispieldatei docs/site-example/site.mk in gluon v2016.1.x enthält:

# Region code required for some images; supported values: us eu
GLUON_REGION ?= eu

Das fehlt bei uns. Ggf. nochmal prüfen wofür das nötig ist und welchen Wert man da übernehmen will.

johannwagner commented 6 years ago

Ist im Pull-Request von mir gefixt. Geht bei der Variable um die Sendeleistung von WLAN, wenn ich mich recht erinnere.

LeSpocky commented 6 years ago

In welchem Pull-Request?

johannwagner commented 6 years ago

https://github.com/FreifunkMD/site-ffmd/pull/93

GLUON_REGION

Some devices (at the moment the TP-Link Archer C7) contain a region code that restricts firmware installations. Set GLUON_REGION to eu or us to make the resulting images installable from the respective stock firmwares.

Aus https://gluon.readthedocs.io/en/v2017.1.x/user/getting_started.html#common-variables

LeSpocky commented 6 years ago

Ja jetzt seh ich's auch. Das gehört eigentlich in einen separaten PR und am besten bevor #93 gemerged wird.

johannwagner commented 6 years ago

Ich hab es da an der Stelle gesetzt, weil es in den Changes zur 2016.2.x noch extra erwähnt wurde, da einige Router Region Locks haben.

johannwagner commented 6 years ago

https://gluon.readthedocs.io/en/v2017.1.x/releases/v2016.2.html#site-mk

LeSpocky commented 6 years ago

Es ist schon Teil von v2016.1.x, daher denke ich, dass wir es vorher noch reinnehmen sollten: http://gluon.readthedocs.io/en/latest/releases/v2016.1.6.html

An der Stelle hätte ich gern wenigstens zwei getrennte Commits, wenn es schon im falschen PR auftaucht. 😉

penguineer commented 6 years ago

Pro zwei Commits, aber ich würde dafür keinen weiteren PR aufmachen. Das ist eine unserer Änderungen zur neuen Version. Kein Grund, das mit unnötigem Verwaltungsoverhead zu versehen.

penguineer commented 6 years ago

@johannwagner Wenn Du den Commit geteilt hast, schreib das hier mal bitte und schließe das Issue. Ich merge dann #93.

johannwagner commented 6 years ago

Commit ist auseinander gezogen. Ich halte es allerdings noch nicht für sinnvoll, die #93 zu mergen, da sie noch nicht mit der build.sh baut.

penguineer commented 6 years ago

Ah, dann schreib das bitte mal da rein. Ich schließe dieses Issue.