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
[No Merge] pythonPackages.*: bump minor versions #99636
Conversation
8c65f78
to
ec45df7
Compare
ec45df7
to
7dd03fb
Compare
buildPythonPackage { | ||
pname = "azure-mgmt-commerce"; | ||
version = "1.0.1"; | ||
version = "1.8.2"; | ||
|
||
src = fetchFromGitHub { | ||
owner = "Azure"; | ||
repo = "azure-sdk-for-python"; | ||
rev = "ee5b47525d6c1eae3b1fd5f65b0421eab62a6e6f"; | ||
sha256 = "0xzdn7da5c3q5knh033vbsqk36vwbm75cx8vf10x0yj58krb4kn4"; | ||
rev = "azure-core_${version}"; | ||
sha256 = "1nx0mwkvwy34dpmkp3gbbjxvz12cnj2xr8lrxg1hq9clqkrl9yl5"; | ||
}; |
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.
hmm, large mono repos....
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.
easiest solution I could think of would be to blacklist the repo. Or instead of making a "one-off" exception, just convert these to using fetchPypi
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.
there's only this and one other azure-mgmt package which are affected by it
thoughts? @FRidh
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.
I'm leaning towards just using fetchpypi, as I don't think tests are ran anyway
@@ -8,13 +8,13 @@ assert !useUnrar -> libarchive != null; | |||
|
|||
buildPythonPackage { |
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.
this is missing a rec, but would otherwise be correct
buildPythonPackage { | |
buildPythonPackage rec { |
related PRs have been merged |
Motivation for this change
DO NOT MERGE
follow up to #99635
example of running:
with github fetcher enabled.
This PR will mostly be used to identify script update errors.
Things done
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)