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

python312Packages.dns-lexicon: 3.16.1 -> 3.17.0 #331985

Merged

Conversation

r-ryantm
Copy link
Contributor

@r-ryantm r-ryantm commented Aug 3, 2024

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.

meta.description for python312Packages.dns-lexicon is: Manipulate DNS records on various DNS providers in a standardized way

meta.homepage for python312Packages.dns-lexicon is: https://github.com/AnalogJ/lexicon

meta.changelog for python312Packages.dns-lexicon is: https://github.com/AnalogJ/lexicon/blob/v3.17.0/CHANGELOG.md

Updates performed
  • Ran passthru.UpdateScript
To inspect upstream changes
Impact

Checks done


  • built on NixOS
  • The tests defined in passthru.tests, if any, passed
  • found 3.17.0 with grep in /nix/store/852jqkriz0zz8i8biv2msbj1sfhpl48p-python3.12-dns_lexicon-3.17.0
  • found 3.17.0 in filename of file in /nix/store/852jqkriz0zz8i8biv2msbj1sfhpl48p-python3.12-dns_lexicon-3.17.0

Rebuild report (if merged into master) (click to expand)
7 total rebuild path(s)

7 package rebuild(s)

First fifty rebuilds by attrpath

certbot-full
lexicon
python311Packages.certbot-dns-ovh
python311Packages.dns-lexicon
python312Packages.certbot-dns-ovh
python312Packages.dns-lexicon
Instructions to test this update (click to expand)

Either download from Cachix:

nix-store -r /nix/store/852jqkriz0zz8i8biv2msbj1sfhpl48p-python3.12-dns_lexicon-3.17.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.

Or, build yourself:

nix-build -A python312Packages.dns-lexicon https://github.com/r-ryantm/nixpkgs/archive/c9d2f9ea12cfb6a3c1a1dd57aa8ac8a3f6882d17.tar.gz

Or:

nix build github:r-ryantm/nixpkgs/c9d2f9ea12cfb6a3c1a1dd57aa8ac8a3f6882d17#python312Packages.dns-lexicon

After you've downloaded or built it, look at the files and if there are any, run the binaries:

ls -la /nix/store/852jqkriz0zz8i8biv2msbj1sfhpl48p-python3.12-dns_lexicon-3.17.0
ls -la /nix/store/852jqkriz0zz8i8biv2msbj1sfhpl48p-python3.12-dns_lexicon-3.17.0/bin


Pre-merge build results

We have automatically built all packages that will get rebuilt due to
this change.

This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.

Result of nixpkgs-review run on x86_64-linux 1

5 packages failed to build:
  • certbot-full
  • python311Packages.certbot-dns-ovh
  • python311Packages.certbot-dns-ovh.dist
  • python312Packages.certbot-dns-ovh
  • python312Packages.certbot-dns-ovh.dist
4 packages built:
  • lexicon (python312Packages.dns-lexicon)
  • lexicon.dist (python312Packages.dns-lexicon.dist)
  • python311Packages.dns-lexicon
  • python311Packages.dns-lexicon.dist

Maintainer pings

cc @aviallon for testing.

@ofborg ofborg bot requested a review from aviallon August 3, 2024 13:42
@drupol drupol merged commit 37afb62 into NixOS:master Aug 4, 2024
27 of 29 checks passed
@r-ryantm r-ryantm deleted the auto-update/python312Packages.dns-lexicon branch August 5, 2024 00:21
@aviallon
Copy link
Contributor

aviallon commented Aug 7, 2024

@drupol why did you merge, certbot-dns-ovh (the only thing using it) fails to build with this updated dependency.

@drupol
Copy link
Contributor

drupol commented Aug 7, 2024

I'll provide the revert asap. Not in front of a computer at the moment.

@drupol
Copy link
Contributor

drupol commented Aug 8, 2024

An issue has been opened at certbot/certbot#9982

Now going to provide the revert PR.

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.

3 participants