[PATCH 1 of 5 STABLE] dockerdeb: fix incorrect number of shifts

Anton Shestakov engored at ya.ru
Sun Apr 24 02:04:27 EDT 2016


24.04.2016, 10:29, "Sean Farley" <sean at farley.io>:
> # HG changeset patch
> # User Sean Farley <sean at farley.io>
> # Date 1461440859 25200
> # Sat Apr 23 12:47:39 2016 -0700
> # Branch stable
> # Node ID f0bec77f148405825a8c3289823e3981246965dc
> # Parent 51f5fae84e4381761c43c6156b3d6caf495529f8
> # EXP-Topic docker-ppa
> dockerdeb: fix incorrect number of shifts
>
> From the comment, it appears that the original intent was to remove the first
> two arguments, so this patch does just that.

Yep, the number of arguments changed from 1 to 2 in 9b86d29867a5 (and I overlooked that).

> diff --git a/contrib/dockerdeb b/contrib/dockerdeb
> --- a/contrib/dockerdeb
> +++ b/contrib/dockerdeb
> @@ -9,11 +9,11 @@ export ROOTDIR=$(cd $BUILDDIR/.. > /dev/
>  checkdocker
>
>  DISTID="$1"
>  CODENAME="$2"
>  PLATFORM="$1-$2"
> -shift # extra params are passed to build process
> +shift; shift # extra params are passed to build process
>
>  OUTPUTDIR=${OUTPUTDIR:=$ROOTDIR/packages/$PLATFORM}
>
>  initcontainer $PLATFORM

LGTM.


More information about the Mercurial-devel mailing list