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

sum-of-multiples: Explicitly pass [3, 5] instead of defaulting #101

Merged
merged 1 commit into from
Mar 26, 2016
Merged

sum-of-multiples: Explicitly pass [3, 5] instead of defaulting #101

merged 1 commit into from
Mar 26, 2016

Conversation

petertseng
Copy link
Member

Closes #100

@petertseng
Copy link
Member Author

This does not bode well. The build fails for cabal-related reasons, reasons completely unrelated to my change (which works fine when tested locally). As you can see, the build for the merge of #98 failed, despite the build for #98 passing, and I'm pretty sure that merge would have been a fast-forward. Will need a cabal fix, then I can rebase to get that fix.

Relevant parts of the failing build logs (in case they become unavailable from travis)

+cabal update
Downloading the latest package list from hackage.haskell.org
+cabal install primitive random tf-random HUnit QuickCheck split text bytestring attoparsec vector parallel stm old-locale lens
Resolving dependencies...
In order, the following would be installed:
text-1.2.2.1 (via: lens-4.13.2.1 semigroups-0.18.1 attoparsec-0.13.0.1 scientific-0.3.4.6 hashable-1.2.4.0) (new version)
hashable-1.2.4.0 (via: lens-4.13.2.1 void-0.7.1 semigroups-0.18.1 unordered-containers-0.2.7.0 scientific-0.3.4.6 nats-1.1) (reinstall) (changes: text-1.2.2.0 -> 1.2.2.1)
nats-1.1 (via: semigroups-0.18.1) (reinstall)
scientific-0.3.4.6 (via: attoparsec-0.13.0.1) (new version)
attoparsec-0.13.0.1 (reinstall) (changes: scientific-0.3.4.4 -> 0.3.4.6, text-1.2.2.0 -> 1.2.2.1)
unordered-containers-0.2.7.0 (via: lens-4.13.2.1 semigroups-0.18.1) (new version)
semigroups-0.18.1 (via: lens-4.13.2.1 adjunctions-4.3 free-4.12.4 semigroupoids-5.0.1 bifunctors-5.2.1 comonad-5 contravariant-1.4 void-0.7.1) (reinstall) (changes: text-1.2.2.0 -> 1.2.2.1, unordered-containers-0.2.6.0 -> 0.2.7.0)
void-0.7.1 (via: lens-4.13.2.1 adjunctions-4.3 contravariant-1.4) (reinstall)
contravariant-1.4 (via: lens-4.13.2.1 kan-extensions-5.0.1 adjunctions-4.3 semigroupoids-5.0.1 profunctors-5.2 comonad-5) (reinstall)
comonad-5 (via: lens-4.13.2.1 kan-extensions-5.0.1 adjunctions-4.3 free-4.12.4 semigroupoids-5.0.1 profunctors-5.2 bifunctors-5.2.1) (reinstall)
bifunctors-5.2.1 (via: lens-4.13.2.1 free-4.12.4 semigroupoids-5.0.1 profunctors-5.2) (reinstall)
profunctors-5.2 (via: lens-4.13.2.1 adjunctions-4.3 free-4.12.4) (reinstall)
semigroupoids-5.0.1 (via: lens-4.13.2.1 kan-extensions-5.0.1 adjunctions-4.3 free-4.12.4) (reinstall)
free-4.12.4 (via: lens-4.13.2.1 kan-extensions-5.0.1 adjunctions-4.3) (reinstall)
adjunctions-4.3 (via: kan-extensions-5.0.1) (reinstall)
kan-extensions-5.0.1 (via: lens-4.13.2.1) (reinstall)
lens-4.13.2.1 (reinstall) (changes: text-1.2.2.0 -> 1.2.2.1, unordered-containers-0.2.6.0 -> 0.2.7.0)
cabal: The following packages are likely to be broken by the reinstalls:
lens-4.13.2
unordered-containers-0.2.6.0
scientific-0.3.4.4
Use --force-reinstalls if you want to install anyway.

The command "./_test/bootstrap.sh" failed and exited with 1 during .

@petertseng
Copy link
Member Author

I think this is because cabal packages are cached and new versions have appeared since we last cached. This would break our cached packages. Clearing the cache will maybe solve this.

@petertseng
Copy link
Member Author

Definitely a cache-related, since https://travis-ci.org/exercism/xhaskell/builds/117969333 on 77dcf3c succeeded

Let's see if I can't do something to help here...

@petertseng
Copy link
Member Author

Good, I can cause the cache to get cleared on this branch, so that makes tests pass for this PR. I suspect master will remain broken until the cache gets cleared.

@kytrinyx
Copy link
Member

I've deleted the caches for master.

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.

None yet

2 participants