search

tweag / topiary

https://topiary.tweag.io/
MIT License
582 stars 29 forks source link

Cut down on flake input dependencies #613

Open toastal opened 1 year ago

toastal commented 1 year ago

Is your feature request related to a problem? Please describe. When I include this flake as an input, I get a number of dependencies that I now have to manage. I don’t think these are necessary: numtide:flake-utils, numtide:nix-filter and can be inlined for less code than the import+usage+flake.lock lines. The default systems should be nixpkgs.lib.systems.flakeExposed & flake-utils doesn’t include some of these systems.

Describe the solution you'd like Inline the for loop from flake-utils. Use builtins.path { path = …; filter = path: type: …; } to replace nix-filter or use lib.fileset.

Describe alternatives you've considered tolerating the dependency tree exploding as each input necessitates different versions with flakes

Additional context

[^1]

[^1]: Please consider giving up MS GitHub or offering a non-proprietary, non-US-corporate-controlled mirror for this free software project. I wish to delete this Microsoft account in the future, but I need more projects like this to support alternative methods to send patches & contribute.

toastal commented 1 week ago

This is still affecting me. As Tweag isn’t maintaining upstream in Nixpkgs, the easiest way to get Topiary is to add it to an input, but there are a lot of unneeded dependencies which is not a good UX for using this.