Skip to content

Conversation

augustss
Copy link

@augustss augustss commented Sep 5, 2025

MicroHs does not have associated type synonyms families nor generics.
So there's a lot of extra CPP to work around that.

@lehins
Copy link
Contributor

lehins commented Sep 5, 2025

@augustss You have a lot of respect from me and from the rest of the community for implementing MicroHS. But my question to you, is why should random package and its maintainers suffer though maintaining a whole lot of CPP that is not even tested in CI, only because MicroHS does not support some common features that random package uses?

In my opinion, if someone would like to use random with MicroHS either:

  • they can use a fork of random that has this PR applied to it
  • or implement the missing features in MicroHS.

With all due respect, but in my good conscience I cannot merge a PR like that into a core package like random.

That being said, I don't have a problem keeping this PR opened for visibility, so others who would like to use MicroHS would know which fork to use.

Copy link
Contributor

@lehins lehins left a comment

Choose a reason for hiding this comment

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

Marking this PR ad DONOT merge due to the reason in my previous message, but keeping it open as a draft for visibility.

@lehins lehins marked this pull request as draft September 5, 2025 18:28
@augustss
Copy link
Author

augustss commented Sep 5, 2025

You can close the PR. You have been more polite about not wanting your package to be portable than some other package maintainers.
I have a fork that works. I'll just start uploading my forked packages to hackage and add a mechanism to have mcabal redirect non-portable packages.
Thanks for the quick response.

@augustss augustss closed this Sep 5, 2025
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.

2 participants