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

Hardwired path in doctest's own testsuite breaks with v2-cabal #294

Open
andreasabel opened this issue Feb 27, 2021 · 1 comment
Open

Hardwired path in doctest's own testsuite breaks with v2-cabal #294

andreasabel opened this issue Feb 27, 2021 · 1 comment
Labels
bug test-suite Concerning doctest's own test-suite v2-cabal Concerning v2-style cabal commands

Comments

@andreasabel
Copy link
Collaborator

andreasabel commented Feb 27, 2021

This non-portable path makes cabal v2-test fail for doctest itself:

doctest "with-cbits" ["Bar.hs", "../../../dist/build/spec/spec-tmp/test/integration/with-cbits/foo.o"]

With v2-cabal and ghc-9.0.1 the path becomes ../../../dist-newstyle/build/x86_64-osx/ghc-9.0.1/doctest-0.19/t/spec/build/spec/spec-tmp/Test/integration/with-cbits/foo.o.

Maybe related: #245

@andreasabel andreasabel added bug v2-cabal Concerning v2-style cabal commands test-suite Concerning doctest's own test-suite labels Feb 27, 2021
andreasabel added a commit to andreasabel/doctest that referenced this issue Feb 27, 2021
This one doesn't fly under v2-cabal.
@andreasabel
Copy link
Collaborator Author

This is still deactivated:

doctest/test/MainSpec.hs

Lines 109 to 120 in 6bb14c4

-- Andreas, 2021-02-27, see issue #294.
-- This test case contains a hard-wired path that does not work
-- with v2-cabal.
-- I tested it under v2-cabal with a very non-portable path.
-- Deactivating the test case until a systematic solution is found...
--
-- it "works with additional object files" $ do
-- -- -- Path for v1-cabal:
-- -- doctest "with-cbits" ["Bar.hs", "../../../dist/build/spec/spec-tmp/test/integration/with-cbits/foo.o"]
-- -- -- Path for v2-cabal with ghc-9.0.1
-- -- doctest "with-cbits" ["Bar.hs", "../../../dist-newstyle/build/x86_64-osx/ghc-9.0.1/doctest-0.19/t/spec/build/spec/spec-tmp/Test/integration/with-cbits/foo.o"]
-- (cases 1)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug test-suite Concerning doctest's own test-suite v2-cabal Concerning v2-style cabal commands
Projects
None yet
Development

No branches or pull requests

1 participant