[pkg-wine-party] [PATCH 4/8] debian/gendeps.sh: bail on error, allow empty lines and comments in input files

[Michael Gilbert]

On Sun, Dec 30, 2012 at 6:08 PM, Hilko Bengen wrote:
> ---
>  debian/gendeps.sh |    5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
>  mode change 100644 => 100755 debian/gendeps.sh
>
> diff --git a/debian/gendeps.sh b/debian/gendeps.sh
> old mode 100644
> new mode 100755
> index 39f88d3..c73c98e
> --- a/debian/gendeps.sh
> +++ b/debian/gendeps.sh
> @@ -3,12 +3,13 @@ SUFFIX="$1"
>  shift
>  ARCHS="$@"
>
> +set -e
> +
>  for ext in deps recs; do
>    for dep in debian/*.${ext}; do
>      package="$(basename "$dep" .${ext})"
>      path="debian/$package$SUFFIX"
> -    deplist=$(sed 's,^,-l,' $dep)
> -  #  echo $path: $deplist
> +    deplist=$(sed '/^#/d;s,^\(..*\),-l\1,' $dep)
>      for arch in $ARCHS; do
>        gcc -m$arch -Wl,--noinhibit-exec -o "$path/extra${ext}$arch" debian/extradep.c $deplist
>      done

I think a better solution would be to eliminate the .deps and .recs
files, and go back to the conventional way of specifying dependencies
via debian/control fields.

Best wishes,
Mike