[PATCH 3/7] fff-nodewatcher: style: fix some long lines

[Tobias Klaus]

Hey,

On Samstag, 14. Mai 2016 18:37:59 CEST mayosemmel wrote:
> Hi Tim,
> Sollte es hier nicht
> SYSTEM_DATA="$SYSTEM_DATA$status_text"
> heißen um Probleme mit Leerzeichen usw. vorzubeugen? Unten natürlich
> dann das selbe.
Ich hätte das sicherheitshalber auch Anführungszeichen gesetzt, allerdings 
scheint das so korrekt zu sein. Anführungszeichen sind ja eher eine Parsehilfe 
für sh. und hier geht ganz klar hervor, dass er zwei Variablen konkatenieren 
soll. Man kann die Anführungszeichen auch bei $status_text weglassen.(grade 
mit shellcheck getestet)

daher:
Reviewed-by: Tobias Klaus <tk+ff at meskal.net>

> Grüße Jan
> 
> > +	SYSTEM_DATA=$SYSTEM_DATA"$status_text"
> > +	SYSTEM_DATA=$SYSTEM_DATA"<hostname>$hostname</hostname>"
> > +	SYSTEM_DATA=$SYSTEM_DATA"${description}"
> > +	SYSTEM_DATA=$SYSTEM_DATA"${geo}"
> > +	SYSTEM_DATA=$SYSTEM_DATA"${position_comment}"
> > +	SYSTEM_DATA=$SYSTEM_DATA"${contact}"
> > +	SYSTEM_DATA=$SYSTEM_DATA"<distname>$distname</distname>"
> > +	SYSTEM_DATA=$SYSTEM_DATA"<distversion>$distversion</distversion>"
> > +	SYSTEM_DATA=$SYSTEM_DATA"$cpu"
> > +	SYSTEM_DATA=$SYSTEM_DATA"$model"
> > +	SYSTEM_DATA=$SYSTEM_DATA"$memory"
> > +	SYSTEM_DATA=$SYSTEM_DATA"$load"
> > +	SYSTEM_DATA=$SYSTEM_DATA"$uptime"
> > +	SYSTEM_DATA=$SYSTEM_DATA"<local_time>$local_time</local_time>"
> > +	SYSTEM_DATA=$SYSTEM_DATA"<batman_advanced_version>$batman_adv_version</
b
> > atman_advanced_version>"
> > +	SYSTEM_DATA=$SYSTEM_DATA"<kernel_version>$kernel_version</kernel_versio
> > n>"
> > +	SYSTEM_DATA=$SYSTEM_DATA"<fastd_version>$fastd_version</fastd_version>"
> > +	SYSTEM_DATA=$SYSTEM_DATA"<nodewatcher_version>$nodewatcher_version</nod
> > ewatcher_version>"
> > +	SYSTEM_DATA=$SYSTEM_DATA"<firmware_version>$FIRMWARE_VERSION</firmware_
> > version>"
> > +	SYSTEM_DATA=$SYSTEM_DATA"<firmware_community>$FIRMWARE_COMMUNITY</firmw
> > are_community>"
> > +	SYSTEM_DATA=$SYSTEM_DATA"<firmware_revision>$BUILD_DATE</firmware_revis
> > ion>"
> > +	SYSTEM_DATA=$SYSTEM_DATA"<openwrt_core_revision>$OPENWRT_CORE_REVISION<
> > /openwrt_core_revision>"
> > +	SYSTEM_DATA=$SYSTEM_DATA"<openwrt_feeds_packages_revision>$OPENWRT_FEED
> > S_PACKAGES_REVISION</openwrt_feeds_packages_revision>"> 
> >  	err "`date`: Collecting information from network interfaces"
> > 
> > @@ -213,7 +237,15 @@ crawl() {
> > 
> >  	err "`date`: Putting all information into a XML-File and save it at
> >  	"$SCRIPT_DATA_FILE> 
> > -	DATA="<?xml version='1.0'
> > standalone='yes'?><data><system_data>$SYSTEM_DATA</system_data><interface
> > _data>$interface_data</interface_data><batman_adv_interfaces>$BATMAN_ADV_I
> > NTERFACES</batman_adv_interfaces><batman_adv_originators>$batman_adv_origi
> > nators</batman_adv_originators><batman_adv_gateway_mode>$batman_adv_gatewa
> > y_mode</batman_adv_gateway_mode><batman_adv_gateway_list>$batman_adv_gatew
> > ay_list</batman_adv_gateway_list><client_count>$client_count</client_count
> > ></data>" +	DATA="<?xml version='1.0' standalone='yes'?><data>"
> > +	DATA=$DATA"<system_data>$SYSTEM_DATA</system_data>"
> > +	DATA=$DATA"<interface_data>$interface_data</interface_data>"
> > +	DATA=$DATA"<batman_adv_interfaces>$BATMAN_ADV_INTERFACES</
batman_adv_int
> > erfaces>"
> > +	DATA=$DATA"<batman_adv_originators>$batman_adv_originators</batman_adv_
> > originators>"
> > +	DATA=$DATA"<batman_adv_gateway_mode>$batman_adv_gateway_mode</batman_ad
> > v_gateway_mode>"
> > +	DATA=$DATA"<batman_adv_gateway_list>$batman_adv_gateway_list</batman_ad
> > v_gateway_list>" +	DATA=$DATA"<client_count>$client_count</client_count>"
> > +	DATA=$DATA"</data>"
> > 
> >  	#write data to hxml file that provides the data on httpd
> >  	SCRIPT_DATA_DIR=$(dirname $SCRIPT_DATA_FILE)

-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 819 bytes
Beschreibung: This is a digitally signed message part.
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20160527/1b6ba476/attachment-0002.sig>