Skip to content

readline: Cleanup for staging#280119

Open
Ericson2314 wants to merge 2 commits intoNixOS:stagingfrom
obsidiansystems:readline-mingw-staging
Open

readline: Cleanup for staging#280119
Ericson2314 wants to merge 2 commits intoNixOS:stagingfrom
obsidiansystems:readline-mingw-staging

Conversation

@Ericson2314
Copy link
Member

Description of changes

We clean up #280099 by making the the mingw patches unconditional, using the slight modifications to one patch in msys2/MINGW-packages#19701 to do so.

This is a separate commit so so #280099 can go to master right away.

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/)
  • 24.05 Release Notes (or backporting 23.05 and 23.11 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.

Ericson2314 and others added 2 commits January 10, 2024 14:31
This commit is specifically designed to avoid a mass rebuild. I'll make
a follow-up PR to staging which will make the patches unconditional,
etc.

Co-Authored-By: Weijia Wang <9713184+wegank@users.noreply.github.com>
We clean up NixOS#280099 by making the the mingw patches unconditional, using
the slight modifications to one patch in
msys2/MINGW-packages#19701 to do so.

This is a separate commit so so NixOS#280099 can go to master right away.
@Ericson2314 Ericson2314 changed the title Readline mingw staging readline: Cleanup for staging Jan 10, 2024
@Ericson2314 Ericson2314 requested a review from wegank January 10, 2024 23:12
@Ericson2314
Copy link
Member Author

Is the failure because this is too early for fetchpatch or something?

@vcunat
Copy link
Member

vcunat commented Feb 3, 2024

@ofborg eval

@wegank wegank added the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label Jan 2, 2025
@wegank wegank added the 2.status: merge conflict This PR has merge conflicts with the target branch label Mar 16, 2025
@stale stale bot removed the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label Mar 16, 2025
@nixpkgs-ci nixpkgs-ci bot added the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label Jun 25, 2025
@xokdvium xokdvium mentioned this pull request Dec 19, 2025
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2.status: merge conflict This PR has merge conflicts with the target branch 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants