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

check: allow "haiku" as a valid OS string. #9048

Merged
merged 1 commit into from
Jun 28, 2023

Conversation

jessicah
Copy link
Contributor

The OSHaiku identifier has been merged in #9006, for a future 3.12 release. However, tools that rely on earlier versions of cabal -- such as hackage -- don't recognise "haiku" as a valid OS identifier.

This allows packages with os(haiku) present to pass CI and be uploaded to hackage until hackage can eventually be upgraded to use cabal 3.12.


Please include the following checklist in your PR:

Bonus points for added automated tests!

The `OSHaiku` identifier has been merged in haskell#9006, for a future
3.13 release. However, tools that rely on earlier versions of
cabal -- such as hackage -- don't recognise "haiku" as a valid OS
identifier.

This allows packages with `os(haiku)` present to pass CI and be
uploaded to hackage in the interim.
@jessicah
Copy link
Contributor Author

Corresponding PR to hackage: haskell/hackage-server#1221

Copy link
Member

@Mikolaj Mikolaj left a comment

Choose a reason for hiding this comment

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

If only Hackage HQ approves of the change, IMHO we need this workaround in cabal ASAP to lessen the chance that, e.g., network contributors file cabal bug reports, because cabal complains about haiku in their local cabal check tests.

Copy link
Collaborator

@ulysses4ever ulysses4ever left a comment

Choose a reason for hiding this comment

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

It's not pretty but if it does the job...

@mergify mergify bot added the merge delay passed Applied (usually by Mergify) when PR approved and received no updates for 2 days label Jun 25, 2023
@Kleidukos Kleidukos removed merge delay passed Applied (usually by Mergify) when PR approved and received no updates for 2 days attention: needs-backport 3.12 merge me Tell Mergify Bot to merge labels Jun 28, 2023
@Kleidukos Kleidukos merged commit 9accc33 into haskell:3.10 Jun 28, 2023
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

Successfully merging this pull request may close these issues.

4 participants