git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: What's cooking in git.git (Jan 2020, #04; Wed, 22)

From
Johannes Schindelin <johannes.schindelin@gmx.de>
Date
Jan 23, 2020, 21:39 UTC
Message-ID
<nycvar.QRO.7.76.6.2001232237590.46@tvgsbejvaqbjf.bet>
In-Reply-To
<20200123175645.GF6837@szeder.dev>
Hi Gábor,
On Thu, 23 Jan 2020, SZEDER Gábor wrote:
Show 61 quoted lines
> On Thu, Jan 23, 2020 at 03:16:26PM +0100, SZEDER Gábor wrote:
> > > What's the ideal endgame wrt the tests?
>
> > Running the 'linux-clang' job with Python 2 and the 'linux-gcc' job
> > with Python 3 would be the simplest and cheapest, I'd think.  We'd
> > only need to add the appropriate 'PYTHON_PATH=...' to out MAKEFLAGS.
> > As far as Travis CI is concerned, their Xenial image (i.e. the Linux
> > image we're using) comes with both 'python2' and 'python3' in PATH, at
> > versions v2.7 and v3.5, with the former being the default.
> >
> > Perhaps we could do the same with the OSX Clang and GCC jobs as well,
> > dunno.  Travis CI's OSX image, too, comes with both 'python2' and
> > 'python3' in PATH, though Python 3 is already at v3.7, but still v2.7
> > is the default.
>
> Replacing that last patch of the series with the diff below works both
> on Linux and macOS and both on Travis CI and Azure Pipelines.
>
> linux-clang with Python 2:
>   https://travis-ci.org/szeder/git/jobs/640912453#L499
>   https://dev.azure.com/gitgitgadget/git/_build/results?buildId=27690&view=logs&j=8f20da19-31b7-5cef-4813-95b8788bd086&t=56027f08-fde3-50ad-0c9a-5ec7df432ed0&l=615
>
> linux-gcc with Python 3:
>   https://travis-ci.org/szeder/git/jobs/640912454#L606
>   https://dev.azure.com/gitgitgadget/git/_build/results?buildId=27690&view=logs&j=275f1d19-1bd8-5591-b06b-07d489ea915a&t=33e5d3ec-87e7-5f80-0281-074c6962cb44&l=652
>
> osx-clang with Python 2:
>   https://travis-ci.org/szeder/git/jobs/640912455#L272
>   https://dev.azure.com/gitgitgadget/git/_build/results?buildId=27690&view=logs&j=b80c90c8-f62d-51c1-0986-3bb8359d9b6f&t=f8b92b00-54c3-55aa-48a6-84ec793cfb94&l=365
>
> osx-gcc with Python 3:
>   https://travis-ci.org/szeder/git/jobs/640912456#L283
>   https://dev.azure.com/gitgitgadget/git/_build/results?buildId=27690&view=logs&j=cfa20e98-6997-523c-4233-f0a7302c929f&t=3de1ae02-4adb-5138-54da-65cec5dd3141&l=394
>
>
>  --- >8 ---
>
> diff --git a/ci/lib.sh b/ci/lib.sh
> index a90d0dc0fd..c3a8cd2104 100755
> --- a/ci/lib.sh
> +++ b/ci/lib.sh
> @@ -162,6 +162,9 @@ linux-clang|linux-gcc)
>  	if [ "$jobname" = linux-gcc ]
>  	then
>  		export CC=gcc-8
> +		MAKEFLAGS="$MAKEFLAGS PYTHON_PATH=$(which python3)"
> +	else
> +		MAKEFLAGS="$MAKEFLAGS PYTHON_PATH=$(which python2)"
>  	fi
>
>  	export GIT_TEST_HTTPD=true
> @@ -182,6 +185,9 @@ osx-clang|osx-gcc)
>  	if [ "$jobname" = osx-gcc ]
>  	then
>  		export CC=gcc-9
> +		MAKEFLAGS="$MAKEFLAGS PYTHON_PATH=$(which python3)"
> +	else
> +		MAKEFLAGS="$MAKEFLAGS PYTHON_PATH=$(which python2)"
>  	fi
>
>  	# t9810 occasionally fails on Travis CI OS X

My only worry is that this makes it even more obscure what purpose which job has. Nothing in the name `osx-gcc` shouts loudly "I want to use Python 3.x!" to me.

Other than that, it is sensible.

Ciao, Dscho

Previous: Johannes SchindelinNext: SZEDER Gábor
Message 16 of 33 in “What's cooking in git.git (Jan 2020, #04; Wed, 22)”
  1. Junio C HamanoJan 22, 2020
  2. Elijah NewrenJan 22, 2020
  3. Junio C HamanoJan 22, 2020
  4. SZEDER GáborJan 22, 2020
  5. Junio C HamanoJan 23, 2020
  6. yz/p4-py3, was Re: What's cooking in git.git (Jan 2020, #04; Wed, 22)Johannes Schindelin, Jan 23, 2020
  7. Yang ZhaoJan 23, 2020
  8. SZEDER GáborJan 27, 2020
  9. SZEDER GáborJan 23, 2020
  10. SZEDER GáborJan 23, 2020
  11. Junio C HamanoJan 23, 2020
  12. Yang ZhaoJan 24, 2020
  13. Johannes SchindelinJan 25, 2020
  14. SZEDER GáborJan 25, 2020
  15. Johannes SchindelinJan 26, 2020
  16. Johannes SchindelinJan 23, 2020
  17. SZEDER GáborJan 24, 2020
  18. Johannes SchindelinJan 25, 2020
  19. Junio C HamanoFeb 5, 2020
  20. SZEDER GáborFeb 6, 2020
  21. Johannes SchindelinFeb 6, 2020
  22. SZEDER GáborFeb 6, 2020
  23. Johannes SchindelinFeb 6, 2020
  24. Johannes SchindelinJan 23, 2020
  25. Denton LiuJan 23, 2020
  26. Junio C HamanoJan 23, 2020
  27. Christian CouderJan 23, 2020
  28. Junio C HamanoJan 23, 2020
  29. Denton LiuJan 26, 2020
  30. Junio C HamanoJan 27, 2020
  31. .mailmap: fix erroneous authorship for Derrick StoleeDenton Liu, Jan 27, 2020
  32. Junio C HamanoJan 28, 2020
  33. Denton LiuJan 29, 2020

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.