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

py-cryptography: does not run-depend on py-setuptools-rust #39386

Merged
merged 3 commits into from
Aug 11, 2023

Conversation

wdconinc
Copy link
Contributor

As of 3.4.2, pyca/cryptography#5779 and backport pyca/cryptography#5781, py-cryptography does not have a runtime dependency on py-setuptools-rust anymore, only a build dependency. Version 3.4.1 is the last version with install_requires=setup_requirements. Since the older versions after 3.4 but before 3.4.2 are not in spack, no specific exception is added to require the runtime dependency for 3.4:3.4.1.

(Yay, rust is garbage collected away again.)

adamjstewart
adamjstewart previously approved these changes Aug 11, 2023
Copy link
Member

@adamjstewart adamjstewart left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Safe to merge but also wouldn't mind being more specific since we've already done the hard work of determining when this is actually true.

Co-authored-by: Adam J. Stewart <ajstewart426@gmail.com>
@adamjstewart adamjstewart enabled auto-merge (squash) August 11, 2023 17:41
@adamjstewart adamjstewart merged commit 63bb2c9 into spack:develop Aug 11, 2023
13 checks passed
@wdconinc wdconinc deleted the wdconinc-patch-1 branch August 11, 2023 18:00
mpokorny pushed a commit to mpokorny/spack that referenced this pull request Sep 18, 2023
* py-cryptography: does not run-depend on py-setuptools-rust

* py-cryptography: depens_on py-setuptools-rust when @3.4.2:

Co-authored-by: Adam J. Stewart <ajstewart426@gmail.com>

* py-cryptography: re-add depends_on type=run for narrow range

---------

Co-authored-by: Adam J. Stewart <ajstewart426@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants