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

vty's test suite doesn't build with Cabal 2 #4918

Closed
felixonmars opened this issue Nov 29, 2017 · 1 comment
Closed

vty's test suite doesn't build with Cabal 2 #4918

felixonmars opened this issue Nov 29, 2017 · 1 comment

Comments

@felixonmars
Copy link
Collaborator

ref: jtdaugherty/vty#132

I am getting the same errors as that report. The author of vty thinks it's a Cabal issue, but I didn't find it reported here.

Preprocessing test suite 'verify-parse-terminfo-caps' for vty-5.18.1..
Building test suite 'verify-parse-terminfo-caps' for vty-5.18.1..
[1 of 4] Compiling Verify           ( test/Verify.hs, dist/build/verify-parse-terminfo-caps/Verify.dyn_o )
[2 of 4] Compiling Verify.Data.Terminfo.Parse ( test/Verify/Data/Terminfo/Parse.hs, dist/build/verify-parse-terminfo-caps/Verify/Data/Terminfo/Parse.dyn_o )
[3 of 4] Compiling Verify.Graphics.Vty.Output ( test/Verify/Graphics/Vty/Output.hs, dist/build/verify-parse-terminfo-caps/Verify/Graphics/Vty/Output.dyn_o )
[4 of 4] Compiling VerifyParseTerminfoCaps ( test/VerifyParseTerminfoCaps.hs, dist/build/verify-parse-terminfo-caps/VerifyParseTerminfoCaps.dyn_o )

test/VerifyParseTerminfoCaps.hs:39:61: warning: [-Wdeprecations]
    In the use of ‘tiGetStr’
    (imported from System.Console.Terminfo, but defined in System.Console.Terminfo.Base):
    Deprecated: "use tiGetOutput instead."
   |
39 | fromCapname ti name = fromJust $ Terminfo.getCapability ti (Terminfo.tiGetStr name)
   |                                                             ^^^^^^^^^^^^^^^^^

test/VerifyParseTerminfoCaps.hs:49:66: warning: [-Wdeprecations]
    In the use of ‘tiGetStr’
    (imported from System.Console.Terminfo, but defined in System.Console.Terminfo.Base):
    Deprecated: "use tiGetOutput instead."
   |
49 |                         return $ case Terminfo.getCapability ti (Terminfo.tiGetStr capName) of
   |                                                                  ^^^^^^^^^^^^^^^^^

<no location info>: error: module ‘Verify’ cannot be found locally

<no location info>: error:
    module ‘Verify.Data.Terminfo.Parse’ cannot be found locally

<no location info>: error:
    module ‘Verify.Graphics.Vty.Output’ cannot be found locally
hvr added a commit to hvr/cabal that referenced this issue Dec 1, 2017
The stub executable is linked against the test-library
which already contains all `other-modules`, so we need
to remove those from the stub-exe's build-info to avoid
build errors due to duplication.

This addresses haskell#4918
hvr added a commit to hvr/cabal that referenced this issue Dec 2, 2017
The stub executable is linked against the test-library
which already contains all `other-modules`, so we need
to remove those from the stub-exe's build-info to avoid
build errors due to duplication.

This addresses haskell#4918
23Skidoo pushed a commit that referenced this issue Dec 4, 2017
The stub executable is linked against the test-library
which already contains all `other-modules`, so we need
to remove those from the stub-exe's build-info to avoid
build errors due to duplication.

This addresses #4918

(cherry picked from commit 0571c99)
@hvr
Copy link
Member

hvr commented Dec 4, 2017

@felixonmars this should be fixed now with the recently released http://hackage.haskell.org/package/Cabal-2.0.1.1 ; i.e. ideally you need http://hackage.haskell.org/package/cabal-install-2.0.0.1 compiled with that Cabal version; you should be able to do that via

cabal install cabal-install-2.0.0.1 --constraint='Cabal >= 2.0.1.1'

@ezyang ezyang closed this as completed Dec 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants