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
yi: fix build #44830
yi: fix build #44830
Conversation
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: yi Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: yi Partial log (click to expand)
|
sexpr = | ||
appendPatch (overrideCabal super.sexpr (drv: { | ||
isExecutable = false; | ||
libraryHaskellDepends = drv.libraryHaskellDepends ++ [self.QuickCheck]; | ||
})) ./patches/sexpr-0.2.1.patch; | ||
|
||
# | ||
yi-core = super.yi-core.override { yi-language = self.yi-language_0_18_0; }; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The reason yi-language
is at version 0.17.1
is because configuration-hackage2nix.yaml
specifies that. The reason it specifies that is because yi-core-0.17.1
is in the latest LTS. Since yi-core
is apparantely not in the LTS it defaults to the latest version which is 0.18.0. So your override is the right thing to do.
You could add a comment to instruct to remove the override once yi-language >= 0.18
is in the LTS.
Thanks @basvandijk, that clears things up, I added a comment for that |
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: yi Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: yi Partial log (click to expand)
|
Motivation for this change
I have no idea why this needs to be fixed like this, and I have left a blank comment because of that, please don't merge until that's cleared up. It's weird for only
yi-language
to be in the wrong version, because there are like 10yi-*
packages that each work with each other.Ping @ryantm @domenkozar
Things done
sandbox
innix.conf
on non-NixOS)./result/bin/
)