Make nix package derivations overridable#1371
Conversation
- Make packages overridable - Make overlays use the previous pkgs version to build packages - Make home-manager and nixos modules use ambient nixpkgs for building - Add option gitBasePackage to nixos and home-manager modules to allow overriding base git package through configuration
|
|
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
|
Wow, got scared by the CLA requirement. |
Co-authored-by: devin-ai-integration[bot] <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-authored-by: devin-ai-integration[bot] <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-authored-by: devin-ai-integration[bot] <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-authored-by: devin-ai-integration[bot] <158243242+devin-ai-integration[bot]@users.noreply.github.com>
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
Co-authored-by: devin-ai-integration[bot] <158243242+devin-ai-integration[bot]@users.noreply.github.com>
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
…s always available when the package is called
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
Now override functions are exposed through the 'scope' attribute
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
- Fix `override` function being called from git-ai attrset instead of its enclosing scope - Fix `wrapped` package being no longer exposed under `packages` - Fix `callPackage` on default.nix shadowing `makeScope`s `override` function
|
Welp. That took a lot of iterating. Apologies for doing this on the PR, I should've been more careful. |
|
Thank you for the contribution! Adding @jwiegley for the Nix stuff (I'm not familiar with all the ins and outs of the Nix config!) |
|
This is some of my first contributions to Nix in OSS, so pointers and feedback are definitely welcome. |
4 participants
Fixes #1346