[issue40163] multissl doesn't support tarballs in /source/old/

[Christian Heimes]

Christian Heimes  added the comment:

Benjamin, Larry,
The problem affects testing of security-only branches and 2.7.

--
components: +Tests
nosy: +benjamin.peterson, larry
type:  -> behavior

___
Python tracker 

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

[issue40163] multissl doesn't support tarballs in /source/old/

[Christian Heimes]

Change by Christian Heimes :


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

___
Python tracker 

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

[issue40163] multissl doesn't support tarballs in /source/old/

[Christian Heimes]

Christian Heimes  added the comment:

Ah crap :/ That's annoying.

This breaks all CI of all our active branches and all open PRs. I'll fix the 
issue and talk to OpenSSL upstream.

--
assignee:  -> christian.heimes
versions: +Python 2.7, Python 3.5, Python 3.6, Python 3.7, Python 3.8

___
Python tracker 

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

[issue40166] UNICODE HOWTO: Change the total number of code points in the introduction section

[Ama Aje My Fren]

Change by Ama Aje My Fren :


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

___
Python tracker 

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

[issue39943] Meta: Clean up various issues in C internals

[Andy Lester]

Change by Andy Lester :


--
pull_requests: +18692
pull_request: https://github.com/python/cpython/pull/19327

___
Python tracker 

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

[issue40167] Cython files don't compile on Mingw-w64 64-bit

[Beier Liu]

Change by Beier Liu :


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

___
Python tracker 

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

[issue40167] Cython files don't compile on Mingw-w64 64-bit

[Beier Liu]

New submission from Beier Liu :

details in https://github.com/cython/cython/issues/3405

--
components: Cross-Build
messages: 365673
nosy: Alex.Willmer, Beier Liu
priority: normal
severity: normal
status: open
title: Cython files don't compile on Mingw-w64 64-bit
type: compile error

___
Python tracker 

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

[issue40166] UNICODE HOWTO: Change the total number of code points in the introduction section

[Ama Aje My Fren]

New submission from Ama Aje My Fren :

The dev version of CPython gets the latest version of Unicode integrated to it 
regularly - and usually within dates of the latest version of Unicode coming 
out.

The Unicode HOWTO documentation has a line in the introduction that refers to 
the number of Unicode code points assigned so far. This document does not 
appear to be changed to concur with the number of actual code points supported 
by CPython or the latest standard by Unicode, Inc 
(http://www.unicode.org/versions/latest/#Summary).

I propose that a change be done to reflect the current number of code points.

--
assignee: docs@python
components: Documentation
messages: 365672
nosy: amaajemyfren, docs@python
priority: normal
severity: normal
status: open
title: UNICODE HOWTO: Change the total number of code points in the 
introduction section
versions: Python 3.9

___
Python tracker 

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

[issue40120] Undefined C behavior going beyond end of struct via a [1] arrays.

[Gregory P. Smith]

Gregory P. Smith  added the comment:

updates:

- extern "C" is indeed really only about linking so it has no bearing.

What I'm hearing from talking to our C++ compiler team is unfortunately sad: 
The C++ standard does not support flexible array member syntax on purpose 
because it leads to problems specific to C++ (ex: what do "new" and "del" do?)

So some compilers will reject such code (just as some accept it treating it as 
C99 does).  Meaning we can't do this in any public header file.

One workaround would indeed be to do something similar to that hashtable code, 
but it is quite annoying and I don't know that we could actually change the 
definition of PyBytesObject that way as its internals could be referenced 
externally.  (though all the bytes should line up regardless so even macros 
before and after such a change would be compatible?)

Within our internal private pure C code we could move to use this feature; 
things in .h files are the cross language issue.

Anyways I'm following up internally to better understand the motivation for 
wanting code to not use the "it's worked forever" technically undefined 
behavior of the trailing [1] member and out of bounds access.

Pondering, I wonder if this could turn into a "-fwrapv" style of situation, we 
depend on that behavior working so we adopted the compiler flag when compilers 
started to care; so at most we might some day need to pass another compiler 
flag to ensure it stays?  we'll see.

I'm inclined not to move forward with my PRs for now.

--

___
Python tracker 

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

[issue40165] Hide stderror from the user if command failes

[Batuhan Taskaya]

Change by Batuhan Taskaya :


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

___
Python tracker 

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

[issue40165] Hide stderror from the user if command failes

[Batuhan Taskaya]

New submission from Batuhan Taskaya :

This is an optional test that would only run if stty size returns a valid 
output, but if errors I dont think it makes sense to just print that error 
inside of test logs, there is already a noticement for skipped test.

--
components: Tests
messages: 365670
nosy: BTaskaya
priority: normal
severity: normal
status: open
title: Hide stderror from the user if command failes
versions: Python 3.9

___
Python tracker 

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

[issue40146] Upgrade Azure Pipelines to OpenSSL 1.1.1f

[STINNER Victor]

STINNER Victor  added the comment:


New changeset 7ed2acc6e89cea07f140fc374a77e8b36442df2e by Miss Islington (bot) 
in branch '3.7':
bpo-40146: Update OpenSSL to 1.1.1f in Azure Pipelines (GH-19320) (GH-19324)
https://github.com/python/cpython/commit/7ed2acc6e89cea07f140fc374a77e8b36442df2e


--

___
Python tracker 

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

[issue40162] Upgrade Travis CI to OpenSSL 1.1.1f

[STINNER Victor]

Change by STINNER Victor :


--
resolution:  -> fixed
stage: patch review -> resolved
status: open -> closed
versions: +Python 3.7, Python 3.8

___
Python tracker 

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

[issue38804] Regular Expression Denial of Service in http.cookiejar

[Larry Hastings]

Larry Hastings  added the comment:


New changeset 55a6a16a46239a71b635584e532feb8b17ae7fdf by Victor Stinner in 
branch '3.5':
bpo-38804: Fix REDoS in http.cookiejar (GH-17157) (#17344)
https://github.com/python/cpython/commit/55a6a16a46239a71b635584e532feb8b17ae7fdf


--

___
Python tracker 

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

[issue40140] test_builtin crashes when runned in parallel mode on solaris

[Batuhan Taskaya]

Batuhan Taskaya  added the comment:

> There is no more Solaris buildbot. Solaris 11.4 will be likely the last 
> release: Oracle no longer supports Solaris.

Well, if needed I can create one but looks like it is going be an obsoleted OS 
soon :/

--

___
Python tracker 

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

[issue40146] Upgrade Azure Pipelines to OpenSSL 1.1.1f

[miss-islington]

miss-islington  added the comment:


New changeset f2296ef9ce586bf2f51c125b085c2b080768040c by Miss Islington (bot) 
in branch '3.8':
bpo-40146: Update OpenSSL to 1.1.1f in Azure Pipelines (GH-19320)
https://github.com/python/cpython/commit/f2296ef9ce586bf2f51c125b085c2b080768040c


--

___
Python tracker 

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

[issue40162] Upgrade Travis CI to OpenSSL 1.1.1f

[miss-islington]

miss-islington  added the comment:


New changeset 1c325c4e0bf31a18d06784006eabf4d5a4a1d706 by Miss Islington (bot) 
in branch '3.8':
bpo-40162: Update Travis CI config to OpenSSL 1.1.1f (GH-19319)
https://github.com/python/cpython/commit/1c325c4e0bf31a18d06784006eabf4d5a4a1d706


--

___
Python tracker 

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

[issue40162] Upgrade Travis CI to OpenSSL 1.1.1f

[miss-islington]

miss-islington  added the comment:


New changeset 1ba6fe43e888668acfbf74038b82c6ee24ab1c41 by Miss Islington (bot) 
in branch '3.7':
bpo-40162: Update Travis CI config to OpenSSL 1.1.1f (GH-19319)
https://github.com/python/cpython/commit/1ba6fe43e888668acfbf74038b82c6ee24ab1c41


--

___
Python tracker 

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

[issue39503] [security][CVE-2020-8492] Denial of service in urllib.request.AbstractBasicAuthHandler

[Ned Deily]

Ned Deily  added the comment:


New changeset 69cdeeb93e0830004a495ed854022425b93b3f3e by Victor Stinner in 
branch '3.6':
bpo-39503: CVE-2020-8492: Fix AbstractBasicAuthHandler (GH-18284) (GH-19304)
https://github.com/python/cpython/commit/69cdeeb93e0830004a495ed854022425b93b3f3e


--
nosy: +ned.deily

___
Python tracker 

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

[issue40146] Upgrade Azure Pipelines to OpenSSL 1.1.1f

[miss-islington]

Change by miss-islington :


--
pull_requests: +18688
pull_request: https://github.com/python/cpython/pull/19323

___
Python tracker 

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

[issue40146] Upgrade Azure Pipelines to OpenSSL 1.1.1f

[miss-islington]

Change by miss-islington :


--
pull_requests: +18689
pull_request: https://github.com/python/cpython/pull/19324

___
Python tracker 

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

[issue40153] json dump with repeated key

[Raymond Hettinger]

Raymond Hettinger  added the comment:

Has this been a problem in practice, or just a theoretical issue?

To make this raise an exception, the JSON encoder would have to add one extra 
test per key.

I think we should just document the possibility.

--
nosy: +rhettinger

___
Python tracker 

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

[issue40146] Upgrade Azure Pipelines to OpenSSL 1.1.1f

[STINNER Victor]

STINNER Victor  added the comment:


New changeset 1767a0490f80c7b90d81051db24ef2b82cd9434f by Victor Stinner in 
branch 'master':
bpo-40146: Update OpenSSL to 1.1.1f in Azure Pipelines (GH-19320)
https://github.com/python/cpython/commit/1767a0490f80c7b90d81051db24ef2b82cd9434f


--

___
Python tracker 

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

[issue40162] Upgrade Travis CI to OpenSSL 1.1.1f

[STINNER Victor]

STINNER Victor  added the comment:


New changeset b1ffb8b72307a556442d09b427c3b29badb9878c by Victor Stinner in 
branch 'master':
bpo-40162: Update Travis CI config to OpenSSL 1.1.1f (GH-19319)
https://github.com/python/cpython/commit/b1ffb8b72307a556442d09b427c3b29badb9878c


--

___
Python tracker 

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

[issue40162] Upgrade Travis CI to OpenSSL 1.1.1f

[miss-islington]

Change by miss-islington :


--
pull_requests: +18687
pull_request: https://github.com/python/cpython/pull/19322

___
Python tracker 

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

[issue40162] Upgrade Travis CI to OpenSSL 1.1.1f

[miss-islington]

Change by miss-islington :


--
nosy: +miss-islington
nosy_count: 1.0 -> 2.0
pull_requests: +18686
pull_request: https://github.com/python/cpython/pull/19321

___
Python tracker 

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

[issue40125] update OpenSSL 1.1.1 in multissltests.py to 1.1.1f

[Ned Deily]

Ned Deily  added the comment:

Also bpo-40164: reminder to update Windows and macOS installer builds

--
nosy: +ned.deily

___
Python tracker 

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

[issue40156] CodeCov/patch job stills runs on pull requests on 3.5 and 3.6 branches

[STINNER Victor]

STINNER Victor  added the comment:

Thanks Ned and Larry.

--
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

[issue40163] multissl doesn't support tarballs in /source/old/

[STINNER Victor]

STINNER Victor  added the comment:

When OpenSSL moves a tarball, all our pre-commit CIs are broken and suddenly, 
all PRs can no longer be merged. We have first write PRs to update the 
configuration of our CI to use the newer OpenSSL version, merge these PRs, and 
then *all* pending PRs must be rebased on top of these merged PRS to retrieve 
the newer CI configuration.

There are currently 1085 pending PRs at https://github.com/python/cpython/pulls 
Well, for most of them, the CI already passed so we can merge them. But if a 
reviewer requires changes, the CI will re-run and then fail :-(

Moreover, fixing multissltests.py doesn't help neither, since again, PRs should 
be rebased to retrieve multissltests.py changes.

I hope that I'm wrong and the situation is not so bad.

--

Another solution would be to enhance our workflow to always rebase PRs on the 
development branch. Something like what https://mergify.io/ does.

I'm not sure what is the configuration of Azure Pipelines, GitHub actions and 
Travis CI. Would it be possible to make them rebase the PRs before running 
tests.

--

___
Python tracker 

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

[issue40164] Upgrade Windows and macOS installer builds to OpenSSL 1.1.1f

[Ned Deily]

New submission from Ned Deily :

1.1.1f released 2020-03-31

Reminder to Windows team to update Windows build.

Reminder to macOS team to update macOS installer build.
(note: please don't submit a PR or patch for this!)

https://www.openssl.org/source/

--
components: Windows, macOS
messages: 365657
nosy: ned.deily, paul.moore, ronaldoussoren, steve.dower, tim.golden, zach.ware
priority: deferred blocker
severity: normal
stage: needs patch
status: open
title: Upgrade Windows and macOS installer builds to OpenSSL 1.1.1f
versions: Python 3.7, Python 3.8, Python 3.9

___
Python tracker 

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

[issue40125] update OpenSSL 1.1.1 in multissltests.py to 1.1.1f

[STINNER Victor]

STINNER Victor  added the comment:

I created bpo-40163: multissl doesn't support tarballs in /source/old/.

--

___
Python tracker 

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

[issue40163] multissl doesn't support tarballs in /source/old/

[STINNER Victor]

New submission from STINNER Victor :

Tools/ssl/multissltests.py expects to find OpenSSL tarballs in:
https://www.openssl.org/source/

Like: https://www.openssl.org/source/openssl-1.1.1f.tar.gz

Problem: OpenSSL moves old versions to https://www.openssl.org/source/old/

If Tools/ssl/multissltests.py fails to download a tarball (HTTP error 404), it 
should try to get it from /source/old/.

It would prevent us to have to upgrade OpenSSL version immediately in all 
Python branches of all CIs (Azure Pipelines and Travis CI) as soon as OpenSSL 
decides to move a tarball. This move is not under our control.

Upgrading OpenSSL is a good practice. Breaking our CI is not :-)

--
components: Demos and Tools
messages: 365654
nosy: christian.heimes, vstinner
priority: normal
severity: normal
status: open
title: multissl doesn't support tarballs in /source/old/
versions: Python 3.9

___
Python tracker 

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

[issue40146] Upgrade Azure Pipelines to OpenSSL 1.1.1f

[STINNER Victor]

Change by STINNER Victor :


--
pull_requests: +18685
pull_request: https://github.com/python/cpython/pull/19320

___
Python tracker 

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

[issue40125] update OpenSSL 1.1.1 in multissltests.py to 1.1.1f

[STINNER Victor]

STINNER Victor  added the comment:

See also bpo-40162: Upgrade Travis CI to OpenSSL 1.1.1f.

--

___
Python tracker 

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

[issue40162] Upgrade Travis CI to OpenSSL 1.1.1f

[STINNER Victor]

Change by STINNER Victor :


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

___
Python tracker 

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

[issue40162] Upgrade Travis CI to OpenSSL 1.1.1f

[STINNER Victor]

New submission from STINNER Victor :

Similary to bpo-40146 "Upgrade Azure Pipelines to OpenSSL 1.1.1f", the Travis 
CI configuration has an OpenSSL version. It's currently 1.1.1d, but the tarball 
of this version moved from /source/ to /source/old/.

We should upgrade Travis CI configuration to OpenSSL 1.1.1f.

Attached PR does that.

--
components: Tests
messages: 365652
nosy: vstinner
priority: normal
severity: normal
status: open
title: Upgrade Travis CI to OpenSSL 1.1.1f
versions: Python 3.9

___
Python tracker 

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

[issue40156] CodeCov/patch job stills runs on pull requests on 3.5 and 3.6 branches

[Ned Deily]

Ned Deily  added the comment:


New changeset ebeabb5b728f009480ced3ca4738c20fa073b507 by Victor Stinner in 
branch '3.6':
bpo-40156: Copy Codecov configuration from master (GH-19306)
https://github.com/python/cpython/commit/ebeabb5b728f009480ced3ca4738c20fa073b507


--

___
Python tracker 

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

[issue40140] test_builtin crashes when runned in parallel mode on solaris

[STINNER Victor]

STINNER Victor  added the comment:

I close the issue, it's now fixed in 3.8 and master (and I'm working on a 3.7 
backport: PR 19318). Thanks Batuhan for the bug report.

--
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

[issue40140] test_builtin crashes when runned in parallel mode on solaris

[STINNER Victor]

Change by STINNER Victor :


--
pull_requests: +18683
pull_request: https://github.com/python/cpython/pull/19318

___
Python tracker 

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

[issue40155] AIX: test_builtin.test_input_no_stdout_fileno() hangs

[STINNER Victor]

STINNER Victor  added the comment:

> bpo-40140: test_builtin.PtyTests registers SIGHUP handler (GH-19314)

With this change, PPC64 AIX 3.x is back to green. I close this issue. Thanks 
for the bug report Michael Felt.

FYI I also backported my fix to 3.8: PR 19316.

--
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

[issue40140] test_builtin crashes when runned in parallel mode on solaris

[STINNER Victor]

STINNER Victor  added the comment:


New changeset 745bd91bab8e57c52d63a2d541465551d7551f78 by Victor Stinner in 
branch '3.8':
bpo-40140: test_builtin.PtyTests registers SIGHUP handler (GH-19314) (GH-19316)
https://github.com/python/cpython/commit/745bd91bab8e57c52d63a2d541465551d7551f78


--

___
Python tracker 

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

[issue36541] Make lib2to3 grammar better match Python, support the := walrus

[Tim Hatch]

Change by Tim Hatch :


--
pull_requests: +18682
pull_request: https://github.com/python/cpython/pull/19317

___
Python tracker 

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

[issue40158] MSBuild Extensions in CPython NuGet Package has Bad Expression

[Chris Martinez]

Chris Martinez  added the comment:

In testing the fix, another issue has arisen. It appears the specified 
expression will never yield a usable path.

Expression 1:
$([msbuild]::GetDirectoryNameOfFileAbove($(MSBuildThisFileDirectory), 
"python_d.exe"))

Expression 2:
$([msbuild]::GetDirectoryNameOfFileAbove($(MSBuildThisFileDirectory), 
"python.exe"))

The package has the abridged structure of:

┌─ build
│  │
│  └─ native
│ │
│ └─ python.props
│
└─ tools
   │
   └─ python.exe


Based on this hierachy, neither exe will resolve because they do not have the 
same common ancestor. Additionally,
I found that "python_d.exe" is always assumed for "Configuration=Debug", but 
"python_d.exe" does not exist in
the package (that I could find). I'm not sure if this means the path is wrong, 
"python_d.exe" was accidentally
omiitted, or this property assignment simply should not exist. This current 
behavior will ultimately result in
the build integration failing because "python_d.exe" is resolved, but it 
doesn't exist.

Interestingly, the property specified in versions 3.7.1 and earlier appear to 
define the correct path as:

$(MSBuildThisFileDirectory)\..\..\tools

My suggestion is to revert back to this older variant. If "python_d.exe" isn't 
needed, then it should be removed.
If it is needed, then the path needs to be fixed. To make the path more robust, 
I also recommend resolving this
path using one of the following forms:

$([MSBuild]::NormalizePath($(MSBuildThisFileDirectory)\..\..\tools))

OR

$([System.IO.Path]::GetFullPath($(MSBuildThisFileDirectory)\..\..\tools))


In my particular case, the tooling I'm plugging this into wasn't happy unless 
the path was absolute. There
doesn't appear to be any reason or downside to resolving the path ahead of 
time. I can easily workaround
that issue, but I suspect resolving an absolute path may be useful to other 
package consumers as well.

As soon as I know what the final form of the property should be, I'll submit 
the PR and update this issue with the link

--

___
Python tracker 

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

[issue40160] documentation example of os.walk should be less destructive

[Raymond Hettinger]

Raymond Hettinger  added the comment:

The proposed replacement doesn't succeed in demonstrating why topdown=False is 
necessary.  Consider doing a rename instead of a deletion or print.

--
nosy: +rhettinger

___
Python tracker 

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

[issue40156] CodeCov/patch job stills runs on pull requests on 3.5 and 3.6 branches

[Larry Hastings]

Larry Hastings  added the comment:


New changeset ed07522a5faa3101f68be8e4b8369310f60860f8 by Victor Stinner in 
branch '3.5':
bpo-40156: Copy Codecov configuration from master (#19309)
https://github.com/python/cpython/commit/ed07522a5faa3101f68be8e4b8369310f60860f8


--

___
Python tracker 

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

[issue17005] Add a topological sort algorithm

[Raymond Hettinger]

Raymond Hettinger  added the comment:

How about I post a PR so we can talk about something concrete.  Then you two 
can either fight it to its death or you can join me in making it is good as 
possible, hopefully the latter :-)

I am not happy with the current API but do accept that both of you are in 
satisfied with it.

--

___
Python tracker 

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

[issue17005] Add a topological sort algorithm

[Raymond Hettinger]

Change by Raymond Hettinger :


--
Removed message: https://bugs.python.org/msg365640

___
Python tracker 

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

[issue40140] test_builtin crashes when runned in parallel mode on solaris

[STINNER Victor]

STINNER Victor  added the comment:

> Victor, PR 19314 works perfectly.

Thanks for testing Batuhan ;-)

By the way, Solaris is no longer officially supported by Python:
https://pythondev.readthedocs.io/platforms.html#best-effort-and-unofficial-platforms

There is no more Solaris buildbot. Solaris 11.4 will be likely the last 
release: Oracle no longer supports Solaris.

We may accept minor changes, but no invasive changes.

--

___
Python tracker 

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

[issue40140] test_builtin crashes when runned in parallel mode on solaris

[STINNER Victor]

Change by STINNER Victor :


--
pull_requests: +18681
pull_request: https://github.com/python/cpython/pull/19316

___
Python tracker 

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

[issue40155] AIX: test_builtin.test_input_no_stdout_fileno() hangs

[STINNER Victor]

STINNER Victor  added the comment:

I pushed this change which should fix the issue on AIX:

New changeset 7a51a7e19f0143f75f8fc9ff68f93ed40937aec6 by Victor Stinner in 
branch 'master':
bpo-40140: test_builtin.PtyTests registers SIGHUP handler (GH-19314)
https://github.com/python/cpython/commit/7a51a7e19f0143f75f8fc9ff68f93ed40937aec6

I'm waiting for buildbots before closing this issue.

--

___
Python tracker 

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

[issue40140] test_builtin crashes when runned in parallel mode on solaris

[STINNER Victor]

STINNER Victor  added the comment:


New changeset 7a51a7e19f0143f75f8fc9ff68f93ed40937aec6 by Victor Stinner in 
branch 'master':
bpo-40140: test_builtin.PtyTests registers SIGHUP handler (GH-19314)
https://github.com/python/cpython/commit/7a51a7e19f0143f75f8fc9ff68f93ed40937aec6


--

___
Python tracker 

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

[issue36541] Make lib2to3 grammar better match Python, support the := walrus

[miss-islington]

Change by miss-islington :


--
nosy: +miss-islington
nosy_count: 9.0 -> 10.0
pull_requests: +18680
pull_request: https://github.com/python/cpython/pull/19315

___
Python tracker 

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

[issue17005] Add a topological sort algorithm

[Raymond Hettinger]

Raymond Hettinger  added the comment:

How about I post a PR so we can talk about something concrete.  Then you two 
can either fight it to its death or you can join me in making it is good as 
possible, hopefully the latter :-)

I am not happy with the current API but do accept that both of you are in love 
with it.

--

___
Python tracker 

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

[issue17005] Add a topological sort algorithm

[Pablo Galindo Salgado]

Pablo Galindo Salgado  added the comment:

Is also notable to mention that you can also provide the graph as a dictionary 
to the constructor:

>>> graph = {D: {B, C}, C: {A}, B: {A}, A:{object}}
>>> ts = TopologicalSorter(graph)

--

___
Python tracker 

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

[issue17005] Add a topological sort algorithm

[Tim Peters]

Tim Peters  added the comment:

Possibly, sure.  But I believe it's hard to beat

add(node, *predecessors)

for usability as a way to build the dependency graph.  For example, a list of 
pairs is a comparative PITA for most use cases I've had.  Whether it's 
following a recipe to bake a cake, or tracing a maze of C include files, it 
seems _most_ natural to get input in the form "this thing depends on these 
other things".  Not the other way around, and neither a sequence of pairs.

_If_ you buy that, then .add() is screamingly natural, and trying to squash a 
pile of .add()s into a single sequence-of-sequences argument seems strained.

Typically I don't get input in one big, single gulp.  It's instead discovered 
one item at a time.  Fine - .add() it and then move on to the next item.  It's 
certainly possible to append the item and its predecessors to a persistent 
(across items) list, and call a function once at the end with that list.

But what does that buy?  I'm building the list solely to meet the function's 
input requirement - the list serves no other purpose.  Instead of calling 
.add() N times, I call .append() N times.  "add" is 3 letters shorter ;-)

--

___
Python tracker 

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

[issue40140] test_builtin crashes when runned in parallel mode on solaris

[Batuhan Taskaya]

Batuhan Taskaya  added the comment:

Victor, PR 19314 works perfectly.

--

___
Python tracker 

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

[issue40161] Name collisions in pythoncore, preventing unity/jumbo build

[Alexander Riccio]

New submission from Alexander Riccio :

This isn't a priority issue I'd say. However, fixing it could yield nice 
benefits. I ran into this while experimenting with JUMBO/Unity builds as part 
of a bit of fun I've been having tweaking build options across the CPython 
ecosystem.

Theoretically, a JUMBO/Unity build could reduce code size, improve performance, 
and maybe even help code analysis detect more bugs by building everything in a 
single compilation unit. Link Time Code Generation is great, but usually isn't 
as good as building everything in a single compilation unit.


An example of an interesting thing noticed while compiling as a Unity build:

This exact variable is defined twice in two separate source files, 
itertoolsmodule.c:4303, and and collectionsmodule.c:1774:

PyDoc_STRVAR(length_hint_doc, "Private method returning an estimate of 
len(list(it)).");

...the default Release configuration includes this exact string 12 (!) times.

There's a lot of stuff like that. It's not actually broken, and sometimes it's 
probably inconvenient to fix it (what are you gonna do? put it in a header?), 
but it would be nice.

--
components: Interpreter Core
messages: 365636
nosy: Alexander Riccio
priority: normal
severity: normal
status: open
title: Name collisions in pythoncore, preventing unity/jumbo build
type: compile error
versions: Python 3.9

___
Python tracker 

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

[issue35455] Solaris thread_time doesn't work with current implementation

[Batuhan Taskaya]

Batuhan Taskaya  added the comment:

This issue is still valid under other solaris/sunos versions. @kulikjak are you 
still interested in resolving this issue?

--
nosy: +BTaskaya
resolution: not a bug -> 
status: closed -> open
versions: +Python 3.9

___
Python tracker 

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

[issue40140] test_builtin crashes when runned in parallel mode on solaris

[STINNER Victor]

STINNER Victor  added the comment:

Batuhan: Ok, now please test PR 19314 which registers a signal handler for 
SIGHUP. It should fix the issue for Solaris. Moreover, I also includes the fix 
for AIX (bpo-40155).

--

___
Python tracker 

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

[issue40140] test_builtin crashes when runned in parallel mode on solaris

[Batuhan Taskaya]

Batuhan Taskaya  added the comment:

isidentical@gcc-solaris11:~/cpython$ ./python -m test test_builtin -m 
test_input_no_stdout_fileno -F -j10 -v
== CPython 3.9.0a5+ (heads/master:dc4e965, Apr 2 2020, 23:53:26) [GCC 5.5.0]
== Solaris-2.11-sun4u-sparc-32bit big-endian
== cwd: /export/home/isidentical/cpython/build/test_python_24804
== CPU count: 8
== encodings: locale=UTF-8, FS=utf-8
0:00:00 load avg: 1.56 Run tests in parallel using 10 child processes
0:00:02 load avg: 1.57 [  1/1] test_builtin crashed (Exit code -1)
test_input_no_stdout_fileno (test.test_builtin.PtyTests) ...
Kill  
process group
Kill  
process group
Kill  
process group
Kill  
process group
Kill  
process group
Kill  
process group
Kill  
process group
Kill  
process group
Kill  
process group

== Tests result: FAILURE ==

1 test failed:
test_builtin

Total duration: 2.2 sec
Tests result: FAILURE
isidentical@gcc-solaris11:~/cpython$ wget 
https://patch-diff.githubusercontent.com/raw/python/cpython/pull/19312.patch
--2020-04-02 23:53:51--  
https://patch-diff.githubusercontent.com/raw/python/cpython/pull/19312.patch
Resolving patch-diff.githubusercontent.com 
(patch-diff.githubusercontent.com)... 140.82.118.4
Connecting to patch-diff.githubusercontent.com 
(patch-diff.githubusercontent.com)|140.82.118.4|:443... connected.
HTTP request sent, awaiting response... 200 OK
Cookie coming from patch-diff.githubusercontent.com attempted to set domain to 
github.com
Length: unspecified [text/plain]
Saving to: ‘19312.patch’

19312.patch [ <=>   
 ]   1.62K  --.-KB/sin 0s  

2020-04-02 23:53:51 (3.38 MB/s) - ‘19312.patch’ saved [4252]
isidentical@gcc-solaris11:~/cpython$ git apply 19312.patch
isidentical@gcc-solaris11:~/cpython$ gmake -j8
...
isidentical@gcc-solaris11:~/cpython$ ./python -m test test_builtin -m 
test_input_no_stdout_fileno -F -j10 -v  
== CPython 3.9.0a5+ (heads/master:dc4e965, Apr 2 2020, 23:53:26) [GCC 5.5.0]
== Solaris-2.11-sun4u-sparc-32bit big-endian
== cwd: /export/home/isidentical/cpython/build/test_python_24850
== CPU count: 8
== encodings: locale=UTF-8, FS=utf-8
0:00:00 load avg: 1.71 Run tests in parallel using 10 child processes
0:00:02 load avg: 1.78 [  1/1] test_builtin crashed (Exit code -1)
test_input_no_stdout_fileno (test.test_builtin.PtyTests) ...
Kill  
process group
Kill  
process group
Kill  
process group
Kill  
process group
Kill  
process group
Kill  
process group
Kill  
process group
Kill  
process group
Kill  
process group

== Tests result: FAILURE ==

1 test failed:
test_builtin

Total duration: 2.7 sec
Tests result: FAILURE

--

___
Python tracker 

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

[issue17005] Add a topological sort algorithm

[Pablo Galindo Salgado]

Pablo Galindo Salgado  added the comment:

> We may need two versions then, a full-featured TopologicalSorter() class and 
> a simple tsort() function that doesn't aspire to be all things to all people.

How this other version would differ from using .add() + .static_order() as Tim 
mentions? Originally we designed static_order() so it will satisfy the simpler 
use cases so I would suggest aspiring to simplify that interface if needed 
instead of adding an extra function.

--

___
Python tracker 

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

[issue40140] test_builtin crashes when runned in parallel mode on solaris

[STINNER Victor]

Change by STINNER Victor :


--
pull_requests: +18679
pull_request: https://github.com/python/cpython/pull/19314

___
Python tracker 

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

[issue19961] MacOSX: Tkinter build failure when building without command-line tools

[Furkan Onder]

Furkan Onder  added the comment:

You can try this solution,
https://stackoverflow.com/questions/11465258/xlib-h-not-found-when-building-graphviz-on-mac-os-x-10-8-mountain-lion#12089021

--
nosy: +furkanonder

___
Python tracker 

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

[issue17005] Add a topological sort algorithm

[Raymond Hettinger]

Raymond Hettinger  added the comment:

> If your alternative isn't equally easy to use in a parallelized
> context, I'll be at best +0.

We may need two versions then, a full-featured TopologicalSorter() class and a 
simple tsort() function that doesn't aspire to be all things to all people.

--

___
Python tracker 

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

[issue40140] test_builtin crashes when runned in parallel mode on solaris

[STINNER Victor]

STINNER Victor  added the comment:

> I tested with both PR 19312 and PR 19308 and I still have the same crash 

Which test is causing the issue? Does it still crash if you comment 
test_input_no_stdout_fileno()? Try to rename it "Xtest_input_no_stdout_fileno" 
to skip it.

What if you only run this test?

./python -m test test_builtin -m test_input_no_stdout_fileno -F -j10 -v

Maybe this test should register a signal handler for SIGHUP?

This bug looks like bpo-38547 which affected test_pty. I fixed it by 
registering a SIGHUP signal handler:

commit a1838ec2592e5082c75c77888f2a7a3eb21133e5
Author: Victor Stinner 
Date:   Mon Dec 9 11:57:05 2019 +0100

bpo-38547: Fix test_pty if the process is the session leader (GH-17519)

Fix test_pty: if the process is the session leader, closing the
master file descriptor raises a SIGHUP signal: simply ignore SIGHUP
when running the tests.

--

___
Python tracker 

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

[issue39704] Disable code coverage

[Larry Hastings]

Larry Hastings  added the comment:

Since explicit is better than implicit: yes, we do need backports.  PRs against 
3.5 are getting marked red because of automated codecov complaints.

--
nosy: +larry

___
Python tracker 

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

[issue40160] documentation example of os.walk should be less destructive

[John Taylor]

John Taylor  added the comment:

https://github.com/python/cpython/pull/19313

I have just signed the CLA.

--

___
Python tracker 

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

[issue40160] documentation example of os.walk should be less destructive

[Roundup Robot]

Change by Roundup Robot :


--
keywords: +patch
nosy: +python-dev
nosy_count: 2.0 -> 3.0
pull_requests: +18678
stage:  -> patch review
pull_request: https://github.com/python/cpython/pull/19313

___
Python tracker 

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

[issue17681] Work with an extra field of gzip and zip files

[Jason Williams]

Jason Williams  added the comment:

What's needed to get this integrated?  It will be great to not have to fork the 
GZIP.

--
nosy: +Jason Williams

___
Python tracker 

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

[issue17005] Add a topological sort algorithm

[STINNER Victor]

Change by STINNER Victor :


--
nosy: +vstinner

___
Python tracker 

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

[issue40140] test_builtin crashes when runned in parallel mode on solaris

[Batuhan Taskaya]

Batuhan Taskaya  added the comment:

I tested with both PR 19312 and PR 19308 and I still have the same crash 
0:00:00 load avg: 0.80 Run tests in parallel using 2 child processes
0:00:01 load avg: 0.79 [1/1/1] test_builtin crashed (Exit code -1)

--

___
Python tracker 

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

[issue40160] documentation example of os.walk should be less destructive

[John Taylor]

New submission from John Taylor :

The example for os.walkdir should be less destructive.  It currently 
recursively removes all files and directories.  I will be submitting a PR on 
GitHub.

--
assignee: docs@python
components: Documentation
messages: 365625
nosy: docs@python, jftuga
priority: normal
severity: normal
status: open
title: documentation example of os.walk should be less destructive
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

[issue40159] Make python -V -V output arguments of configure

[Volker Weißmann]

Change by Volker Weißmann :


--
stage:  -> 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

[issue40159] Make python -V -V output arguments of configure

[Zachary Ware]

Zachary Ware  added the comment:

How about `python3 -c 'import 
sysconfig;print(sysconfig.get_config_vars()["CONFIG_ARGS"])'`?

--
nosy: +zach.ware

___
Python tracker 

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

[issue17005] Add a topological sort algorithm

[Pablo Galindo Salgado]

Pablo Galindo Salgado  added the comment:

I just want to echo what Tim mentioned with the extra data point that some of 
the maintainers of some popular and wide-used open-source libraries that indeed 
have to deal with this problem or the parallel version of the problem (like 
gaborbernat in this thread, the maintainer of "tox" and "virtualenv") do indeed 
find the current API desirable.

--

___
Python tracker 

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

[issue40140] test_builtin crashes when runned in parallel mode on solaris

[STINNER Victor]

STINNER Victor  added the comment:

Batuhan: Can you please test if PR 19312 fix the issue for you on Solaris?

--

___
Python tracker 

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

[issue40159] Make python -V -V output arguments of configure

[Volker Weißmann]

New submission from Volker Weißmann :

As you might know, you can e.g. compile a version with
../configure --with-pydebug
or with
../configure
Currently, there is no easy way to find out how an installation on your machine 
was compiled. It would be nice if python -V -V would output every argument of 
configure.

--
messages: 365620
nosy: Volker Weißmann
priority: normal
severity: normal
status: open
title: Make python -V -V output arguments of configure
type: enhancement

___
Python tracker 

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

[issue40155] AIX: test_builtin.test_input_no_stdout_fileno() hangs

[STINNER Victor]

STINNER Victor  added the comment:

Michael Felt: Can you please test if PR 19312 fix the issue for you on AIX?

--

___
Python tracker 

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

[issue40140] test_builtin crashes when runned in parallel mode on solaris

[STINNER Victor]

Change by STINNER Victor :


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

___
Python tracker 

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

[issue31160] Enhance support.reap_children()

[STINNER Victor]

Change by STINNER Victor :


--
pull_requests: +18675
pull_request: https://github.com/python/cpython/pull/19312

___
Python tracker 

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

[issue40155] AIX: test_builtin.test_input_no_stdout_fileno() hangs

[STINNER Victor]

Change by STINNER Victor :


--
pull_requests: +18677
pull_request: https://github.com/python/cpython/pull/19312

___
Python tracker 

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

[issue17005] Add a topological sort algorithm

[Tim Peters]

Tim Peters  added the comment:

Raymond, what application do you have that wouldn't be completely addressed by 
sticking to just .add() (to record dependencies) and .static_order() (to 
retrieve a linear order)?

Larry Hastings and I originally worked out the fancier bits of the interface to 
deal with problems he actually had, and for which no existing Python topsort 
implementation we could find was of any use:  extract maximal parallelism.  If 
you don't want that, fine, stick to the two simple bits.

The bits to support parallelism are very easy to use to write correct 
parallelized code, but of course can seem baffling if you don't give a rip 
about parallelism.  But in that case you have no need to learn about them 
either.

If your alternative isn't equally easy to use in a parallelized context, I'll 
be at best +0.

About "fast", this is linear time, in the sum of the number of items and 
dependencies.  Including the part checking for a cycle, which is by far the 
"hardest" part.  So it's asymptotically optimal, although I've never seen a 
real context in which topsort speed made a lick of difference.

In the real world, in a parallelized context it can be important to check for a 
cycle _before_ running a topsort:  actions are performed ASAP based on 
order-deduced-so-far, and it can be no good to find out "oh! I can't finish 
this" at the end.  There's actually nothing gratuitous here.  If it seems 
"over-engineered", that's because it's addressing problems you haven't had yet 
;-)

--

___
Python tracker 

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

[issue38972] [venv] Link to instructions to change PowerShell execution policy for venv activation

[miss-islington]

miss-islington  added the comment:


New changeset b7345c24a4d962e2adbafc86e4af77de9e3ef09e by Miss Islington (bot) 
in branch '3.8':
bpo-38972: Link to instructions to change PowerShell execution policy (GH-19131)
https://github.com/python/cpython/commit/b7345c24a4d962e2adbafc86e4af77de9e3ef09e


--

___
Python tracker 

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

[issue40133] Provide additional matchers for unittest.mock

[Diego Elio Pettenò]

Change by Diego Elio Pettenò :


--
nosy: +gregory.p.smith

___
Python tracker 

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

[issue38972] [venv] Link to instructions to change PowerShell execution policy for venv activation

[Brett Cannon]

Change by Brett Cannon :


--
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

[issue38972] [venv] Link to instructions to change PowerShell execution policy for venv activation

[miss-islington]

Change by miss-islington :


--
nosy: +miss-islington
nosy_count: 6.0 -> 7.0
pull_requests: +18674
pull_request: https://github.com/python/cpython/pull/19311

___
Python tracker 

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

[issue38972] [venv] Link to instructions to change PowerShell execution policy for venv activation

[Brett Cannon]

Brett Cannon  added the comment:


New changeset 45217af29c7f380089af17beb48a5ea0560bbb9d by Derek Keeler in 
branch 'master':
bpo-38972: Link to instructions to change PowerShell execution policy (GH-19131)
https://github.com/python/cpython/commit/45217af29c7f380089af17beb48a5ea0560bbb9d


--

___
Python tracker 

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

[issue40127] Documentation of SSL library

[Christophe Nanteuil]

Christophe Nanteuil  added the comment:

I modified the PR according to the source code: 
"if all three are None and SSLContext.verify_mode is not set to CERT_NONE, this 
function uses the system's default CA certificates."

The way the system is configured may depend on multiple parameters but I hope 
this statement is clearer and it disturbs me to read that the function "can 
choose", all the more for a security module.

--

___
Python tracker 

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

[issue40133] Provide additional matchers for unittest.mock

[Roundup Robot]

Change by Roundup Robot :


--
keywords: +patch
nosy: +python-dev
nosy_count: 2.0 -> 3.0
pull_requests: +18673
stage:  -> patch review
pull_request: https://github.com/python/cpython/pull/19310

___
Python tracker 

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

[issue17005] Add a topological sort algorithm

[Raymond Hettinger]

Raymond Hettinger  added the comment:

At some point in the next two or three weeks, I'll have a chance to work on 
this more and to offer a competing patch.  IMO, the current checkin is 
over-engineered, both in its API and implementation.  This could have been a 
simple, fast tool written as one or two short Python functions.

Also, I would like to try to out the API alternatives on some groups of 
engineers to get some user feedback.

For me, as the API currently stands, I would have to write a wrapper to make it 
usable for my applications.

--

___
Python tracker 

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

[issue40158] MSBuild Extensions in CPython NuGet Package has Bad Expression

[Steve Dower]

Steve Dower  added the comment:

The closing parentheses are needed - a PR would be appreciated for that.

The quotes around a variable reference are unnecessary. At a parser level, it 
just changes it from a variable reference to a string literal with 
substitutions (unlike most shells, which do a textual substitution before 
parsing).

--

___
Python tracker 

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

[issue40156] CodeCov/patch job stills runs on pull requests on 3.5 and 3.6 branches

[STINNER Victor]

STINNER Victor  added the comment:

> I need to do a little more reading on it, but I expect if you make an 
> equivalent PR for 3.5 I'll merge it.  Thanks for taking this on, Victor!

Done with PR 19309.

--

___
Python tracker 

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

[issue1635741] Py_Finalize() doesn't clear all Python objects at exit

[STINNER Victor]

STINNER Victor  added the comment:


New changeset 7a6f3bcc43ed729f8038524528c0b326b5610506 by Hai Shi in branch 
'master':
bpo-1635741: Fix refleak in _locale init error handling (GH-19307)
https://github.com/python/cpython/commit/7a6f3bcc43ed729f8038524528c0b326b5610506


--

___
Python tracker 

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

[issue40158] MSBuild Extensions in CPython NuGet Package has Bad Expression

[Chris Martinez]

New submission from Chris Martinez :

CPython provides a NuGet package as a mechanism to support non-installed Python 
distributions. The package includes MSBuild support to integrate with its build 
process.

The expressions on lines 32 and 33 in the file:

https://github.com/python/cpython/blob/master/PC/layout/support/props.py

are both missing closing parentheses, which results in literal text instead of 
the resolve file paths. This appears to be introduced in version 3.7.2 of the 
package onward, including the current pre-release 3.9.0-a5.

In addition, several build conditions use the form " $(Property) == 'value' ", 
but should instead use " '$(Property)' == 'value' ". By not surrounding the 
property value with '', the condition may resolve as "  == '' ", which is an 
invalid expression and will cause a build failure. This doesn't appear to have 
caused an issue yet, but it easily could.

If there is no further discussion or objection, I can submit a PR with the 
required fixes.

--
components: Build, Demos and Tools, Distutils, Installation, Windows
messages: 365610
nosy: dstufft, eric.araujo, paul.moore, steve.dower, sydefekt, tim.golden, 
zach.ware
priority: normal
severity: normal
status: open
title: MSBuild Extensions in CPython NuGet Package has Bad Expression
type: compile error
versions: Python 3.7, Python 3.8, Python 3.9

___
Python tracker 

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

[issue40156] CodeCov/patch job stills runs on pull requests on 3.5 and 3.6 branches

[STINNER Victor]

Change by STINNER Victor :


--
pull_requests: +18672
pull_request: https://github.com/python/cpython/pull/19309

___
Python tracker 

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

[issue39704] Disable code coverage

[Guido van Rossum]

Change by Guido van Rossum :


--
nosy:  -gvanrossum

___
Python tracker 

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

[issue39837] Remove Azure Pipelines from GitHub PRs

[Brett Cannon]

Brett Cannon  added the comment:

> I'm not aware of Travis CI current issue. There were issues in the past, as 
> with any CI, right ;-) Travis CI looks quite reliable these days.

That's what everyone said when Travis was required and before it went flaky the 
last time. ;)

The point is I don't want to keep flipping on and off required checks based on 
whatever CI people deem flaky or not at any one time.

--

___
Python tracker 

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

[issue40148] Add PurePath.with_stem()

[Brett Cannon]

Brett Cannon  added the comment:

I personally would rather not add more methods that are doing simple string 
manipulations.

--
nosy: +brett.cannon

___
Python tracker 

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

[issue40155] AIX: test_builtin.test_input_no_stdout_fileno() hangs

[Michael Felt]

Change by Michael Felt :


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

___
Python tracker 

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

[issue31160] Enhance support.reap_children()

[Michael Felt]

Change by Michael Felt :


--
pull_requests: +18670
pull_request: https://github.com/python/cpython/pull/19308

___
Python tracker 

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