It appears that when the subshell has a ||, the 'set -e' within
subshell gets invalidated.

This will work as expected:

 ( set -e; false; echo "should not get here" )

While this will not work as expected:

 ( set -e; false; echo "should not get here" ) || false

We resolve it by using $? to detect the status of subshell. We also let
the exitcode indicate how many packages that failed.

While here we also refactor it so most of the loop happens within the
subshell. This lets us set (or increase) rc variable once, and it
reduces number of forks which gives slightly better performance.

We actually want be able to re-run abump without needing reset the
pkgver in case we had to fix things.

This reverts commit 8198ded8.

Conflicts:
	abump.in

Otherwise abuild-sign will end with error

Jim Pryor authored 11 years ago and Natanael Copa committed 11 years ago

This permits introducing branch-specific modifications to abuild.conf settings.
Any settings there override /etc/abuild.conf and $HOME/.abuild/abuild.conf.

Jim Pryor authored 11 years ago and Natanael Copa committed 11 years ago

* includes renaming pkgname, pkgver

Jim Pryor authored 11 years ago and Natanael Copa committed 11 years ago

* make vars local
* refactor calculation of default_name
* make non_interactive=1
* tweak error message
* tweak sed command

Jim Pryor authored 11 years ago and Natanael Copa committed 11 years ago

as discussed in http://lists.alpinelinux.org/alpine-devel/3122.html

Jim Pryor authored 11 years ago and Natanael Copa committed 11 years ago

Also remove incomplete efforts at listing "Depends on: ..."

Jim Pryor authored 11 years ago and Natanael Copa committed 11 years ago

as discussed in http://lists.alpinelinux.org/alpine-devel/3122.html