Skip to content

flakes: clarify flake-compat/flake-inputs#1209

Open
KiaraGrouwstra wants to merge 2 commits into
NixOS:masterfrom
KiaraGrouwstra:flake-compat
Open

flakes: clarify flake-compat/flake-inputs#1209
KiaraGrouwstra wants to merge 2 commits into
NixOS:masterfrom
KiaraGrouwstra:flake-compat

Conversation

@KiaraGrouwstra

@KiaraGrouwstra KiaraGrouwstra commented Feb 9, 2026

Copy link
Copy Markdown
Contributor

Update the /concepts/flakes page to note flake-compat and flake-inputs both work from the publisher's as well as the consumer's perspective.

Note that while flake-compat is documented better, flake-compat seems to still lack support for
overrides.

Update the /concepts/flakes page to note flake-compat and flake-inputs
both work from the publisher's as well as the consumer's perspective.

Note that while flake-compat is documented better, flake-compat seems to
still [lack support for
overrides](https://git.lix.systems/lix-project/flake-compat/issues/82).

Signed-off-by: cinereal <[email protected]>
@KiaraGrouwstra KiaraGrouwstra requested a review from a team as a code owner February 9, 2026 19:38
@KiaraGrouwstra

Copy link
Copy Markdown
Contributor Author

CI complains 429 (Too Many Requests)

Comment thread source/concepts/flakes.md
- Handle dependencies inline using functions such as the [fetchers] or [`builtins.fetchTree`].

[fetchers]: https://nixos.org/manual/nixpkgs/stable/#chap-pkgs-fetchers
[`flake-inputs`]: https://github.com/fricklerhandwerk/flake-inputs

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I don't think this should be advertised here. I'm not actively maintaining it, and IMO the Lix fork is better in multiple regards:

  • the diff to nixos/flake-compat is more obvious (this was not a goal for my hack, but is relevant when using forks)
  • it has tests
  • it can turn off local path copying (because my diff is arbitrary large, it's too much of a hassle for me right now to adopt the patch)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

i initially intended to supersede flake-inputs by flake-compat here, yeah - it's only when i realized flake-compat isn't fully superior yet (see above) that i considered to instead mention both still.
i mentioned them about your approach there, so hopefully as they address that we can just leave out mention of flake-inputs here.

@hsjobeki

hsjobeki commented Apr 8, 2026

Copy link
Copy Markdown
Collaborator

@KiaraGrouwstra What is the state of this PR? I think we should simply drop the reference to "flake-inputs" ?

@KiaraGrouwstra

KiaraGrouwstra commented Apr 8, 2026

Copy link
Copy Markdown
Contributor Author

@KiaraGrouwstra KiaraGrouwstra marked this pull request as draft April 8, 2026 10:57
@KiaraGrouwstra KiaraGrouwstra marked this pull request as ready for review May 2, 2026 06:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants