BLD: revert adding PEP 621 metadata, it confuses setuptools#22687
BLD: revert adding PEP 621 metadata, it confuses setuptools#22687mattip merged 3 commits intonumpy:mainfrom
Conversation
|
Argh, now |
eli-schwartz
left a comment
There was a problem hiding this comment.
Random bash scripting observation.
bfcc9dc to
3c880a0
Compare
3c880a0 to
1b47584
Compare
|
Current test failure is: I'm confused; I don't see any mention of |
|
@stefanv we just added |
|
Ahh, I guess weird PR timing there, the meson config just doesn't know about |
|
OK, I just added |
|
Can you remove the upload_wheels changes? That will be part of #22690. |
1b47584 to
fe31f6b
Compare
Done. |
|
This is probably ready to merge then? |
|
Edit: yes, it is in the |
|
So with this change it appears that strictly We have some jobs in pandas to test with the nightly wheels and are encountering this error https://github.com/pandas-dev/pandas/actions/runs/3589194341/jobs/6041379775 |
|
That action indeed includes the changes from this PR. It's unlikely that this caused the problem though - this reverted an earlier changed, and all that is left in
The real question here is why you're getting a build from source, rather than picking up a wheel. |
|
The problem is that some macOS wheel builds failed, but an sdist was uploaded to the nightly bucket. @charris @mattip I don't think we should upload an sdist to nightlies at all, that is going to result in these kinds of problems as Pandas is seeing. If a nightly wheel is missing for some platform, one should get a wheel built a week before. |
|
@mroeschke I've deleted the most recent sdist, so I think your next CI run should not fail. Could you confirm that after it has run? |
|
Ah thanks @rgommers. A recent build has started to pass now! https://github.com/pandas-dev/pandas/actions/runs/3596938344/jobs/6058167508 |
| from setuptools import setup | ||
| # workaround for broken --no-build-isolation with newer setuptools, | ||
| # see gh-21288 | ||
| metadata["packages"] = [] |
There was a problem hiding this comment.
Does this need a backport to 1.24.x?
There was a problem hiding this comment.
Sorry I missed this question. Yes, I think it's a good idea to backport this change. I'll open a PR now.
There was a problem hiding this comment.
(bit late, sorry about this. it can go into the next bug fix release)
See https://github.com/numpy/numpy/pull/22663/files#r1034053192.
The one
setup.pytweak is to even make it possible to test newersetuptoolsversions without build isolation (applies fix suggested in gh-21288).