Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feature]: Upload sdists to pypi #2579

Open
alex opened this issue Oct 1, 2024 · 2 comments
Open

[Feature]: Upload sdists to pypi #2579

alex opened this issue Oct 1, 2024 · 2 comments

Comments

@alex
Copy link

alex commented Oct 1, 2024

🚀 Feature Request

Currently only wheels are uploaded to pypi, but sdists are not. It would be helpful if sdists were supported.

Example

No response

Motivation

Homebrew always builds packages from source. As a result, it cannot download playwright from PyPI, and instead needs download the source out-of-band from the git tag. This means that formulas which use playwright cannot particulate in automatic version upgrades.

If the sdists were uploaded to PyPI, this would make keeping python formulas that use playwright up to date dramatically easier (including resolving security issues).

@mxschmitt
Copy link
Member

Are you interested in publishing Playwright for Python to homebrew or how do you stumble over that?

Inside the wheels there is the matching Node.js binary for the OS/arch combination which Playwright internally is using. In theory we could upload the sdist but we didn't, since we had no need for it and it was like a protection for users, that if there was no matching wheel available, their OS/arch wasn't supported. Pypy gave usually a nice error message for that.

@alex
Copy link
Author

alex commented Oct 1, 2024

playwright-python is currently a dependency of one homebrew package: https://github.com/Homebrew/homebrew-core/blob/master/Formula/t/theharvester.rb#L202-L205

As you can see, for all the other dependencies, we download sdists from PyPI, but for this we download the source from github. This has the unfortunate problem of interfering with our automation for automatically handling dependencies.

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

No branches or pull requests

2 participants