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

Tim Niemeyer tim.niemeyer at mastersword.de
Do Jul 16 19:51:34 CEST 2015


Hi Steffen

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.

Tim

>  }
>  
>  build() {
> -- 
> 2.4.5
> 

-------------- 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/20150716/752e8d8c/attachment-0002.sig>


Mehr Informationen über die Mailingliste franken-dev