[PATCH 1/2] fff-gateway: add package

[Fabian Bläse]

Hey,

On 15.03.19 21:42, Tim Niemeyer wrote:
> Moin
> 
> Am Freitag, den 15.03.2019, 21:22 +0100 schrieb Fabian Bläse:
>> +define Package/fff-gateway
>> +    SECTION:=base
>> +    CATEGORY:=Freifunk
>> +    TITLE:= Freifunk-Franken gateway configuration
>> +    URL:=https://www.freifunk-franken.de
>> +    DEPENDS:=+fff-network
> Wieso depended das darauf?
Das ist so, weil in dieses Package noch ein Script für die Netzwerkkonfiguration dazu kommt, welches die gerätespezifischen Konfigurationen und Funktionen aus dem fff-network Package verwendet.
Aktuell ist es auch generell so, dass bei ersten Boot zunächst die klassischen fff-network Skripte ausgeführt werden, woraufhin der User dann die gateway-Skripte verwenden kann.
Vielleicht wäre es besser, für die Geräte eine default gateway-config abzulegen und beim ersten Boot danach zu konfigurieren. Dann würde man das fff-network los werden.

>> +#load uci functions
>> +. /lib/functions.sh
>> +. /usr/share/libubox/jshn.sh
> Wird das hier verwendet?
Stimmt, das gehört hier nicht hin. Gilt auch für die anderen von dir entdeckten sources.

>> +configure() {
>>> +	# ensure client config exists
>>>> +	if ! uci get gateway. at client[0] > /dev/null; then
>>> +		echo "ERROR: No client config or error parsing config!"
>>> +		exit 1
>> +	fi
> 
> Warum wird bei configure geprüft ob es dieses client config gibt und
> bei allen andere nicht?
Auch das ist wohl ein Überrest. Was bei so einem grundlegenden Umbau alles kaputt geht.. O.o
Hier sollte man wohl einfach - wenn überhaupt - nur überprüfen, ob eine Gateway Konfiguration existiert.

Gruß
Fabian

-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 833 bytes
Beschreibung: OpenPGP digital signature
URL         : <https://{'listname': 'franken-dev-freifunk.net', 'hostname': 'lists.freifunk.net'}/pipermail/franken-dev-freifunk.net/attachments/20190315/254547b6/attachment.sig>