-
Notifications
You must be signed in to change notification settings - Fork 41
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
Could the Agda library be published to Nix Packages under agdaPackages
?
#387
Comments
I'm not a Nix user myself but I'd be happy with any PRs to improve Nix support for agda2hs. |
This would be great! There's a few things we can improve with the current nix setup, perhaps it would make sense to do a sprint and file within one PR? My wishlist would be:
The first one is least intrusive against nixpkgs, the others will probably require some patches, e.g. so that we can re-use upstream agda derivation. |
I added the lib in my fork, but haven't filed a PR against nixpkgs because I'm a bit confused about maintainers etiquette in nixpkgs. |
@liesnikov Quick note: before making a PR be sure to move the contents of |
@thelissimus I think it's a slightly different setup, what I do is a carbon copy of how other libraries are set up, which do use preBuild. That is, unless I misunderstood your link and the maintainers want to do that everywhere. |
@liesnikov To my understanding, Anyways, if you're uncertain you can just clarify things during the PR review. If they require changes you can just amend the commit and force push until the maintainers are satisfied, no big deal. |
(apologies for radio silence! 2025 started strong 😭) From @liesnikov's option, I am happy to have at least the first one. Personally, it's a terrible user experience to use |
I use Nix and I'm playing around with
agda2hs
, but it's been hard to do all the wiring necessary for this to work. I do not know how much work would that entail, but I am happy to help with that.The text was updated successfully, but these errors were encountered: