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

shntool: 3.0.10 -> 3.0.10+git20130108.4ca41f4-1 #357635

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tdeebswihart
Copy link

@tdeebswihart tdeebswihart commented Nov 20, 2024

Things done

I'm updating shntool to source its code from the Debian folks and to use their patches; I encountered issues using shntool that were fixed by these.

While here I updated its homepage to the debian tracker; the original homepage now returns a 404.

Full details about the Debian package for this can be found on their tracker.

  • 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: nix-shell --option sandbox true -E "with import <nixpkgs> {}; callPackage ./package.nix {}"
  • 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 20, 2024
@tdeebswihart
Copy link
Author

I attempted to review this using nixpkgs-review but I don't have a graphical Linux environment in which to test the flacon package, which is the only one that depends on this.

Copy link
Contributor

@jcumming jcumming left a comment

Choose a reason for hiding this comment

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

Thanks for submitting this!

Changes look good.

I haven't tested it with my workflows; in the unlikely chance something breaks we can fix it up later.

pkgs/by-name/sh/shntool/package.nix Outdated Show resolved Hide resolved
@tdeebswihart
Copy link
Author

Thanks for submitting this!

Changes look good.

I haven't tested it with my workflows; in the unlikely chance something breaks we can fix it up later.

Sounds good! I've removed the commented-out patches line (that predated pulling the debian sources directly), so it's good to go

@jcumming
Copy link
Contributor

@NixOS/nixpkgs-merge-bot merge

@nixpkgs-merge-bot
Copy link
Contributor

@jcumming merge not permitted (#305350):
pr author is not in restricted authors list, in the list are: r-ryantm

@ofborg ofborg bot requested a review from jcumming November 22, 2024 15:35
@Aleksanaa Aleksanaa changed the title Update shntool to patch bugs shntool: 3.0.10 -> 3.0.10+git20130108.4ca41f4-1 Nov 23, 2024
@Aleksanaa
Copy link
Member

@ofborg build shntool

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