gh-100238: Use setuptools in peg-generator and reenable tests#104798
gh-100238: Use setuptools in peg-generator and reenable tests#104798lysnikolaou merged 9 commits into
Conversation
|
@lysnikolaou you need to probably rebase the changes in the PR that removes the endings from lines (also two windows tests are failing). |
Sorry, something went wrong.
|
We still have the failing windows tests. This may be something regarding the name of the binary perhaps? |
Sorry, something went wrong.
|
It appears so and I'm not sure why. The only other test that uses this ( |
Sorry, something went wrong.
Do you know if
|
Sorry, something went wrong.
|
Also that looks odd because there should be a temporary directory there no (as in |
Sorry, something went wrong.
That's okay, since that's the command that creates the venv. |
Sorry, something went wrong.
pablogsal
left a comment
There was a problem hiding this comment.
Great job @lysnikolaou ❤️
Sorry, something went wrong.
|
Probabaly no backport for this, right? |
Sorry, something went wrong.
I don't think so, but I also don't think it would hurt |
Sorry, something went wrong.
|
@pablogsal I built #105056 on top. I think it would be reasonable to backport this to 3.12, so I can backport mine as is. WDYT? |
Sorry, something went wrong.
|
Thanks @lysnikolaou for the PR 🌮🎉.. I'm working now to backport this PR to: 3.12. |
Sorry, something went wrong.
…ythonGH-104798) (cherry picked from commit afa759f) Co-authored-by: Lysandros Nikolaou <lisandrosnik@gmail.com>
|
I've asked Petr to label this with the 3.12 label so we know the CI results before a decision is made. |
Sorry, something went wrong.
|
(I pushed the button to open the 3.12 PR for Miro; I'm not implying it should be merged.) |
Sorry, something went wrong.
|
I'm okay with backporting, since this is just re-enabling a test, so it really wouldn't hurt. I'll leave it up to @pablogsal though. |
Sorry, something went wrong.
The problem with the back port is that we have ref leaks with this PR on |
Sorry, something went wrong.
That's right, I meant after fixing the refleaks. |
Sorry, something went wrong.
In that case I am ok back porting 👍 |
Sorry, something went wrong.
Since pythongh-104798 (Use setuptools in peg-generator and reenable tests), the TestCParser test case has been producing ref leaks.
pythonGH-106024) Since pythongh-104798 (Use setuptools in peg-generator and reenable tests), the TestCParser test case has been producing ref leaks. (cherry picked from commit 41ad4df) Co-authored-by: Erlend E. Aasland <erlend.aasland@protonmail.com>
⚠️⚠️⚠️ Buildbot failure ⚠️⚠️⚠️Hi! The buildbot AMD64 Windows11 Bigmem 3.12 has failed when building commit 6371029. What do you need to do:
You can take a look at the buildbot page here: https://buildbot.python.org/all/#builders/1117/builds/131 Summary of the results of the build (if available): == Tests result: ENV CHANGED == 431 tests OK. 10 slowest tests:
1 test altered the execution environment: 35 tests skipped: Total duration: 43 min 51 sec Click to see traceback logsremote: Enumerating objects: 13, done.
remote: Counting objects: 8% (1/12)
remote: Counting objects: 16% (2/12)
remote: Counting objects: 25% (3/12)
remote: Counting objects: 33% (4/12)
remote: Counting objects: 41% (5/12)
remote: Counting objects: 50% (6/12)
remote: Counting objects: 58% (7/12)
remote: Counting objects: 66% (8/12)
remote: Counting objects: 75% (9/12)
remote: Counting objects: 83% (10/12)
remote: Counting objects: 91% (11/12)
remote: Counting objects: 100% (12/12)
remote: Counting objects: 100% (12/12), done.
remote: Compressing objects: 11% (1/9)
remote: Compressing objects: 22% (2/9)
remote: Compressing objects: 33% (3/9)
remote: Compressing objects: 44% (4/9)
remote: Compressing objects: 55% (5/9)
remote: Compressing objects: 66% (6/9)
remote: Compressing objects: 77% (7/9)
remote: Compressing objects: 88% (8/9)
remote: Compressing objects: 100% (9/9)
remote: Compressing objects: 100% (9/9), done.
remote: Total 13 (delta 3), reused 8 (delta 3), pack-reused 1
From https://github.com/python/cpython
* branch 3.12 -> FETCH_HEAD
Note: switching to '637102980d12e5ff99b69ddffd5f05ecfe0adeb6'.
You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by switching back to a branch.
If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -c with the switch command. Example:
git switch -c <new-branch-name>
Or undo this operation with:
git switch -
Turn off this advice by setting config variable advice.detachedHead to false
HEAD is now at 637102980d [3.12] gh-100238: Use setuptools in peg-generator and reenable tests (GH-104798) (#105135)
Switched to and reset branch '3.12'
Could Not Find R:\buildarea\3.12.ambv-bb-win11.bigmem\build\Lib\*.pyc
The system cannot find the file specified.
Could Not Find R:\buildarea\3.12.ambv-bb-win11.bigmem\build\PCbuild\python*.zip
Could Not Find R:\buildarea\3.12.ambv-bb-win11.bigmem\build\PCbuild\python*.zip |
Sorry, something went wrong.
edited by bedevere-bot
LoadingUh oh!
There was an error while loading. Please reload this page.
Copy link Copy MarkdownSorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.