-
-
Notifications
You must be signed in to change notification settings - Fork 36
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
Able to set version using environment variable #62
Comments
In the sdist generated by If the sdist still required the plugin for some reason and there were something like an environment variable as a workaround, I'm curious if you'd have a problem when you're building project A and it depends on project B that also uses poetry-dynamic-versioning. Would they end up with the same version? Do you always get the dependencies from wheels, even if that means building them from the sdist separately and one-by-one first? |
sorry, I forgot to mention the need for using source code as well. As many maintainers don't export tests in their sdist; but tests are invaluable when you need to tweak dependency ranges to make a coherent package set. |
Hey just wanted to mention that this is blocking me right now. We sometimes install a package as a local-editable package into docker images. But these docker images do not get the full git repo. I need a way to set the version through an environment variable and effectively disable this plugin for the one Cheers! |
Sdists are often neglected. Example: https://github.com/NixOS/nixpkgs/pull/145528/files#r747757644
No, each dependency is built in their own sandbox.
We almost never use wheels, because nix doesn't assume FHS, so we generally end up needing to groom and patch things before installation. Hard to apply patches after a wheel is created, or the package is installed. Current practice that I've been doing is this: https://github.com/NixOS/nixpkgs/blob/83a144c2014f67366357da9b32cc0d5a1554f448/pkgs/tools/misc/copier/default.nix#L47-L53. Which is letting the version be defaulted to |
Alright, that makes sense. Thanks for explaining further. I'm open to adding this. |
This is now implemented. I'm planning to get a release out within the next few days. |
@mtkennerly thanks a lot. Distro maintainers everywhere will thank you :) |
For package maintainers, we often receive the sdist, which is devoid of any vcs information, so we need another way to communicate the version number of the package to the build toolchain. For
setuptools-scm
, this isSETUPTOOLS_SCM_PRETEND_VERSION
, however, I'm not aware of an analogous feature for this package.To be clear, this use case is when the version is already determined, and we are just following upstream has distributed.
The text was updated successfully, but these errors were encountered: