| Message ID | 20190525121110.1352-1-freifunk@adrianschmutzler.de |
|---|---|
| State | Accepted |
| Headers | show |
diff --git a/src/packages/fff/fff-gateway/Makefile b/src/packages/fff/fff-gateway/Makefile index 7c1dd559..0f0bbb34 100644 --- a/src/packages/fff/fff-gateway/Makefile +++ b/src/packages/fff/fff-gateway/Makefile @@ -1,8 +1,7 @@ include $(TOPDIR)/rules.mk PKG_NAME:=fff-gateway -PKG_VERSION:=1 -PKG_RELEASE:=1 +PKG_RELEASE:=2 PKG_BUILD_DIR:=$(BUILD_DIR)/fff-gateway diff --git a/src/packages/fff/fff-gateway/files/usr/sbin/configuregateway b/src/packages/fff/fff-gateway/files/usr/sbin/configuregateway index 0f040d79..3ea67e56 100755 --- a/src/packages/fff/fff-gateway/files/usr/sbin/configuregateway +++ b/src/packages/fff/fff-gateway/files/usr/sbin/configuregateway @@ -24,7 +24,7 @@ execute_subshell() { unset -f "$1" . "$script" - if type "$1" | grep -q "shell function"; then + if type "$1" | grep -q "function" ; then "$1" fi )
Wollen wir wirklich den Pkg Release für jede Änderung anheben, die wir da dran machen? .. Abgesehen von diesem Kommentar: Reviwed-by: Fabian Bläse <fabian@blaese.de> Fabian
Hallo Fabian,
ich mache das inzwischen so (außer bei Änderungen von Kommentaren oder so), auch bei meinem eigenen Zeug.
Eigentlich ist das PKG_RELEASE auch so gedacht.
Für mich persönlich hat das insbesondere den Vorteil, dass ich in meiner FW auch immer weiß, welchen Stand von dem offiziellen Zeug ich grade als Basis habe.
Wenn du also mich fragst: Ja.
Ob da du/die anderen Lust drauf haben, ist natürlich ein anderes Thema.
Für die off. Firmware selbst hat das kaum Relevanz.
Beste Grüße
Adrian
From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf Of Fabian Bläse
Sent: Samstag, 25. Mai 2019 15:08
To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken-dev@freifunk.net
Subject: Re: [PATCH] configuregateway: Only match "function" in subshell
Wollen wir wirklich den Pkg Release für jede Änderung anheben, die wir da dran machen? ..
Abgesehen von diesem Kommentar:
Reviwed-by: Fabian Bläse <fabian@blaese.de <mailto:fabian@blaese.de> >
Fabian
Verdammt, da hab ich mich wohl vertippt. Reviewed-by: Fabian Bläse <fabian@blaese.de> On 25.05.19 15:07, Fabian Bläse wrote: > Wollen wir wirklich den Pkg Release für jede Änderung anheben, die wir da dran machen? .. > > Abgesehen von diesem Kommentar: > Reviwed-by: Fabian Bläse <fabian@blaese.de> > > Fabian >
Hi Adrian, Reviewed-by: Robert Langhammer <rlanghammer@web.de> Am 25.05.19 um 14:11 schrieb Adrian Schmutzler: > When running OpenWrt master, "type <functionname>" only returns > "<functionname> is a function". Thus, to make the implementation > future-proof and since it is also working with the current state, > change the grep in execute_subshell to match "function". > > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> > --- > src/packages/fff/fff-gateway/Makefile | 3 +-- > src/packages/fff/fff-gateway/files/usr/sbin/configuregateway | 2 +- > 2 files changed, 2 insertions(+), 3 deletions(-) > > diff --git a/src/packages/fff/fff-gateway/Makefile b/src/packages/fff/fff-gateway/Makefile > index 7c1dd559..0f0bbb34 100644 > --- a/src/packages/fff/fff-gateway/Makefile > +++ b/src/packages/fff/fff-gateway/Makefile > @@ -1,8 +1,7 @@ > include $(TOPDIR)/rules.mk > > PKG_NAME:=fff-gateway > -PKG_VERSION:=1 > -PKG_RELEASE:=1 > +PKG_RELEASE:=2 > > PKG_BUILD_DIR:=$(BUILD_DIR)/fff-gateway > > diff --git a/src/packages/fff/fff-gateway/files/usr/sbin/configuregateway b/src/packages/fff/fff-gateway/files/usr/sbin/configuregateway > index 0f040d79..3ea67e56 100755 > --- a/src/packages/fff/fff-gateway/files/usr/sbin/configuregateway > +++ b/src/packages/fff/fff-gateway/files/usr/sbin/configuregateway > @@ -24,7 +24,7 @@ execute_subshell() { > unset -f "$1" > . "$script" > > - if type "$1" | grep -q "shell function"; then > + if type "$1" | grep -q "function" ; then > "$1" > fi > )
applied.
When running OpenWrt master, "type <functionname>" only returns "<functionname> is a function". Thus, to make the implementation future-proof and since it is also working with the current state, change the grep in execute_subshell to match "function". Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> --- src/packages/fff/fff-gateway/Makefile | 3 +-- src/packages/fff/fff-gateway/files/usr/sbin/configuregateway | 2 +- 2 files changed, 2 insertions(+), 3 deletions(-)