Skip to content

Change System.GPU to Control.Parallel and more...

Álvaro López García requested to merge github/fork/dagit/master into master

Created by: dagit

To be able to test this on osx I had to change some potentially unrelated things in OpenCL.cabal.

I left the test-suite section in, but that has to be modified to build on OSX due to a bug in Cabal where you can't put conditionals in a test-suite section:

I tried to wrap the whole test-suite section in a conditional but that failed to parse. I tried to use a flag and set buildable to the value of the flag but that also failed. I'm out of ideas for how to manage that. It really just comes down to us needing to specify extra-libraries on non-OSX platforms. Specifying frameworks should be fine on all platforms.

I found that the included headers did not match the OSX headers and so that was causing parse errors for c2hs. So I hardcoded the default install path for OpenCL headers on OSX. The one thing here is that I tested on 10.6 which ships OpenCL 1.0 instead of 1.1. It's possible my changes will not be compatible with OSX 10.7.

If we change the module names, I think we need to bump to 1.1.x due to the Package Version Policy: http://www.haskell.org/haskellwiki/Package_versioning_policy

Thanks! Jason

Merge request reports