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

Implement a haskell-suite compiler support #1530

Merged
merged 1 commit into from
Oct 14, 2013
Merged

Implement a haskell-suite compiler support #1530

merged 1 commit into from
Oct 14, 2013

Conversation

UnkindPartition
Copy link
Contributor

Haskell suite adds a new compiler flavor to Cabal, except it is not tied to any particular compiler. Hopefully, this may be one of the last compiler flavors added to Cabal. (related: #57)

It is already used by the haskell suite interface generator.
(See http://documentup.com/haskell-suite/haskell-names#module-interfaces/generating-interfaces)

As far as I know, Fay is also interested in using haskell-packages, but the necessity to build a patched version of Cabal is a major hindrance for them.

@bergmark
Copy link
Contributor

bergmark commented Oct 3, 2013

Yes we'd much rather use haskell-packages than add more stuff to Cabal :)

@23Skidoo
Copy link
Member

23Skidoo commented Oct 5, 2013

Looks nicely self-contained, so merging shouldn't be much of a risk.

@dcoutts, can you take a look?

@dcoutts
Copy link
Contributor

dcoutts commented Oct 14, 2013

Yes, looks OK.

The bit about getting registered packages, and registering packages could probably share code with the Distribution.Simple.Program.HcPkg code, since it seems to share the same command line syntax. That HcPkg module does allow the program to be specified so it's fine for it to be the "${haskell-suite-compiler} pkg" syntax.

@23Skidoo
Copy link
Member

OK, merging.

@feuerbach If you want to refactor the code to use Distribution.Simple.Program.HcPkg, please do a separate pull request.

23Skidoo added a commit that referenced this pull request Oct 14, 2013
Implement a haskell-suite compiler support
@23Skidoo 23Skidoo merged commit 47a3957 into haskell:master Oct 14, 2013
@UnkindPartition
Copy link
Contributor Author

Thanks! I'll look into HcPkg.

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