Skip to content

Conversation

uninsane
Copy link
Contributor

@uninsane uninsane commented Jul 29, 2025

also includes some changes to rdma-core and unixtools.sysctl necessary to avoid dependency cycles.

Things done

  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
    • NixOS tests in nixos/tests. nixosTests.{ulogd,curl-impersonate,libreswan,keepalived,libreswan-nat,ndppd} (these tests all use tcpdump, a CLI interface to libpcap)
    • Package tests at passthru.tests.
    • Tests in lib/tests or pkgs/test for functions and "core" functionality.
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

Add a 👍 reaction to pull requests you find important.

uninsane added 4 commits July 29, 2025 00:06
and move bluez to the correct `buildInputs` set (not
`nativeBuildInputs`)
- enable `strictDeps`
- use `lib.enableFeature` where applicable
- sort package lists alphabetically
ethtool and iproute2 are listed in the "recommends" section
of upstream's [[packaging docs]]. they aren't actually used by
rdma-core, and don't appear in its runtime closure.
rather, these are packages which are useful to RDMA
developers (e.g. iproute2's `rdma` command).

[[packaging docs]]: https://github.com/linux-rdma/rdma-core/blob/f5442854eccd4cc9d1f20bf200171c26e6859381/debian/control#L32
@nixpkgs-ci nixpkgs-ci bot added the 2.status: merge conflict This PR has merge conflicts with the target branch label Aug 14, 2025
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant