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

Build script bug and updating dependency constraints. #1

Closed
wants to merge 2 commits into from

Conversation

jship
Copy link
Contributor

@jship jship commented Dec 18, 2015

* Update project to be compatible with stack using LTS snapshot 3.18.
* Add gitignore file that is aware of both stack- and cabal-based binaries/intermediate files.

  • Fixed issue with non-existent Hierarchy.png image trying to be copied in the build script.
  • Relaxed doctest and filepath dependency constraints.
  • Bumped min version of numeric-qq to 0.1.3.
  • Set range of acceptable versions of base.

@jship
Copy link
Contributor Author

jship commented Dec 18, 2015

Travis check failed due to dependencies. Will update the Travis config to use stack later today.

Based on comments from a pull request I put in on numeric-qq, following the same requirements on this PR. Not using stack, but making sure stack will work out-of-the-box with LTS-3.18.

@jship
Copy link
Contributor Author

jship commented Dec 18, 2015

Thought Travis was failing due to dependencies but it was actually from the customized Setup.hs build script. This script was trying to copy a non-existent 'Hierarchy.png' file into the Haddock output directory. This custom setup code was based on a version of @ekmett's lens library.

Removed call to copyFiles from Setup.hs as a hierarchy image is not distributed in the repo.

Travis is passing now using stack's LTS-2, LTS-3, and the latest nightly snapshot. This covers GHC 7.8.4, 7.10.2, and 7.10.3.

@jship jship changed the title Stack support and .gitignore. Build script bug and updating dependency constraints. Dec 19, 2015
@jship
Copy link
Contributor Author

jship commented Dec 21, 2015

Closing this pull request as the changes are captured in my latest PR: #2

@jship jship closed this Dec 21, 2015
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.

1 participant