Skip to content
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

Build failed with GHC 8.8 #4807

Closed
felixonmars opened this issue Jan 22, 2020 · 7 comments
Closed

Build failed with GHC 8.8 #4807

felixonmars opened this issue Jan 22, 2020 · 7 comments

Comments

@felixonmars
Copy link
Contributor

Steps to Reproduce

Build idris with GHC 8.8 (Cabal 3.0)

Expected Behavior

Build passes

Observed Behavior

Setup.hs:356:6: error: Not in scope: ‘preSDist’
    |
356 |    , preSDist = idrisPreSDist
    |      ^^^^^^^^

Setup.hs:357:6: error: Not in scope: ‘sDistHook’
    |
357 |    , sDistHook = idrisSDist (sDistHook simpleUserHooks)
    |      ^^^^^^^^^

Setup.hs:358:6: error: Not in scope: ‘postSDist’
    |
358 |    , postSDist = idrisPostSDist
    |      ^^^^^^^^^
@melted
Copy link
Contributor

melted commented Jan 22, 2020

The new style builds will not function, because hooks we use are removed and the replacement functionality doesn't work for us: #4757

So you will have to use cabal v1-install. Last I tried, and it was a while ago, the build fell down on dependencies that weren't 8.8 ready. But I guess it's time to sort that out.

@melted
Copy link
Contributor

melted commented Jan 22, 2020

So, I'm trying again with cabal v1-install --allow-newer

@melted
Copy link
Contributor

melted commented Jan 22, 2020

Ah, it fails at cheapskate again, not weird as that is unmaintained. I guess we could submit a PR anyway. Or fork it. Or just vendor it. The good news is that regex-base has been fixed.
Fixed it locally and started a build.

@melted
Copy link
Contributor

melted commented Jan 22, 2020

Ok, and now we get there, hmm, has the sdist hooks been removed for v1 builds too.

Well, that's terrible, that means putting all the files in the cabal files again. And we would have to rename all the test shellscripts because cabal has overly restrictive rules for wildcards: haskell/cabal#5883.

@melted
Copy link
Contributor

melted commented Jan 22, 2020

To build it's possible to remove the hooks. The dist generation breaks with that, but I guess I can at least fix the compilation errors.

@melted
Copy link
Contributor

melted commented Jan 22, 2020

#4808

@melted
Copy link
Contributor

melted commented Jan 25, 2020

Fixed

@melted melted closed this as completed Jan 25, 2020
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

No branches or pull requests

2 participants