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

rpm: remove darwin from supported platforms #196350

Merged
merged 1 commit into from
Oct 17, 2022
Merged

Conversation

viraptor
Copy link
Contributor

Description of changes

Since 4.18.0, rpm does not compile on platforms without mkfifoat (so for example macos).
This can be rectified by either providing a patch that reverts the at* change for !stdenv.isLinux, or by removing darwin from supported platforms.

What do you think @copumpkin ?

Things done
  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • 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/)
  • 22.11 Release Notes (or backporting 22.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
    • (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes
  • Fits CONTRIBUTING.md.

@r-burns
Copy link
Contributor

r-burns commented Oct 17, 2022

Upstream issue here rpm-software-management/rpm#2222

@Mic92
Copy link
Member

Mic92 commented Oct 17, 2022

I think it's fair to drop platform support until upstream released a fix.

@Mic92 Mic92 merged commit 6179b27 into NixOS:master Oct 17, 2022
cbourjau added a commit to cbourjau/nixpkgs that referenced this pull request Dec 12, 2022
RPM is no longer available for
Darwin ( NixOS#196350 ). Libsolv is used by downstream
packages such as micromamba which have no requirement on RPM.
@glensc
Copy link

glensc commented Jan 4, 2023

upstream doesn't plan to support macos < 13. read the notes from the upstream issue link

cbourjau added a commit to cbourjau/nixpkgs that referenced this pull request Jan 15, 2023
RPM is no longer available for
Darwin ( NixOS#196350 ). Libsolv is used by downstream
packages such as micromamba which have no requirement on RPM.

(cherry picked from commit a5160ce)
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