Skip to content

Bump for ghc-10 - #12374

Open
philderbeast wants to merge 7 commits into
haskell:masterfrom
cabalism:bump/ghc-10
Open

philderbeast wants to merge 7 commits into
haskell:masterfrom
cabalism:bump/ghc-10

Conversation

@philderbeast

Copy link
Copy Markdown
Collaborator

Allow building (and passing validate) with pre-releases of ghc-10. Adds allow-newer exceptions for ghc-10 and removes those that were added for ghc-9.14. Uses my forks of hashable and vector-th-unbox.

I tested this locally with the pre-release available through ghcup.

$ ghc --version
The Glorious Glasgow Haskell Compilation System, version 10.0.0.20260917

  • Patches conform to the coding conventions.
  • Is this a PR that fixes CI? If so, it will need to be backported to older cabal release branches (ask maintainers for directions).

}
}

-- | allow-newer: filepath:base only while we're using a pre-release of ghc-10

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why this one can't be done in a project file, like the rest

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That is better. I did this and left a conditional that will trigger a test failure with GHC >= 10.0.1.

guessPackageName =<< case packageDir flags of
Flag a -> return a
NoFlag -> last . splitDirectories <$> getCurrentDirectory
NoFlag -> fromMaybe "" . safeLast . splitDirectories <$> getCurrentDirectory

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this empty string principled and always correct, or should this be error ..., or rather the cabal equivalent.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was able to reduce this to just getCurrentDirectory.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants