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

cargo-hakari: 0.9.33 → 0.9.35 #371399

Closed
wants to merge 4 commits into from
Closed

cargo-hakari: 0.9.33 → 0.9.35 #371399

wants to merge 4 commits into from

Conversation

Rishik-Y
Copy link
Contributor

@Rishik-Y Rishik-Y commented Jan 6, 2025

#371364

Maintainer Ping

@figsoda @macalinao @Nartsiss

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.

@momeemt
Copy link
Member

momeemt commented Jan 6, 2025

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 371399


aarch64-darwin

❌ 1 package failed to build:
  • cargo-hakari


src = fetchFromGitHub {
owner = "guppy-rs";
repo = "guppy";
rev = "cargo-hakari-${version}";
sha256 = "sha256-oJZiGXsOl00Bim/olYYSqt/p3j6dTw25IURcwdXYrAo=";
sha256 = "sha256-xMgNk82U32AtMeZJ8v2M0faHLAPkXrdIJHwOJcqE9m8=";
Copy link
Member

Choose a reason for hiding this comment

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

error: hash mismatch in fixed-output derivation '/nix/store/yq4m71a5lmfxjz0qrr9a6ccxcj58292g-source.drv':
         specified: sha256-xMgNk82U32AtMeZJ8v2M0faHLAPkXrdIJHwOJcqE9m8=
            got:    sha256-+IjtK4kSm2vThgIxDsBLpoh0j9cDhhEqI6Hr2BmC7hc=
error: 1 dependencies of derivation '/nix/store/b5hfzv01rhdczwqhxkk5qd63ay87y79m-cargo-hakari-0.9.35-vendor.tar.gz.drv' failed to build
error: 2 dependencies of derivation '/nix/store/vvmw3i2j7pjfk3qyls2qakbbflhrxmbc-cargo-hakari-0.9.35.drv' failed to build
error: 1 dependencies of derivation '/nix/store/09c934x64sh5r66if5gqvsv354qbwsds-review-shell.drv' failed to build

The src hash of guppy-rs may be incorrect.

Copy link
Contributor

Choose a reason for hiding this comment

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

See also #331244 which I think updates both

@jonhoo
Copy link
Contributor

jonhoo commented Jan 8, 2025

@momeemt mind retrying your review?

@nartsisss
Copy link
Member

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 371399


aarch64-darwin

✅ 1 package built:
  • cargo-hakari

@nartsisss
Copy link
Member

nartsisss commented Jan 8, 2025

Please, squash your commits and force-push branch


src = fetchFromGitHub {
owner = "guppy-rs";
repo = "guppy";
rev = "cargo-hakari-${version}";
sha256 = "sha256-oJZiGXsOl00Bim/olYYSqt/p3j6dTw25IURcwdXYrAo=";
sha256 = "sha256-+IjtK4kSm2vThgIxDsBLpoh0j9cDhhEqI6Hr2BmC7hc=";
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
sha256 = "sha256-+IjtK4kSm2vThgIxDsBLpoh0j9cDhhEqI6Hr2BmC7hc=";
hash = "sha256-+IjtK4kSm2vThgIxDsBLpoh0j9cDhhEqI6Hr2BmC7hc=";

Copy link
Member

@nartsisss nartsisss Jan 8, 2025

Choose a reason for hiding this comment

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

We need to use hash, since we have already switched to the SRI hash format.

@@ -6,16 +6,16 @@

rustPlatform.buildRustPackage rec {
pname = "cargo-hakari";
version = "0.9.33";
version = "0.9.35";

src = fetchFromGitHub {
owner = "guppy-rs";
repo = "guppy";
rev = "cargo-hakari-${version}";
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
rev = "cargo-hakari-${version}";
tag = "cargo-hakari-${version}";

@jonhoo
Copy link
Contributor

jonhoo commented Jan 9, 2025

@Rishik-Y I'd be happy to open a new PR with the updated + squashed changes if you're short on time :)

@momeemt
Copy link
Member

momeemt commented Jan 9, 2025

@momeemt mind retrying your review?

As nartsisss-san said, I think these changes commits should be squashed a commit. It will be finished, then I would like to approve this pr 👍

@jonhoo jonhoo mentioned this pull request Jan 9, 2025
13 tasks
@jonhoo
Copy link
Contributor

jonhoo commented Jan 9, 2025

Updated and squashed in #372314

@Rishik-Y
Copy link
Contributor Author

Rishik-Y commented Jan 9, 2025

@Rishik-Y I'd be happy to open a new PR with the updated + squashed changes if you're short on time :)

I am so sorry, i was indeed short on time, figuring out this issue on my part, so was unable to focus on other task
I will close this PR as you have fixed it

@Rishik-Y Rishik-Y closed this Jan 9, 2025
@jonhoo
Copy link
Contributor

jonhoo commented Jan 9, 2025

All good, thanks for bringing it forward in the first place anyway, and hope you resolved your issue!

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