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

Empty other-modules leads to wrong output #30

Closed
michaelpj opened this issue Nov 25, 2020 · 3 comments
Closed

Empty other-modules leads to wrong output #30

michaelpj opened this issue Nov 25, 2020 · 3 comments
Assignees
Labels
bug Something isn't working

Comments

@michaelpj
Copy link

We have a stanza like this:

executable prism-unlock-exchange
    hs-source-dirs: prism/unlock-exchange
    other-modules:
    build-depends:
        base >=4.9 && <5

which gives output like this:

    - path: "plutus-scb/prism/unlock-exchange"
      component: "plutus-scb:exe:prism-unlock-exchange"
    - path: "plutus-scb/prism/unlock-exchange/build-depends:.hs"
      component: "plutus-scb:exe:prism-unlock-exchange"
    - path: "plutus-scb/prism/unlock-exchange/base.hs"
      component: "plutus-scb:exe:prism-unlock-exchange"

i.e. it's taking the cabal syntax after other-modules as, well, modules.

Not sure if this is related to #17.

@michaelpj
Copy link
Author

Maybe also: #29 (comment)

@jneira jneira added the bug Something isn't working label Nov 25, 2020
@jneira
Copy link
Collaborator

jneira commented Nov 25, 2020

I wonder if it is time to revive #13 and use directly cabal parsers.

@Avi-D-coder
Copy link
Owner

@michaelpj When you get a chance could you make sure your issue is fixed, before I bump ghcide and hls dependence constraint.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants