diff options
author | Justin Davis <jrcd83@gmail.com> | 2012-06-01 19:34:51 +0200 |
---|---|---|
committer | Justin Davis <jrcd83@gmail.com> | 2012-06-01 19:34:51 +0200 |
commit | 29be38d2671db067c86e07fa6177cd14055a32ea (patch) | |
tree | 5e5dbe6b0276c76d93990af7f5ba2aa0f058da37 /bin/genpkg | |
parent | b3fb393e03a885c07647f23d1c54ad60a2adec3a (diff) | |
download | genpkg-29be38d2671db067c86e07fa6177cd14055a32ea.tar.gz genpkg-29be38d2671db067c86e07fa6177cd14055a32ea.tar.xz |
Fix error handling when prepkg fails.
Typo fixed in prepkg. Do not duplicate pkgver unless
we are going to modify the package. Pipes screw up
the error code returned. vervar silently ignores a
missing _ver field.
Diffstat (limited to 'bin/genpkg')
-rwxr-xr-x | bin/genpkg | 16 |
1 files changed, 8 insertions, 8 deletions
@@ -104,20 +104,20 @@ do # Duplicate the version in case modpkg modifies it. vervar will remove a # redundant ver field that is the same as pkgver. trap 'rm -f PKGDATA' 1 2 15 - if PREPSPATH="$prepd" PKGVAR="$vard" prepkg "$pkg" | dupver > PKGDATA - then - echo "$pkgd/$pkg" - else - rm PKGDATA - exit "$?" - fi + PREPSPATH="$prepd" PKGVAR="$vard" prepkg "$pkg" > PKGDATA + ret=$? + case $ret in + 0) echo "$pkgd/$pkg" ;; + *) rm PKGDATA + exit $ret + esac trap '' 1 2 15 mod="$modsd/$pkg" if [ -f "$mod" -a -r "$mod" ] then trap 'rm -f PKGDATA.new' 1 2 15 - if modpkg "$mod" < PKGDATA > PKGDATA.new + if cat PKGDATA | dupver | modpkg "$mod" > PKGDATA.new then echo "Modified with $mod." 1>&2 mv PKGDATA.new PKGDATA |