-
-
Notifications
You must be signed in to change notification settings - Fork 17.6k
pyjwkest: remove 'future' dependency to fix build #466877
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
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
Author
|
nixpkgs-review said: |
Aleksanaa
reviewed
Dec 2, 2025
Aleksanaa
approved these changes
Dec 2, 2025
This dependency is unused in pyjwkest, but disabled in nixpkgs. There is an upstream pull request pending to remove the dependency: IdentityPython/pyjwkest#107 Before this change, these 2 Python packages failed to build: nix build .#python313Packages.pyjwkest nix build .#python313Packages.oic With the following message: error: … while evaluating the attribute 'drvPath' at /nix/store/65sypvf1f26fyi1vq6h25wg9z5gipgy8-source/lib/customisation.nix:429:7: 428| // { 429| drvPath = | ^ 430| assert condition; … while evaluating the attribute 'drvPath' at /nix/store/65sypvf1f26fyi1vq6h25wg9z5gipgy8-source/lib/customisation.nix:429:7: 428| // { 429| drvPath = | ^ 430| assert condition; … while calling the 'derivationStrict' builtin at <nix/derivation-internal.nix>:37:12: 36| 37| strict = derivationStrict drvAttrs; | ^ 38| (stack trace truncated; use '--show-trace' to show the full, detailed trace) error: future-1.0.0 not supported for interpreter python3.13 (I am using the oic package via the pretix-oidc plugin.)
Contributor
Author
|
nixpkgs-review output for the updated PR: |
Member
|
Contributor
|
Successfully created backport PR for |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
6.topic: python
Python is a high-level, general-purpose programming language.
8.has: port to stable
This PR already has a backport to the stable release.
10.rebuild-darwin: 1-10
This PR causes between 1 and 10 packages to rebuild on Darwin.
10.rebuild-linux: 1-10
This PR causes between 1 and 10 packages to rebuild on Linux.
12.approvals: 1
This PR was reviewed and approved by one person.
backport release-25.11
Backport PR automatically
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This dependency is unused in pyjwkest, but disabled in nixpkgs. There is an upstream pull request pending to remove the dependency: IdentityPython/pyjwkest#107
Before this change, these 2 Python packages failed to build:
With the following message:
(I am using the oic package via the pretix-oidc plugin.)
Things done
passthru.tests.nixpkgs-reviewon this PR. See nixpkgs-review usage../result/bin/.Add a 👍 reaction to pull requests you find important.