[arch-projects] [devtools][PATCH V2][makechrootpkg] Remove broken unknown arguments check and fix usage

Pierre Schmitz pierre at archlinux.de
Sun May 4 01:38:59 EDT 2014


Am 16.03.2014 20:34, schrieb Joel Teichroeb:
> If getopts comes across an unknown argument, $arg it set to '?' and 
> $OPTARG
> is unset. Therefore the getopts line detecting unknown arguments 
> doesn't
> work. Arguments to pass to makepkg are already handled by passing all 
> the
> aguments after the end-of-options marker (--), but this wasn't 
> documented
> in the usage text.
> 
> ---
>  makechrootpkg.in | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/makechrootpkg.in b/makechrootpkg.in
> index d03b703..9db0ddd 100644
> --- a/makechrootpkg.in
> +++ b/makechrootpkg.in
> @@ -35,8 +35,8 @@ src_owner=${SUDO_USER:-$USER}
>  usage() {
>   echo "Usage: ${0##*/} [options] -r <chrootdir> [--] [makepkg args]"
>   echo ' Run this script in a PKGBUILD dir to build a package inside a'
> - echo ' clean chroot. All unrecognized arguments passed to this 
> script'
> - echo ' will be passed to makepkg.'
> + echo ' clean chroot. Arguments passed to this script after the'
> + echo ' end-of-options marker (--) will be passed to makepkg.'
>   echo ''
>   echo ' The chroot dir consists of the following directories:'
>   echo ' <chrootdir>/{root, copy} but only "root" is required'
> @@ -78,7 +78,6 @@ while getopts 'hcur:I:l:nTD:d:' arg; do
>   l) copy="$OPTARG" ;;
>   n) run_namcap=true; makepkg_args="$makepkg_args -i" ;;
>   T) temp_chroot=true; copy+="-$$" ;;
> - *) makepkg_args="$makepkg_args -$arg $OPTARG" ;;
>   esac
>  done

Could you send this as a plain text email? Git does not like HTML mails. 
(Hint: Don't use gmail)

Greetings,

Pierre

-- 
Pierre Schmitz, https://pierre-schmitz.com


More information about the arch-projects mailing list