[PATCH] packages/fff: add new fff-dhcp package

Fabian Bläse fabian at blaese.de
Mi Jan 16 19:53:33 CET 2019


Hallo Adrian,

On 10.01.19 00:05, mail at adrianschmutzler.de wrote:
> Hallo,
> 
> ich hatte da noch zwischendrin drei Anmerkungen:
> 
>> > +
>> > +define Package/fff-dhcp
> 
> Hier und weiter unten könnte man wieder $PKG_NAME verwenden. Da das aber wohl bei allen GW-Firmware Patches nicht der Fall ist, kann ich das auch am Schluss für alle in einem separaten Patch nachholen (oder sein lassen).
Könnte man machen, ja. Hab jetzt nicht geguckt, was wir aktuell machen.
Ich würde sagen in einem extra Patch nachholen.

>> > +
>> > +uci batch <<EOF
>> > +  delete dhcp. at dnsmasq[0]
>> > +  delete dhcp.lan
>> > +  delete dhcp.wan
>> > +  add dhcp dnsmasq
> 
> Das macht eine Ausgabe. Keine Ahnung, wo die Ausgabe dann hingeschrieben wird und ob man das hier abfangen könnte.
Puh, gute Frage.
Ich bin mir recht sicher, dass sie nicht in eine Date geschrieben wird. Wahrscheinlich gibt das uci batch das tatsächlich einfach aus.

>> > +EOF
>> > +
>> > +uci commit
> 
> Hier würde "uci commit dhcp" ausreichen.
> Wegen mir direkt beim Commit oder wenn du nicht willst auch gar nicht.
Jo, das kann man machen. Das würde ich direkt beim applien machen.

Gruß
Fabian

P.S.: Dein Mailer macht komische Plaintext Mails (vermutlich schon immer) mit viel zu vielen Newlines bei Absätzen...

-------------- 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         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20190116/8c2930e9/attachment.sig>


Mehr Informationen über die Mailingliste franken-dev