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
Fixing/adding python27 derivations #93978
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cc @FRidh
GitPython Venusian Uritools Sphinxcontrib-blockdiag - Adding Protobuf 3.3.0 derivation - Fix tremor repository which was moved to GitLab
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To comply with CONTRIBUTING.md please have the commit messages name be of the format
<pkg-name>: <subject-line>
for more examples, please look at https://github.com/NixOS/nixpkgs/blob/master/.github/CONTRIBUTING.md#submitting-changes
please split up the changes into a commit to package affected. having a single large commit makes it difficult to discern what change potentially caused a regression. Also makes cherry-picking between branches very difficult.
also: When backporting changes, please follow https://github.com/NixOS/nixpkgs/blob/master/.github/CONTRIBUTING.md#backporting-changes. Namely, you should be doing |
I'm completely ignorant about this step. Could you give me more specific instructions pretty please?
From here I don't know what should I do. Any help is much appreciated, thanks so much! |
backports are applied to master branch, then the release branch (you are going directly to release branch) for backport PRs, you want to target |
Motivation for this change
As the channel keeps supporting python27 some derivations need to be fixed or added to still working is this python version.
Things done
Add python27 support for the following derivations:
GitPython
Venusian
Uritools
Sphinxcontrib-blockdiag
Adding Protobuf 3.3.0 derivation
Fix tremor repository which was moved to GitLab
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)