-
-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
Nix CI is broken #10509
Comments
Hi John,
I don't think that error is related to Nix. The executable
".cabal-wrapped" is "cabal". It's just a renamed version of the
cabal-install executable. The executable name "cabal" is used by a
Nix-generated wrapper script that adds important tools to $PATH to make
sure cabal finds them, but that's all there is to it. Nix doesn't know
or care about happy.
My guess is that the build uses an old version of cabal-install that
can't deal with the relatively recent version of happy. So one could
either upgrade the build to a newer cabal-install version or downgrade
the version of happy to remedy this situation.
Cheers, Peter
|
Thank you Peter. Both the cabal version and the happy version are the versions from the nixpkgs repository -- so, if what you say is right, isn't a problem with nix if these don't work together? In any case, I'd be happy to be told how to change default.nix to specify either a newer cabal-install version or an older happy version. I'm sorry, I don't know nix well enough. |
PS. I'd been assuming that the |
On a general note, the CI workflow uses |
See https://github.com/jgm/pandoc/actions/runs/12617858361/job/35160603870#step:6:2569
The problem seems to be that nix's cabal wrapper can't determine the version of happy, which is a build tool for toml-parser. happy is installed.
happy --version
returnswhich I'm guessing is not in the format cabal-wrapped expects?
So I think this is a bug in Haskell nix tooling, but I'm completely ignorant about how this works, so I'm hoping someone more knowledgeable can point me in the right direction or create an upstream bug report.
@tarleb
@srid
@peti
The text was updated successfully, but these errors were encountered: