| Message ID | 20190203164850.1288-2-freifunk@adrianschmutzler.de |
|---|---|
| State | Accepted |
| Headers | show |
diff --git a/src/packages/fff/tunneldigger/Makefile b/src/packages/fff/tunneldigger/Makefile index d27827b6..45f5bcee 100644 --- a/src/packages/fff/tunneldigger/Makefile +++ b/src/packages/fff/tunneldigger/Makefile @@ -41,7 +41,7 @@ define Package/$(PKG_NAME)/install endef define Package/$(PKG_NAME)/conffiles -/etc/config/tunneldigger + /etc/config/tunneldigger endef $(eval $(call BuildPackage,$(PKG_NAME)))
Hi Warum ist das ein zweiter Patch? Reviewed-by: Tim Niemeyer <tim@tn-x.org> Tim On Sun, 2019-02-03 at 17:48 +0100, Adrian Schmutzler wrote: > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> > --- > src/packages/fff/tunneldigger/Makefile | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/packages/fff/tunneldigger/Makefile > b/src/packages/fff/tunneldigger/Makefile > index d27827b6..45f5bcee 100644 > --- a/src/packages/fff/tunneldigger/Makefile > +++ b/src/packages/fff/tunneldigger/Makefile > @@ -41,7 +41,7 @@ define Package/$(PKG_NAME)/install > endef > > define Package/$(PKG_NAME)/conffiles > -/etc/config/tunneldigger > + /etc/config/tunneldigger > endef > > $(eval $(call BuildPackage,$(PKG_NAME)))
Hallo Tim, im Patch 1/2 habe ich Spaces durch Tabs ersetzt. Im Patch 2/2 war vorher KEINE Einrückung vorhanden, ich habe also nicht Whitespaces ersetzt, sondern ein Tab neu hingemacht. Wenn das Absicht war und durch den Patch kaputt geht, dann will ich das einzeln reverten können. Grüße Adrian From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf Of Tim Niemeyer Sent: Sonntag, 3. Februar 2019 18:01 To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken-dev@freifunk.net Subject: Re: [PATCH 2/2] tunneldigger: Fix indentation for conffile in Makefile Hi Warum ist das ein zweiter Patch? Reviewed-by: Tim Niemeyer <tim@tn-x.org <mailto:tim@tn-x.org> > Tim On Sun, 2019-02-03 at 17:48 +0100, Adrian Schmutzler wrote: > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de <mailto:freifunk@adrianschmutzler.de> > > --- > src/packages/fff/tunneldigger/Makefile | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/packages/fff/tunneldigger/Makefile > b/src/packages/fff/tunneldigger/Makefile > index d27827b6..45f5bcee 100644 > --- a/src/packages/fff/tunneldigger/Makefile > +++ b/src/packages/fff/tunneldigger/Makefile > @@ -41,7 +41,7 @@ define Package/$(PKG_NAME)/install > endef > > define Package/$(PKG_NAME)/conffiles > -/etc/config/tunneldigger > + /etc/config/tunneldigger > endef > > $(eval $(call BuildPackage,$(PKG_NAME)))
Hi On Sun, 2019-02-03 at 18:02 +0100, mail@adrianschmutzler.de wrote: > Hallo Tim, > > im Patch 1/2 habe ich Spaces durch Tabs ersetzt. > > Im Patch 2/2 war vorher KEINE Einrückung vorhanden, ich habe also > nicht Whitespaces ersetzt, sondern ein Tab neu hingemacht. > > Wenn das Absicht war und durch den Patch kaputt geht, dann will ich > das einzeln reverten können. Ah.. Ja, das könnte in der Tat ein wichtiger Punkt sein. Kannst du es testen vorm Applien. Tim > Grüße > > Adrian > > From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf > Of Tim Niemeyer > Sent: Sonntag, 3. Februar 2019 18:01 > To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken-dev@fre > ifunk.net > Subject: Re: [PATCH 2/2] tunneldigger: Fix indentation for conffile > in Makefile > > Hi > Warum ist das ein zweiter Patch? > Reviewed-by: Tim Niemeyer <tim@tn-x.org> > Tim > On Sun, 2019-02-03 at 17:48 +0100, Adrian Schmutzler wrote: > > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> > > --- > > src/packages/fff/tunneldigger/Makefile | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/src/packages/fff/tunneldigger/Makefile > > b/src/packages/fff/tunneldigger/Makefile > > index d27827b6..45f5bcee 100644 > > --- a/src/packages/fff/tunneldigger/Makefile > > +++ b/src/packages/fff/tunneldigger/Makefile > > @@ -41,7 +41,7 @@ define Package/$(PKG_NAME)/install > > endef > > > > define Package/$(PKG_NAME)/conffiles > > -/etc/config/tunneldigger > > + /etc/config/tunneldigger > > endef > > > > $(eval $(call BuildPackage,$(PKG_NAME)))
Ja, ich habe das jetzt mal alles ungetestet auf die Liste geworfen und werde jetzt mal ein Build mit allen Änderungen durchlaufen lassen. From: Tim Niemeyer [mailto:tim@tn-x.org] Sent: Sonntag, 3. Februar 2019 18:04 To: mail@adrianschmutzler.de; franken-dev@freifunk.net Subject: Re: [PATCH 2/2] tunneldigger: Fix indentation for conffile in Makefile Hi On Sun, 2019-02-03 at 18:02 +0100, mail@adrianschmutzler.de <mailto:mail@adrianschmutzler.de> wrote: > Hallo Tim, > > im Patch 1/2 habe ich Spaces durch Tabs ersetzt. > > Im Patch 2/2 war vorher KEINE Einrückung vorhanden, ich habe also > nicht Whitespaces ersetzt, sondern ein Tab neu hingemacht. > > Wenn das Absicht war und durch den Patch kaputt geht, dann will ich > das einzeln reverten können. Ah.. Ja, das könnte in der Tat ein wichtiger Punkt sein. Kannst du es testen vorm Applien. Tim > Grüße > > Adrian > > From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf > Of Tim Niemeyer > Sent: Sonntag, 3. Februar 2019 18:01 > To: Adrian Schmutzler <freifunk@adrianschmutzler.de <mailto:freifunk@adrianschmutzler.de> >; franken-dev@fre > ifunk.net > Subject: Re: [PATCH 2/2] tunneldigger: Fix indentation for conffile > in Makefile > > Hi > Warum ist das ein zweiter Patch? > Reviewed-by: Tim Niemeyer <tim@tn-x.org <mailto:tim@tn-x.org> > > Tim > On Sun, 2019-02-03 at 17:48 +0100, Adrian Schmutzler wrote: > > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de <mailto:freifunk@adrianschmutzler.de> > > > --- > > src/packages/fff/tunneldigger/Makefile | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/src/packages/fff/tunneldigger/Makefile > > b/src/packages/fff/tunneldigger/Makefile > > index d27827b6..45f5bcee 100644 > > --- a/src/packages/fff/tunneldigger/Makefile > > +++ b/src/packages/fff/tunneldigger/Makefile > > @@ -41,7 +41,7 @@ define Package/$(PKG_NAME)/install > > endef > > > > define Package/$(PKG_NAME)/conffiles > > -/etc/config/tunneldigger > > + /etc/config/tunneldigger > > endef > > > > $(eval $(call BuildPackage,$(PKG_NAME)))
Hallo, ich habe mal die Firmware mit allen Patches von heute durchgebaut. Das klappt soweit super, keine Crashs und korrekte Änderung der Namen: fff-node-20180304-alpha-95-gc64ade6e-ar71xx-t-tl-wdr3500-v1-factory.bin Das einzige Problem ist: Die echten tiny-Devices werden nicht gebaut: adsc@buildfff:/data/buildfff$ ls ./build/ar71xx/bin/targets/ar71xx/tiny/ config.seed openwrt-ar71xx-tiny-archer-c25-v1-squashfs-factory.bin openwrt-ar71xx-tiny-archer-c25-v1-squashfs-sysupgrade.bin openwrt-ar71xx-tiny-archer-c60-v1-squashfs-factory.bin openwrt-ar71xx-tiny-archer-c60-v1-squashfs-sysupgrade.bin openwrt-ar71xx-tiny-archer-c7-v2-squashfs-factory.bin openwrt-ar71xx-tiny-archer-c7-v2-squashfs-factory-eu.bin openwrt-ar71xx-tiny-archer-c7-v2-squashfs-factory-us.bin openwrt-ar71xx-tiny-archer-c7-v2-squashfs-sysupgrade.bin openwrt-ar71xx-tiny-cpe210-220-v1-squashfs-factory.bin openwrt-ar71xx-tiny-cpe210-220-v1-squashfs-sysupgrade.bin openwrt-ar71xx-tiny-cpe210-v2-squashfs-factory.bin openwrt-ar71xx-tiny-cpe210-v2-squashfs-sysupgrade.bin openwrt-ar71xx-tiny-cpe210-v3-squashfs-factory.bin openwrt-ar71xx-tiny-cpe210-v3-squashfs-sysupgrade.bin openwrt-ar71xx-tiny-cpe510-520-v1-squashfs-factory.bin openwrt-ar71xx-tiny-cpe510-520-v1-squashfs-sysupgrade.bin openwrt-ar71xx-tiny-gl-ar150-squashfs-sysupgrade.bin openwrt-ar71xx-tiny.manifest openwrt-ar71xx-tiny-root.squashfs openwrt-ar71xx-tiny-tl-mr3020-v1-squashfs-factory.bin openwrt-ar71xx-tiny-tl-mr3020-v1-squashfs-sysupgrade.bin openwrt-ar71xx-tiny-tl-wdr3500-v1-squashfs-factory.bin openwrt-ar71xx-tiny-tl-wdr3500-v1-squashfs-sysupgrade.bin openwrt-ar71xx-tiny-tl-wdr3600-v1-squashfs-factory.bin openwrt-ar71xx-tiny-tl-wdr3600-v1-squashfs-sysupgrade.bin openwrt-ar71xx-tiny-tl-wdr4300-v1-squashfs-factory.bin openwrt-ar71xx-tiny-tl-wdr4300-v1-squashfs-sysupgrade.bin openwrt-ar71xx-tiny-tl-wdr4310-v1-squashfs-factory.bin openwrt-ar71xx-tiny-tl-wdr4310-v1-squashfs-sysupgrade.bin openwrt-ar71xx-tiny-tl-wr1043nd-v1-squashfs-factory.bin openwrt-ar71xx-tiny-tl-wr1043nd-v1-squashfs-sysupgrade.bin openwrt-ar71xx-tiny-tl-wr1043nd-v2-squashfs-factory.bin openwrt-ar71xx-tiny-tl-wr1043nd-v2-squashfs-sysupgrade.bin openwrt-ar71xx-tiny-tl-wr1043nd-v3-squashfs-factory.bin openwrt-ar71xx-tiny-tl-wr1043nd-v3-squashfs-sysupgrade.bin openwrt-ar71xx-tiny-tl-wr1043nd-v4-squashfs-factory.bin openwrt-ar71xx-tiny-tl-wr1043nd-v4-squashfs-sysupgrade.bin openwrt-ar71xx-tiny-tl-wr1043n-v5-squashfs-factory.bin openwrt-ar71xx-tiny-tl-wr1043n-v5-squashfs-sysupgrade.bin openwrt-ar71xx-tiny-tl-wr842n-v2-squashfs-factory.bin openwrt-ar71xx-tiny-tl-wr842n-v2-squashfs-sysupgrade.bin openwrt-ar71xx-tiny-ubnt-bullet-m-squashfs-factory.bin openwrt-ar71xx-tiny-ubnt-bullet-m-squashfs-sysupgrade.bin openwrt-ar71xx-tiny-ubnt-loco-m-xw-squashfs-factory.bin openwrt-ar71xx-tiny-ubnt-loco-m-xw-squashfs-sysupgrade.bin openwrt-ar71xx-tiny-ubnt-nano-m-squashfs-factory.bin openwrt-ar71xx-tiny-ubnt-nano-m-squashfs-sysupgrade.bin openwrt-ar71xx-tiny-ubnt-unifiac-lite-squashfs-sysupgrade.bin openwrt-ar71xx-tiny-ubnt-unifi-squashfs-factory.bin openwrt-ar71xx-tiny-ubnt-unifi-squashfs-sysupgrade.bin openwrt-ar71xx-tiny-uImage-lzma.bin openwrt-ar71xx-tiny-vmlinux.bin openwrt-ar71xx-tiny-vmlinux.elf openwrt-ar71xx-tiny-vmlinux.lzma openwrt-ar71xx-tiny-vmlinux-lzma.elf packages sha256sums Ich wüsste jetzt nicht, was die Patches von heute darauf für einen Einfluss haben. Kann es sein, dass das 18.06.2 da was an eurem Hack kaputt gemacht hat? Grüße Adrian From: Tim Niemeyer [mailto:tim@tn-x.org] Sent: Sonntag, 3. Februar 2019 18:04 To: mail@adrianschmutzler.de; franken-dev@freifunk.net Subject: Re: [PATCH 2/2] tunneldigger: Fix indentation for conffile in Makefile Hi On Sun, 2019-02-03 at 18:02 +0100, mail@adrianschmutzler.de <mailto:mail@adrianschmutzler.de> wrote: > Hallo Tim, > > im Patch 1/2 habe ich Spaces durch Tabs ersetzt. > > Im Patch 2/2 war vorher KEINE Einrückung vorhanden, ich habe also > nicht Whitespaces ersetzt, sondern ein Tab neu hingemacht. > > Wenn das Absicht war und durch den Patch kaputt geht, dann will ich > das einzeln reverten können. Ah.. Ja, das könnte in der Tat ein wichtiger Punkt sein. Kannst du es testen vorm Applien. Tim > Grüße > > Adrian > > From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf > Of Tim Niemeyer > Sent: Sonntag, 3. Februar 2019 18:01 > To: Adrian Schmutzler <freifunk@adrianschmutzler.de <mailto:freifunk@adrianschmutzler.de> >; franken-dev@fre > ifunk.net > Subject: Re: [PATCH 2/2] tunneldigger: Fix indentation for conffile > in Makefile > > Hi > Warum ist das ein zweiter Patch? > Reviewed-by: Tim Niemeyer <tim@tn-x.org <mailto:tim@tn-x.org> > > Tim > On Sun, 2019-02-03 at 17:48 +0100, Adrian Schmutzler wrote: > > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de <mailto:freifunk@adrianschmutzler.de> > > > --- > > src/packages/fff/tunneldigger/Makefile | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/src/packages/fff/tunneldigger/Makefile > > b/src/packages/fff/tunneldigger/Makefile > > index d27827b6..45f5bcee 100644 > > --- a/src/packages/fff/tunneldigger/Makefile > > +++ b/src/packages/fff/tunneldigger/Makefile > > @@ -41,7 +41,7 @@ define Package/$(PKG_NAME)/install > > endef > > > > define Package/$(PKG_NAME)/conffiles > > -/etc/config/tunneldigger > > + /etc/config/tunneldigger > > endef > > > > $(eval $(call BuildPackage,$(PKG_NAME)))
Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> --- src/packages/fff/tunneldigger/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)