Skip to content

Conversation

@rgommers
Copy link
Member

@rgommers rgommers commented Oct 14, 2024

@rgommers rgommers added component: build Meson Items related to the introduction of Meson as the new build system for NumPy labels Oct 14, 2024
@rgommers rgommers marked this pull request as draft October 14, 2024 12:00
@github-actions github-actions bot added the 36 - Build Build related PR label Oct 14, 2024
@rgommers
Copy link
Member Author

The only CI failures are unrelated (due to gh-27548). The BLAS detection in those two failing jobs worked fine, so this looks about ready.

Will add another fix and retest.

@charris charris added the 09 - Backport-Candidate PRs tagged should be backported label Oct 14, 2024
@charris
Copy link
Member

charris commented Oct 14, 2024

Tagged this for backport, I will be making at least one more 2.1 release.

@charris
Copy link
Member

charris commented Oct 14, 2024

Is this based off current main? The macos failure might go away with a rebase.

@rgommers
Copy link
Member Author

rgommers commented Oct 14, 2024

Yeah it's up-to-date, only a few commits behind. The macOS conda-forge failure is coming in with the AIX linker change in the vendored Meson it looked like, the job is passing on the first commit in this PR EDIT: looks like it was flaky - it's failing again. Still puzzled by it though, not sure what's going on there.

I'll drop the change for now - the primary goal here is to update to Meson 1.5.x and get free-threading on Windows going.

@rgommers
Copy link
Member Author

Ugh, the macOS failure is a flake that just happened to also touch on icc. It shows up on gh-27544 from 3 days ago as well, and also on several other issues from years ago (e.g., gh-17635). None of the other issues have an explanation.

@mattip
Copy link
Member

mattip commented Oct 14, 2024

I see the test run is using xdist with two workers. Is there something in the test that is not thread safe? Maybe some other test is importing and changing sys.modules in parallel with this one?

@mattip
Copy link
Member

mattip commented Oct 14, 2024

The failures are in numpy.distutils which is deprecated. Can we avoid testing whether any of its API has changed?

@charris
Copy link
Member

charris commented Oct 14, 2024

macOS failure is a flake

The reason I ask is that it was common until a cache update. See #27532.

@rgommers
Copy link
Member Author

The reason I ask is that it was common until a cache update.

Hmm, not impossible, although I think my initial PR included that fix already (it was anchored on a commit only 2 days old). That said, your "wrong setuptools version" is quite plausible.

The failures are in numpy.distutils which is deprecated. Can we avoid testing whether any of its API has changed?

An easy way to do that would be to bump this CI job to use Python 3.12 (assuming nothing else is broken there). I'll check that.

@mattip
Copy link
Member

mattip commented Oct 14, 2024

Did you mean to change this?

python-version: ["3.11"]

@charris
Copy link
Member

charris commented Oct 14, 2024

Note that the problems seemed to start with a new release of 3.11.

@rgommers
Copy link
Member Author

Did you mean to change this?

Yes indeed. Done in gh-27555.

Will be included upstream in Meson 1.6.0; cherry-picked in
numpy/meson#17.
@rgommers rgommers added the 39 - free-threading PRs and issues related to support for free-threading CPython (a.k.a. no-GIL, PEP 703) label Oct 15, 2024
@rgommers rgommers marked this pull request as ready for review October 15, 2024 09:08
@rgommers
Copy link
Member Author

Okay, this is finally ready now. I updated the PR description with what it adds/fixes. Should be back-portable for 2.1.3.

@charris
Copy link
Member

charris commented Oct 15, 2024

Thanks Ralf.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

36 - Build Build related PR 39 - free-threading PRs and issues related to support for free-threading CPython (a.k.a. no-GIL, PEP 703) component: build Meson Items related to the introduction of Meson as the new build system for NumPy

Projects

None yet

Development

Successfully merging this pull request may close these issues.

numpy 2.0.1 build issue on ibm AIX OS

3 participants