[PATCH 15/20] - optimization: one redirect is enough

[Steffen Pankratz]

On Thu, 16 Jul 2015 20:34:04 +0200
Tim Niemeyer <tim.niemeyer at mastersword.de> wrote:

> Am Donnerstag, den 16.07.2015, 20:25 +0200 schrieb Steffen Pankratz:
> > On Thu, 16 Jul 2015 19:51:34 +0200
> > Tim Niemeyer <tim.niemeyer at mastersword.de> wrote:
> > 
> > Hi Tim
> > 
> > > Am Mittwoch, den 15.07.2015, 21:48 +0200 schrieb Steffen Pankratz:
> > > > Signed-off-by: Steffen Pankratz <kratz00 at gmx.de>
> > > > ---
> > > >  buildscript | 10 ++++++----
> > > >  1 file changed, 6 insertions(+), 4 deletions(-)
> > > > 
> > > > diff --git a/buildscript b/buildscript
> > > > index 708b367..6653602 100755
> > > > --- a/buildscript
> > > > +++ b/buildscript
> > > > @@ -168,10 +168,12 @@ prebuild() {
> > > >  	#insert actual firware version informations into release file
> > > >  	version=$(git describe --tags --dirty)
> > > >  
> > > > -	echo "FIRMWARE_VERSION=\"$version\"" > $target/files/etc/firmware_release
> > > > -	echo "BUILD_DATE=\"$(date)\"" >> $target/files/etc/firmware_release
> > > > -	echo "OPENWRT_CORE_REVISION=\"${OPENWRTREV}\"" >> $target/files/etc/firmware_release
> > > > -	echo "OPENWRT_FEEDS_PACKAGES_REVISION=\"${PACKAGEREV}\"" >> $target/files/etc/firmware_release
> > > > +	{
> > > > +		echo "FIRMWARE_VERSION=\"$version\""
> > > > +		echo "BUILD_DATE=\"$(date)\""
> > > > +		echo "OPENWRT_CORE_REVISION=\"${OPENWRTREV}\""
> > > > +		echo "OPENWRT_FEEDS_PACKAGES_REVISION=\"${PACKAGEREV}\""
> > > > +	} >> $target/files/etc/firmware_release
> > 
> > > Zum einen wird das nicht drauf gehen, weil die beiden Änderungen davor
> > > noch überarbeitet werden müssen. Zum anderen: Wenn schon denn schon:
> > > cat <<EOF >> /tmp/test
> > > FIRMWARE_VERSION="$version"
> > > BUILD_DATE="$(date)"
> > > OPENWRT_CORE_REVISION="${OPENWRTREV}"
> > > OPENWRT_FEEDS_PACKAGES_REVISION="${PACKAGEREV}"
> > > EOF
> > > 
> > > Naja, das Einrücken ist dann blöde weil du in Patch 20 auf spaces gehst.
> > > Wenn man das nicht tun würde, könnte man <<-EOF nehmen und auch
> > > einrücken.
> > 
> > Wegen dem nodewatch.sh gucke ich auf jeden Fall noch mal.
> > Aber wieso sollte die EOF Loesung besser sein?
> Sieht hübscher aus.

Aussehen ist Geschmackssache und die EOF Loesung hat definitiv Nachteile.
Sowas geht nicht:
    cat <<EOF >> /tmp/test2
        moep
    EOF

DAS EOF muss am Zeilenanfang stehen, sonst -> warning: here-document at
line 1 delimited by end-of-file (wanted `EOF')
Das macht dann die Uebersichtlichkeit kaputt, da der ganze Code eingerueckt ist,
da in einer Funktion gekapselt.


Gruss
-Steffen

-- 
Hermes powered by Manjaro Linux (Linux 4.1.2)

Best regards, Steffen Pankratz.
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : nicht verfügbar
Dateityp    : application/pgp-signature
Dateigröße  : 181 bytes
Beschreibung: OpenPGP digital signature
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20150716/41156a6a/attachment-0002.sig>