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

Don't use hpp as the preprocessor on Windows. #26

Merged
merged 1 commit into from
Oct 27, 2017

Conversation

TravisWhitaker
Copy link
Contributor

Building master on Windows with GHC 8.2.1 yields this on my Windows 10 machine:

C:\Users\Travis\sources\GLUtil\src\Graphics\GLUtil\ShaderProgram.hs:1:1: error:
    File name does not match module name:
    Saw: `Main'
    Expected: `Graphics.GLUtil.ShaderProgram'
  |
1 | {-# LANGUAGE CPP #-}
  | ^

--  While building package GLUtil-0.9.1.1 using:
      C:\sr\setup-exe-cache\x86_64-windows\Cabal-simple_Z6RU0evB_2.0.0.2_ghc-8.2.1.exe --builddir=.stack-work\dist\e53504d9 build lib:GLUtil --ghc-options " -ddump-hi -ddump-to-file"
    Process exited with code: ExitFailure 1

The issue seems to be due to hpp. I'd be happy to help figure out what's going on with hpp on Windows (I don't have this problem on my Linux machines), but in the meantime using cpphs on Windows makes the package buildable.

@TravisWhitaker
Copy link
Contributor Author

Should fix #25.

@acowley acowley merged commit 38fd971 into acowley:master Oct 27, 2017
@acowley
Copy link
Owner

acowley commented Oct 27, 2017

Thank you, I’ll push to hackage asap!

@TravisWhitaker
Copy link
Contributor Author

Thanks for the speedy response.

Any chance of getting this package back in the Stackage snapshots? It was last in nightly-2015-12-14.

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