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.python-bitcoinlib: rename from bitcoinlib #358492

Merged
merged 1 commit into from
Nov 26, 2024

Conversation

cooparo
Copy link

@cooparo cooparo commented Nov 23, 2024

It think is confusing calling this package bitcoinlib because this is the name of a different package on the Python Package Index, so maybe it's better to use the same name.

See:

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

Add a 👍 reaction to pull requests you find important.

@NixOSInfra NixOSInfra added the 12. first-time contribution This PR is the author's first one; please be gentle! label Nov 23, 2024
Copy link
Member

@dotlambda dotlambda left a comment

Choose a reason for hiding this comment

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

Please also change the attribute name in pkgs/top-level/python-packages.nix and replace all occurrences treewide.

@nix-owners nix-owners bot requested a review from natsukium November 24, 2024 09:15
@cooparo
Copy link
Author

cooparo commented Nov 24, 2024

Please also change the attribute name in pkgs/top-level/python-packages.nix and replace all occurrences treewide.

Now should be okay

@dotlambda
Copy link
Member

Could you make the commit message say something like "rename to python-bitcoinlib"?

@cooparo
Copy link
Author

cooparo commented Nov 24, 2024

Could you make the commit message say something like "rename to python-bitcoinlib"?

I dont't understand, the commit message is "refactor pname to python-bitcoinlib" already, what else should I write?

@ofborg ofborg bot added 8.has: clean-up 8.has: package (new) This PR adds a new package labels Nov 25, 2024
@con-f-use con-f-use self-requested a review November 25, 2024 14:10
@dotlambda dotlambda changed the title python312Packages.bitcoinlib: refactor pname to python-bitcoinlib python312Packages.bitcoinlib: rename to python-bitcoinlib Nov 25, 2024
@dotlambda dotlambda changed the title python312Packages.bitcoinlib: rename to python-bitcoinlib python312Packages.python-bitcoinlib: rename from bitcoinlib Nov 25, 2024
@dotlambda
Copy link
Member

Could you make the commit message say something like "rename to python-bitcoinlib"?

I dont't understand, the commit message is "refactor pname to python-bitcoinlib" already, what else should I write?

My suggestion is in the PR title. The problem is that we're not only changing the pname.

@cooparo cooparo force-pushed the update-python312Packages.bitcoinlib branch from e85694c to 000028d Compare November 26, 2024 10:24
@cooparo
Copy link
Author

cooparo commented Nov 26, 2024

I think I have just merged my inability to use git and my inability to spell correctly bitcoinlib. In the end, I hope did right.

@dotlambda
Copy link
Member

Please squash your commits.

@cooparo cooparo force-pushed the update-python312Packages.bitcoinlib branch from d7b6a67 to 7233ce0 Compare November 26, 2024 16:23
@cooparo
Copy link
Author

cooparo commented Nov 26, 2024

Ok, should be good (hope it's the last time I say it)

@dotlambda
Copy link
Member

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 358492


x86_64-linux

✅ 12 packages built:
  • opentimestamps-client
  • opentimestamps-client.dist
  • prometheus-bitcoin-exporter
  • python311Packages.opentimestamps
  • python311Packages.opentimestamps.dist
  • python311Packages.python-bitcoinlib
  • python311Packages.python-bitcoinlib.dist
  • python312Packages.opentimestamps
  • python312Packages.opentimestamps.dist
  • python312Packages.python-bitcoinlib
  • python312Packages.python-bitcoinlib.dist
  • zeronet-conservancy

@dotlambda dotlambda merged commit b70d51f into NixOS:master Nov 26, 2024
16 of 17 checks passed
@cooparo cooparo deleted the update-python312Packages.bitcoinlib branch November 26, 2024 17:53
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.

4 participants