| Message ID | 1463236588-29393-4-git-send-email-tim@tn-x.org |
|---|---|
| State | Accepted, archived |
| Commit | 6f4c9cddba99889bf80da89f0e500f5bc97bd07e |
| Headers | show |
diff --git a/buildscript b/buildscript index 887de3d..2a7b85e 100755 --- a/buildscript +++ b/buildscript @@ -279,14 +279,23 @@ cp_firmware() { buildrelease() { if [ "$1" = "all" ];then - buildall + all=true elif [ "$1" = "fast" ];then - build fast - elif [ "$1" = "all" ] && [ "$2" = "fast" ];then - buildall fast + fast=fast + fi + + if [ "$2" = "all" ];then + all=true + elif [ "$2" = "fast" ];then + fast=fast + fi + + if [ $all ]; then + buildall $fast else - build + build $fast fi + cd bin for binary in *.bin; do md5sum "$binary" > ./"$binary".md5 @@ -458,7 +467,7 @@ case "$1" in echo "Usage: $0 $1 [all] [fast]" echo else - buildrelease "$2" + buildrelease "$2" "$3" fi ;; *)
Hi Tim, ich fände es gut, wenn man weiterhin die Möglichkeit hat zwischen "fast" und nicht "fast" wählen zu können. Grüße Jan Am Samstag, den 14.05.2016, 16:36 +0200 schrieb Tim Niemeyer: > Signed-off-by: Tim Niemeyer <tim@tn-x.org> > --- > > buildscript | 21 +++++++++++++++------ > 1 file changed, 15 insertions(+), 6 deletions(-) > > diff --git a/buildscript b/buildscript > index 887de3d..2a7b85e 100755 > --- a/buildscript > +++ b/buildscript > @@ -279,14 +279,23 @@ cp_firmware() { > > buildrelease() { > if [ "$1" = "all" ];then > - buildall > + all=true > elif [ "$1" = "fast" ];then > - build fast > - elif [ "$1" = "all" ] && [ "$2" = "fast" ];then > - buildall fast > + fast=fast > + fi > + > + if [ "$2" = "all" ];then > + all=true > + elif [ "$2" = "fast" ];then > + fast=fast > + fi > + > + if [ $all ]; then > + buildall $fast > else > - build > + build $fast > fi > + > cd bin > for binary in *.bin; do > md5sum "$binary" > ./"$binary".md5 > @@ -458,7 +467,7 @@ case "$1" in > echo "Usage: $0 $1 [all] [fast]" > echo > else > - buildrelease "$2" > + buildrelease "$2" "$3" > fi > ;; > *) > -- > 2.1.4 >
Am Samstag, den 14.05.2016, 18:08 +0200 schrieb mayosemmel: > Hi Tim, > > ich fände es gut, wenn man weiterhin die Möglichkeit hat zwischen "fast" > und nicht "fast" wählen zu können. Hat man mit dem Patch ja. Nur kann man jetzt beliebig fast und all kombinieren. Vorher wäre eine Kombination nur einer bestimmten Reihenfolge möglich gewesen. Tim > > Grüße Jan > > Am Samstag, den 14.05.2016, 16:36 +0200 schrieb Tim Niemeyer: > > Signed-off-by: Tim Niemeyer <tim@tn-x.org> > > --- > > > > buildscript | 21 +++++++++++++++------ > > 1 file changed, 15 insertions(+), 6 deletions(-) > > > > diff --git a/buildscript b/buildscript > > index 887de3d..2a7b85e 100755 > > --- a/buildscript > > +++ b/buildscript > > @@ -279,14 +279,23 @@ cp_firmware() { > > > > buildrelease() { > > if [ "$1" = "all" ];then > > - buildall > > + all=true > > elif [ "$1" = "fast" ];then > > - build fast > > - elif [ "$1" = "all" ] && [ "$2" = "fast" ];then > > - buildall fast > > + fast=fast > > + fi > > + > > + if [ "$2" = "all" ];then > > + all=true > > + elif [ "$2" = "fast" ];then > > + fast=fast > > + fi > > + > > + if [ $all ]; then > > + buildall $fast > > else > > - build > > + build $fast > > fi > > + > > cd bin > > for binary in *.bin; do > > md5sum "$binary" > ./"$binary".md5 > > @@ -458,7 +467,7 @@ case "$1" in > > echo "Usage: $0 $1 [all] [fast]" > > echo > > else > > - buildrelease "$2" > > + buildrelease "$2" "$3" > > fi > > ;; > > *) > > -- > > 2.1.4 > > >
In der Tat. Ich hatte da ein Dollar Zeichen übersehen. Die Variable kann ja leer sein. Somit auch hier: Reviewed-by: Jan Kraus <mayosemmel@gmail.com> Am Samstag, den 14.05.2016, 18:10 +0200 schrieb Tim Niemeyer: > Am Samstag, den 14.05.2016, 18:08 +0200 schrieb mayosemmel: > > Hi Tim, > > > > ich fände es gut, wenn man weiterhin die Möglichkeit hat zwischen "fast" > > und nicht "fast" wählen zu können. > Hat man mit dem Patch ja. Nur kann man jetzt beliebig fast und all > kombinieren. Vorher wäre eine Kombination nur einer bestimmten > Reihenfolge möglich gewesen. > > Tim > > > > Grüße Jan > > > > Am Samstag, den 14.05.2016, 16:36 +0200 schrieb Tim Niemeyer: > > > Signed-off-by: Tim Niemeyer <tim@tn-x.org> > > > --- > > > > > > buildscript | 21 +++++++++++++++------ > > > 1 file changed, 15 insertions(+), 6 deletions(-) > > > > > > diff --git a/buildscript b/buildscript > > > index 887de3d..2a7b85e 100755 > > > --- a/buildscript > > > +++ b/buildscript > > > @@ -279,14 +279,23 @@ cp_firmware() { > > > > > > buildrelease() { > > > if [ "$1" = "all" ];then > > > - buildall > > > + all=true > > > elif [ "$1" = "fast" ];then > > > - build fast > > > - elif [ "$1" = "all" ] && [ "$2" = "fast" ];then > > > - buildall fast > > > + fast=fast > > > + fi > > > + > > > + if [ "$2" = "all" ];then > > > + all=true > > > + elif [ "$2" = "fast" ];then > > > + fast=fast > > > + fi > > > + > > > + if [ $all ]; then > > > + buildall $fast > > > else > > > - build > > > + build $fast > > > fi > > > + > > > cd bin > > > for binary in *.bin; do > > > md5sum "$binary" > ./"$binary".md5 > > > @@ -458,7 +467,7 @@ case "$1" in > > > echo "Usage: $0 $1 [all] [fast]" > > > echo > > > else > > > - buildrelease "$2" > > > + buildrelease "$2" "$3" > > > fi > > > ;; > > > *) > > > -- > > > 2.1.4 > > > > > >
Signed-off-by: Tim Niemeyer <tim@tn-x.org> --- buildscript | 21 +++++++++++++++------ 1 file changed, 15 insertions(+), 6 deletions(-)