[PATCH 15/20] - optimization: one redirect is enough
Steffen Pankratz
kratz00 at gmx.de
Do Jul 16 20:25:51 CEST 2015
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?
Die Nachteile bezueglich Tabs/Einrueckung hast du selber gerade aufgezeigt :)
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/a943ee66/attachment-0002.sig>
Mehr Informationen über die Mailingliste franken-dev