[PATCH 2 of 3 py3 v3] tests: make a variable for hg binary location in test-check-py3-commands
Gregory Szorc
gregory.szorc at gmail.com
Mon Mar 13 16:30:06 EDT 2017
On Sun, Mar 12, 2017 at 3:42 PM, Augie Fackler <raf at durin42.com> wrote:
> # HG changeset patch
> # User Augie Fackler <augie at google.com>
> # Date 1489304265 14400
> # Sun Mar 12 03:37:45 2017 -0400
> # Node ID 2cd4d7ea0b19c4d5c55ccf8026ff1d7aa493a8cb
> # Parent a8f3bbc6259aebdeeef9f58309800f4070081214
> tests: make a variable for hg binary location in test-check-py3-commands
>
> The number of which calls in here is starting to get silly.
>
This is strictly better. But we may want to just introduce a $PY3HG
environment variable from run-tests.py. As long as it isn't quoted, shell
expansion should "just work." Or if shell expansion is a problem, we can
install a py3hg wrapper script that sets python3 in the shebang and alias
that to $PY3HG or something.
>
> diff --git a/tests/test-check-py3-commands.t b/tests/test-check-py3-
> commands.t
> --- a/tests/test-check-py3-commands.t
> +++ b/tests/test-check-py3-commands.t
> @@ -3,10 +3,11 @@
> This test helps in keeping a track on which commands we can run on
> Python 3 and see what kind of errors are coming up.
> The full traceback is hidden to have a stable output.
> + $ HGBIN=`which hg`
>
> $ for cmd in version debuginstall ; do
> > echo $cmd
> - > $PYTHON3 `which hg` $cmd 2>&1 2>&1 | tail -1
> + > $PYTHON3 $HGBIN $cmd 2>&1 2>&1 | tail -1
> > done
> version
> warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR
> PURPOSE.
> @@ -20,7 +21,7 @@ The full traceback is hidden to have a s
> $ cat >> $HGRCPATH <<EOF
> > %include $TESTTMP/included-hgrc
> > EOF
> - $ $PYTHON3 `which hg` version | tail -1
> + $ $PYTHON3 $HGBIN version | tail -1
> *** failed to import extension babar from imaginary_elephant: *:
> 'imaginary_elephant' (glob)
> warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR
> PURPOSE.
>
> _______________________________________________
> Mercurial-devel mailing list
> Mercurial-devel at mercurial-scm.org
> https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.mercurial-scm.org/pipermail/mercurial-devel/attachments/20170313/25fc50c4/attachment.html>
More information about the Mercurial-devel
mailing list