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

swagger2 test failure (Data.HashSet is ambiguous) #1010

Closed
DanBurton opened this issue Nov 29, 2015 · 6 comments
Closed

swagger2 test failure (Data.HashSet is ambiguous) #1010

DanBurton opened this issue Nov 29, 2015 · 6 comments

Comments

@DanBurton
Copy link
Contributor

swagger2-0.4/src/Data/Swagger/Schema/Internal.hs:18:8:
    Ambiguous module name ‘Data.HashSet’:
      it was found in multiple packages:
      unordered-containers-0.2.5.1@unord_7E41cj9RvqSJeZ4jSnbs1a hashmap-1.3.0.1@hashm_622M1jMBjyMIwDJT1SEKMF
Test suite doctest: FAIL
@juhp
Copy link
Contributor

juhp commented Nov 30, 2015

Ok, I opened an upstream issue for this too: GetShopTV/swagger2#15

@DanBurton
Copy link
Contributor Author

We should probably ban hashmap from stackage. It looks like it is being pulled in from the unordered-containers test suite.

See: https://raw.githubusercontent.com/fpco/stackage-nightly/master/nightly-2015-11-30.yaml

  hashmap:
    users:
    - unordered-containers
    constraints:
      library-profiling: true
      flags: {}
      build-benchmarks: true
      skip-build: false
      tests: expect-success
      haddocks: expect-success
      version-range: -any

@juhp
Copy link
Contributor

juhp commented Dec 1, 2015

So maybe we should open an issue against unordered-containers?

@DanBurton
Copy link
Contributor Author

And/or maybe we should add unordered-containers to skipped tests.

@juhp
Copy link
Contributor

juhp commented Dec 1, 2015

cc @tibbe

juhp added a commit that referenced this issue Dec 2, 2015
should be fixed now in 0.4.1

This reverts commit df55bcb.
@juhp juhp closed this as completed Dec 2, 2015
@juhp
Copy link
Contributor

juhp commented Dec 2, 2015

Dan if you think something more (general) should be done here maybe you can open another issue for that.

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

No branches or pull requests

2 participants