[issue42574] Travis can't build the 3.8 branch right now

[Pablo Galindo Salgado]

Change by Pablo Galindo Salgado :


--
resolution:  -> fixed
stage: patch review -> resolved
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42574] Travis can't build the 3.8 branch right now

[Pablo Galindo Salgado]

Pablo Galindo Salgado  added the comment:


New changeset b6d6f5367da1f2e3f749c358ae8563c56a0cc395 by Pablo Galindo in 
branch '3.8':
bpo-42574: Use format() instead of f-string in Tools/clinic/clinic.py to allow 
using older Python versions (GH-23685)
https://github.com/python/cpython/commit/b6d6f5367da1f2e3f749c358ae8563c56a0cc395


--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42574] Travis can't build the 3.8 branch right now

[Pablo Galindo Salgado]

Change by Pablo Galindo Salgado :


--
pull_requests: +22548
pull_request: https://github.com/python/cpython/pull/23685

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42574] Travis can't build the 3.8 branch right now

[Pablo Galindo Salgado]

Pablo Galindo Salgado  added the comment:

I would suggest to use some other string formatting for that one line. The 
reason is that technically (although very unlikely) downstream consumers could 
be relying on some old PYTHON_FOR_REGEN in the Makefile for 3.8 and I would not 
like to change that in a patch release.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42574] Travis can't build the 3.8 branch right now

[Ammar Askar]

Ammar Askar  added the comment:

Oh whoops, got sniped by Brandt while investigating :)

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42574] Travis can't build the 3.8 branch right now

[Ammar Askar]

Ammar Askar  added the comment:

>From a quick glance at 
>https://github.com/python/cpython/pulls?page=2&q=base%3A3.8+-label%3Atype-documentation

It looks like travis on 3.8 last passed at 
https://github.com/python/cpython/pull/23364 around 19 days ago

The culprit commit is likely 
https://github.com/python/cpython/commit/66dd5338a1ca98921c8e6c51228541ef8ed8076a
 from 19 days ago since it added the first bit of f-string syntax to clinic.py, 
it was committed directly without a PR which is likely why this wasn't caught.

--
nosy: +ammar2

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42574] Travis can't build the 3.8 branch right now

[Brandt Bucher]

Brandt Bucher  added the comment:

So I suppose we can either:

- set PYTHON_FOR_REGEN to something 3.6+ on these backport branches
- use some other string formatting for that one line

I sort of prefer the second, since I *think* this is the only thing keeping 
older Pythons from working here.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42574] Travis can't build the 3.8 branch right now

[Brandt Bucher]

Brandt Bucher  added the comment:

It has been failing for some time. It's just not a required check, so it's easy 
to miss on a backport branch.

I think the addition of an f-string in PR 23371 a few weeks ago is what started 
making this fail (since the system python can't run clinic anymore). It looks 
like it has been failing ever since.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42574] Travis can't build the 3.8 branch right now

[Pablo Galindo Salgado]

Pablo Galindo Salgado  added the comment:

I am a bit confusedwhy is this failing now?

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42574] Travis can't build the 3.8 branch right now

[Brandt Bucher]

Brandt Bucher  added the comment:

Pablo, maybe you can shed some light on this. It looks like the PEG parser PR 
explicitly sets PYTHON_FOR_REGEN=python3.8 using pyenv in .travis.yml for the 
master (and now 3.9) branches. The change was made in the we-like-parsers repo 
here:

https://github.com/we-like-parsers/cpython/pull/88

Should we be doing this in the other branches as well?

--
nosy: +pablogsal

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42574] Travis can't build the 3.8 branch right now

[Brandt Bucher]

Change by Brandt Bucher :


--
keywords: +patch
pull_requests: +22522
stage:  -> patch review
pull_request: https://github.com/python/cpython/pull/23652

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42574] Travis can't build the 3.8 branch right now

[Brandt Bucher]

New submission from Brandt Bucher :

Travis seems to be using the wrong Python executable for (at least) the "make 
-j4 regen-all" step on the 3.8 branch. I have a hunch it's using the system 
python3 executable (3.5?). It causes the following failure when building:

...
python3 ./Tools/scripts/update_file.py ./Include/graminit.h 
./Include/graminit.h.new
  File "./Tools/clinic/clinic.py", line 1772
filename_new = f"{filename}.new"
   ^
SyntaxError: invalid syntax
Makefile:574: recipe for target 'clinic' failed
make: *** [clinic] Error 1

Recent examples:

https://travis-ci.com/github/python/cpython/jobs/454447280
https://travis-ci.com/github/python/cpython/jobs/454551266
https://travis-ci.com/github/python/cpython/jobs/454650029
https://travis-ci.com/github/python/cpython/jobs/454907763

I know Travis has been fairly problematic for us in the past.

--
components: Build
messages: 382555
nosy: brandtbucher
priority: normal
severity: normal
status: open
title: Travis can't build the 3.8 branch right now
versions: Python 3.8

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com