Message ID | 20190930201217.1394447-1-fabian@blaese.de |
---|---|
State | Superseded |
Headers | show |
diff --git a/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher b/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher index 09f6843..0d4ef80 100755 --- a/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher +++ b/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher @@ -109,6 +109,9 @@ crawl() { fastd_version="<fastd_version>$(/usr/bin/fastd -v | awk '{ print $2 }')</fastd_version>" fi nodewatcher_version=$SCRIPT_VERSION + if [ -x /usr/sbin/babeld ]; then + babel_version="<babel_version>$(/usr/bin/babeld -V 2>&1)</babel_version>" + fi if [ -f "$SCRIPT_STATUS_FILE" ]; then status_text="<status_text>$(cat "$SCRIPT_STATUS_FILE")</status_text>" @@ -160,6 +163,7 @@ crawl() { SYSTEM_DATA=$SYSTEM_DATA"<kernel_version>$kernel_version</kernel_version>" SYSTEM_DATA=$SYSTEM_DATA"$fastd_version" SYSTEM_DATA=$SYSTEM_DATA"<nodewatcher_version>$nodewatcher_version</nodewatcher_version>" + SYSTEM_DATA=$SYSTEM_DATA"$babel_version" SYSTEM_DATA=$SYSTEM_DATA"<firmware_version>$FIRMWARE_VERSION</firmware_version>" SYSTEM_DATA=$SYSTEM_DATA"<firmware_revision>$BUILD_DATE</firmware_revision>" SYSTEM_DATA=$SYSTEM_DATA"<openwrt_core_revision>$OPENWRT_CORE_REVISION</openwrt_core_revision>"
Bitte PKG_RELEASE und SCRIPT_VERSION bumpen. Ist das eigentlich normal, dass die Version ins stderr kommt? Sonst schicke ich da mal nen Patch zu babel? Grüße Adrian > -----Original Message----- > From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf > Of Fabian Bläse > Sent: Montag, 30. September 2019 22:12 > To: franken-dev@freifunk.net > Subject: [PATCH] nodewatcher: Send babeld version > > Signed-off-by: Fabian Bläse <fabian@blaese.de> > --- > src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher > b/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher > index 09f6843..0d4ef80 100755 > --- a/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher > +++ b/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher > @@ -109,6 +109,9 @@ crawl() { > fastd_version="<fastd_version>$(/usr/bin/fastd -v | awk '{ print $2 > }')</fastd_version>" > fi > nodewatcher_version=$SCRIPT_VERSION > + if [ -x /usr/sbin/babeld ]; then > + babel_version="<babel_version>$(/usr/bin/babeld -V > 2>&1)</babel_version>" > + fi > > if [ -f "$SCRIPT_STATUS_FILE" ]; then > status_text="<status_text>$(cat > "$SCRIPT_STATUS_FILE")</status_text>" > @@ -160,6 +163,7 @@ crawl() { > > SYSTEM_DATA=$SYSTEM_DATA"<kernel_version>$kernel_version</kernel_ > version>" > SYSTEM_DATA=$SYSTEM_DATA"$fastd_version" > > SYSTEM_DATA=$SYSTEM_DATA"<nodewatcher_version>$nodewatcher_ver > sion</nodewatcher_version>" > + SYSTEM_DATA=$SYSTEM_DATA"$babel_version" > > SYSTEM_DATA=$SYSTEM_DATA"<firmware_version>$FIRMWARE_VERSION > </firmware_version>" > > SYSTEM_DATA=$SYSTEM_DATA"<firmware_revision>$BUILD_DATE</firmwa > re_revision>" > > SYSTEM_DATA=$SYSTEM_DATA"<openwrt_core_revision>$OPENWRT_COR > E_REVISION</openwrt_core_revision>" > -- > 2.23.0
Hi, die Babelversion wird übrigens auch direkt ausgespuckt, wenn man sich ans configuration interface hängt. Vielleicht wär das also besser im awk script aufgebhoben? Grüße, lemmi On 30.09.19 22:12, Fabian Bläse wrote: > Signed-off-by: Fabian Bläse <fabian@blaese.de> > --- > src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher b/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher > index 09f6843..0d4ef80 100755 > --- a/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher > +++ b/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher > @@ -109,6 +109,9 @@ crawl() { > fastd_version="<fastd_version>$(/usr/bin/fastd -v | awk '{ print $2 }')</fastd_version>" > fi > nodewatcher_version=$SCRIPT_VERSION > + if [ -x /usr/sbin/babeld ]; then > + babel_version="<babel_version>$(/usr/bin/babeld -V 2>&1)</babel_version>" > + fi > > if [ -f "$SCRIPT_STATUS_FILE" ]; then > status_text="<status_text>$(cat "$SCRIPT_STATUS_FILE")</status_text>" > @@ -160,6 +163,7 @@ crawl() { > SYSTEM_DATA=$SYSTEM_DATA"<kernel_version>$kernel_version</kernel_version>" > SYSTEM_DATA=$SYSTEM_DATA"$fastd_version" > SYSTEM_DATA=$SYSTEM_DATA"<nodewatcher_version>$nodewatcher_version</nodewatcher_version>" > + SYSTEM_DATA=$SYSTEM_DATA"$babel_version" > SYSTEM_DATA=$SYSTEM_DATA"<firmware_version>$FIRMWARE_VERSION</firmware_version>" > SYSTEM_DATA=$SYSTEM_DATA"<firmware_revision>$BUILD_DATE</firmware_revision>" > SYSTEM_DATA=$SYSTEM_DATA"<openwrt_core_revision>$OPENWRT_CORE_REVISION</openwrt_core_revision>"
Hmm, das mit den Versionen.. Ganz korrekterweise müsste man das schon bei jeder Änderungen bumpen. Aber da wir die Pakete ja nur selbst verwenden (wobei.. ist das wirklich so) hab ich das aus $Faulheit immer übersprungen und hätte das dann gesammelt gemacht. On 30.09.19 22:52, mail@adrianschmutzler.de wrote: > Bitte PKG_RELEASE und SCRIPT_VERSION bumpen. > > Ist das eigentlich normal, dass die Version ins stderr kommt? Hier gehen die Meinungen etwas auseinander.. Der GNU Coding Standard [1] sagt stdout. Allerdings sagt der auch "--version", nicht "-V". POSIX sagt: "standard output (for writing conventional output) and standard error (for writing diagnostic output)" [2] Versionsnummer fällt imho eher unter "diagnostic output", da das ja keine Nutzdaten des Programms ausgibt.. Shell-Prompts sind da beispielsweise auch so eine Sache. bash schreibt den Prompt auf stderr, dash schreibt den Prompt auf stdout.. Gruß Fabian [1] http://www.gnu.org/prep/standards/html_node/_002d_002dversion.html#g_t_002d_002dversion [2] https://pubs.opengroup.org/onlinepubs/9699919799/functions/stdin.html
Hey lemmi, das geht aber nur, wenn babel läuft. Außerdem würde das den awk weiter verkomplizieren. In so fern würde ich die getrennte Lösung eher präferieren. Gruß Fabian On 30.09.19 22:59, lemmi wrote: > Hi, > > die Babelversion wird übrigens auch direkt ausgespuckt, wenn man sich ans configuration interface hängt. Vielleicht wär das also besser im awk script aufgebhoben? > > Grüße, > > lemmi > > On 30.09.19 22:12, Fabian Bläse wrote: >> Signed-off-by: Fabian Bläse <fabian@blaese.de> >> --- >> src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher | 4 ++++ >> 1 file changed, 4 insertions(+) >> >> diff --git a/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher b/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher >> index 09f6843..0d4ef80 100755 >> --- a/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher >> +++ b/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher >> @@ -109,6 +109,9 @@ crawl() { >> fastd_version="<fastd_version>$(/usr/bin/fastd -v | awk '{ print $2 }')</fastd_version>" >> fi >> nodewatcher_version=$SCRIPT_VERSION >> + if [ -x /usr/sbin/babeld ]; then >> + babel_version="<babel_version>$(/usr/bin/babeld -V 2>&1)</babel_version>" >> + fi >> if [ -f "$SCRIPT_STATUS_FILE" ]; then >> status_text="<status_text>$(cat "$SCRIPT_STATUS_FILE")</status_text>" >> @@ -160,6 +163,7 @@ crawl() { >> SYSTEM_DATA=$SYSTEM_DATA"<kernel_version>$kernel_version</kernel_version>" >> SYSTEM_DATA=$SYSTEM_DATA"$fastd_version" >> SYSTEM_DATA=$SYSTEM_DATA"<nodewatcher_version>$nodewatcher_version</nodewatcher_version>" >> + SYSTEM_DATA=$SYSTEM_DATA"$babel_version" >> SYSTEM_DATA=$SYSTEM_DATA"<firmware_version>$FIRMWARE_VERSION</firmware_version>" >> SYSTEM_DATA=$SYSTEM_DATA"<firmware_revision>$BUILD_DATE</firmware_revision>" >> SYSTEM_DATA=$SYSTEM_DATA"<openwrt_core_revision>$OPENWRT_CORE_REVISION</openwrt_core_revision>" > >
On 30.09.19 23:35, Fabian Bläse wrote: > Hey lemmi, > > das geht aber nur, wenn babel läuft. Guter punkt. > Außerdem würde das den awk weiter verkomplizieren. > > In so fern würde ich die getrennte Lösung eher präferieren. Vermutlich hast du in dem Fall recht. Habe nur gerade das Feature in mein monitoring Teil eingebaut und dort war es praktisch. > Gruß > Fabian > > On 30.09.19 22:59, lemmi wrote: >> Hi, >> >> die Babelversion wird übrigens auch direkt ausgespuckt, wenn man sich ans configuration interface hängt. Vielleicht wär das also besser im awk script aufgebhoben? >> >> Grüße, >> >> lemmi >> >> On 30.09.19 22:12, Fabian Bläse wrote: >>> Signed-off-by: Fabian Bläse <fabian@blaese.de> >>> --- >>> src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher | 4 ++++ >>> 1 file changed, 4 insertions(+) >>> >>> diff --git a/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher b/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher >>> index 09f6843..0d4ef80 100755 >>> --- a/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher >>> +++ b/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher >>> @@ -109,6 +109,9 @@ crawl() { >>> fastd_version="<fastd_version>$(/usr/bin/fastd -v | awk '{ print $2 }')</fastd_version>" >>> fi >>> nodewatcher_version=$SCRIPT_VERSION >>> + if [ -x /usr/sbin/babeld ]; then >>> + babel_version="<babel_version>$(/usr/bin/babeld -V 2>&1)</babel_version>" >>> + fi >>> if [ -f "$SCRIPT_STATUS_FILE" ]; then >>> status_text="<status_text>$(cat "$SCRIPT_STATUS_FILE")</status_text>" >>> @@ -160,6 +163,7 @@ crawl() { >>> SYSTEM_DATA=$SYSTEM_DATA"<kernel_version>$kernel_version</kernel_version>" >>> SYSTEM_DATA=$SYSTEM_DATA"$fastd_version" >>> SYSTEM_DATA=$SYSTEM_DATA"<nodewatcher_version>$nodewatcher_version</nodewatcher_version>" >>> + SYSTEM_DATA=$SYSTEM_DATA"$babel_version" >>> SYSTEM_DATA=$SYSTEM_DATA"<firmware_version>$FIRMWARE_VERSION</firmware_version>" >>> SYSTEM_DATA=$SYSTEM_DATA"<firmware_revision>$BUILD_DATE</firmware_revision>" >>> SYSTEM_DATA=$SYSTEM_DATA"<openwrt_core_revision>$OPENWRT_CORE_REVISION</openwrt_core_revision>" >>
hi ich glaube das FabLab ist aktuell am nähesten am aktuellen Master oder? Daher von dort der Test: On 30.09.19 22:12, Fabian Bläse wrote: > Signed-off-by: Fabian Bläse <fabian@blaese.de> > --- > src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher b/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher > index 09f6843..0d4ef80 100755 > --- a/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher > +++ b/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher > @@ -109,6 +109,9 @@ crawl() { > fastd_version="<fastd_version>$(/usr/bin/fastd -v | awk '{ print $2 }')</fastd_version>" > fi > nodewatcher_version=$SCRIPT_VERSION > + if [ -x /usr/sbin/babeld ]; then > + babel_version="<babel_version>$(/usr/bin/babeld -V 2>&1)</babel_version>" root@fln:~# /usr/bin/babeld -V 2>&1 -ash: /usr/bin/babeld: not found root@fln:~# ls /usr/sbin/babeld /usr/sbin/babeld und jetzt vorm absenden seh ich sogar das es sich mitm if komplett widerspricht ;) Gruß Christian > + fi > > if [ -f "$SCRIPT_STATUS_FILE" ]; then > status_text="<status_text>$(cat "$SCRIPT_STATUS_FILE")</status_text>" > @@ -160,6 +163,7 @@ crawl() { > SYSTEM_DATA=$SYSTEM_DATA"<kernel_version>$kernel_version</kernel_version>" > SYSTEM_DATA=$SYSTEM_DATA"$fastd_version" > SYSTEM_DATA=$SYSTEM_DATA"<nodewatcher_version>$nodewatcher_version</nodewatcher_version>" > + SYSTEM_DATA=$SYSTEM_DATA"$babel_version" > SYSTEM_DATA=$SYSTEM_DATA"<firmware_version>$FIRMWARE_VERSION</firmware_version>" > SYSTEM_DATA=$SYSTEM_DATA"<firmware_revision>$BUILD_DATE</firmware_revision>" > SYSTEM_DATA=$SYSTEM_DATA"<openwrt_core_revision>$OPENWRT_CORE_REVISION</openwrt_core_revision>" >
Stimmt. Ich war mir sicher, das gefixt zu haben. Aber ich habs wohl vergessen. -> siehe v2 Gruß Fabian On 01.10.19 20:55, Christian Dresel wrote: > hi > > ich glaube das FabLab ist aktuell am nähesten am aktuellen Master oder? > Daher von dort der Test: > > On 30.09.19 22:12, Fabian Bläse wrote: >> Signed-off-by: Fabian Bläse <fabian@blaese.de> >> --- >> src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher | 4 ++++ >> 1 file changed, 4 insertions(+) >> >> diff --git a/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher b/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher >> index 09f6843..0d4ef80 100755 >> --- a/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher >> +++ b/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher >> @@ -109,6 +109,9 @@ crawl() { >> fastd_version="<fastd_version>$(/usr/bin/fastd -v | awk '{ print $2 }')</fastd_version>" >> fi >> nodewatcher_version=$SCRIPT_VERSION >> + if [ -x /usr/sbin/babeld ]; then >> + babel_version="<babel_version>$(/usr/bin/babeld -V 2>&1)</babel_version>" > > root@fln:~# /usr/bin/babeld -V 2>&1 > -ash: /usr/bin/babeld: not found > root@fln:~# ls /usr/sbin/babeld > /usr/sbin/babeld > > und jetzt vorm absenden seh ich sogar das es sich mitm if komplett > widerspricht ;) > > Gruß > > Christian > >> + fi >> >> if [ -f "$SCRIPT_STATUS_FILE" ]; then >> status_text="<status_text>$(cat "$SCRIPT_STATUS_FILE")</status_text>" >> @@ -160,6 +163,7 @@ crawl() { >> SYSTEM_DATA=$SYSTEM_DATA"<kernel_version>$kernel_version</kernel_version>" >> SYSTEM_DATA=$SYSTEM_DATA"$fastd_version" >> SYSTEM_DATA=$SYSTEM_DATA"<nodewatcher_version>$nodewatcher_version</nodewatcher_version>" >> + SYSTEM_DATA=$SYSTEM_DATA"$babel_version" >> SYSTEM_DATA=$SYSTEM_DATA"<firmware_version>$FIRMWARE_VERSION</firmware_version>" >> SYSTEM_DATA=$SYSTEM_DATA"<firmware_revision>$BUILD_DATE</firmware_revision>" >> SYSTEM_DATA=$SYSTEM_DATA"<openwrt_core_revision>$OPENWRT_CORE_REVISION</openwrt_core_revision>" >>
Signed-off-by: Fabian Bläse <fabian@blaese.de> --- src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher | 4 ++++ 1 file changed, 4 insertions(+)