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

Missing setup.py on releases? #305

Closed
luizribeiro opened this issue Jul 26, 2023 · 3 comments
Closed

Missing setup.py on releases? #305

luizribeiro opened this issue Jul 26, 2023 · 3 comments

Comments

@luizribeiro
Copy link

While packaging tomlkit 0.11.8 for NixOS we noticed that starting on 0.11.7 the released packages no longer include setup.py (see full discussion on NixOS/nixpkgs#245440).

While setup.py is no longer actually necessary, this creates a circular dependency to build tomlkit moving forward - as tomlkit itself would require poetry to be able to build tomlkit, while poetry itself depends on tomlkit to be built.

Was the removal of setup.py intended?

@luizribeiro
Copy link
Author

Ah, it seems like this happened because of python-poetry/poetry-core#318

I guess we could explicitly set tool.poetry.build.generate-setup-file here so we can still build tomlkit without poetry?

I'm going to put up a PR for that.

luizribeiro added a commit to luizribeiro/tomlkit that referenced this issue Jul 26, 2023
@frostming
Copy link
Contributor

frostming commented Jul 27, 2023

this creates a circular dependency to build tomlkit moving forward - as tomlkit itself would require poetry to be able to build tomlkit, while poetry itself depends on tomlkit to be built.

This is not true, tomlkit only requires poetry-core to build and the latter has zero-dependency, it doesn't vendor tomlkit either. You may need to investigate carefully to find out the real cause. Or otherwise you may have misconfigured your build process.

@luizribeiro
Copy link
Author

luizribeiro commented Jul 27, 2023

Thank you for the clarification, @frostming!

It looks like nixpkgs has the wrong dependencies for poetry-core indeed.

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

No branches or pull requests

2 participants