Skip to content

libmtp: fix built against gettext-0.25#421167

Merged
alyssais merged 1 commit intoNixOS:stagingfrom
trofi:libmtp-gettext-0.25
Jul 2, 2025
Merged

libmtp: fix built against gettext-0.25#421167
alyssais merged 1 commit intoNixOS:stagingfrom
trofi:libmtp-gettext-0.25

Conversation

@trofi
Copy link
Contributor

@trofi trofi commented Jun 30, 2025

Without the change the build fails as:

configure.ac:265: the top level
configure:14220: error: possibly undefined macro: AC_LIB_PREPARE_PREFIX
  If this token and others are legitimate, please use m4_pattern_allow.
  See the Autoconf documentation.

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/)
  • Nixpkgs 25.11 Release Notes (or backporting 24.11 and 25.05 Nixpkgs Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
  • NixOS 25.11 Release Notes (or backporting 24.11 and 25.05 NixOS Release notes)
    • (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, pkgs/README.md, maintainers/README.md and other contributing documentation in corresponding paths.

Add a 👍 reaction to pull requests you find important.

@trofi trofi mentioned this pull request Jun 30, 2025
13 tasks
@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-linux: 101-500 This PR causes between 101 and 500 packages to rebuild on Linux. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. labels Jun 30, 2025
@nix-owners nix-owners bot requested a review from lovesegfault June 30, 2025 06:27
Copy link
Member

Choose a reason for hiding this comment

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

Usually we'd write this on multiple lines…

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sounds good. Switched to command per line

Without the change the build fails as:

    configure.ac:265: the top level
    configure:14220: error: possibly undefined macro: AC_LIB_PREPARE_PREFIX
      If this token and others are legitimate, please use m4_pattern_allow.
      See the Autoconf documentation.
@trofi trofi force-pushed the libmtp-gettext-0.25 branch from 4fe4a19 to 415af28 Compare June 30, 2025 07:07
@nixpkgs-ci nixpkgs-ci bot added the 12.approvals: 1 This PR was reviewed and approved by one person. label Jun 30, 2025
@alyssais
Copy link
Member

@ofborg build libmtp

@alyssais alyssais merged commit be84b99 into NixOS:staging Jul 2, 2025
30 of 32 checks passed
@trofi trofi deleted the libmtp-gettext-0.25 branch July 3, 2025 05:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 101-500 This PR causes between 101 and 500 packages to rebuild on Linux. 12.approvals: 1 This PR was reviewed and approved by one person.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants