Skip to content

BLD: fix potential issue with escape sequences in __config__.py#25670

Merged
charris merged 1 commit intonumpy:maintenance/1.26.xfrom
charris:backport-25669
Jan 23, 2024
Merged

BLD: fix potential issue with escape sequences in __config__.py#25670
charris merged 1 commit intonumpy:maintenance/1.26.xfrom
charris:backport-25669

Conversation

@charris
Copy link
Copy Markdown
Member

@charris charris commented Jan 23, 2024

Backport of #25669.

The current code isn't robust to backslashes (from path segments on Windows) in linker flags. No bug report for NumPy, but the same problem did show up in SciPy already. Fix is the same as scipy/scipy#19945.

This actually pulls in all of __config__.py from main because there have been a number of other changes.

[skip cirrus] [skip circle]

@charris charris added 03 - Maintenance 08 - Backport Used to tag backport PRs 36 - Build Build related PR labels Jan 23, 2024
@charris charris added this to the 1.26.4 release milestone Jan 23, 2024
@charris
Copy link
Copy Markdown
Member Author

charris commented Jan 23, 2024

@rgommers I assume that bringing in the whole file from main won't cause problems.

Backport of numpy#25669.

The current code isn't robust to backslashes (from path segments on
Windows) in linker flags. No bug report for NumPy, but the same problem
did show up in SciPy already. Fix is the same as scipy/scipy#19945.

This actually pulls in all of `__config__.py` from main because there
have been a number of other changes.

[skip cirrus] [skip circle]
@charris charris merged commit e48572e into numpy:maintenance/1.26.x Jan 23, 2024
@charris charris deleted the backport-25669 branch January 23, 2024 18:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

03 - Maintenance 08 - Backport Used to tag backport PRs 36 - Build Build related PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants