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

Hackage hls-call-hierarchy-plugin-1.1.0.0 incompatible with hls-1.9 #3427

Closed
juhp opened this issue Dec 29, 2022 · 13 comments
Closed

Hackage hls-call-hierarchy-plugin-1.1.0.0 incompatible with hls-1.9 #3427

juhp opened this issue Dec 29, 2022 · 13 comments
Labels
status: needs triage type: bug Something isn't right: doesn't work as intended, documentation is missing/outdated, etc..

Comments

@juhp
Copy link
Contributor

juhp commented Dec 29, 2022

I know the Build from Source instructions say to use git, but just checking whether you plan to upload all the latest deps to Hackage?
Building from Hackage worked for 1.8 and earlier, but maybe you don't intend to support that way anymore?
(In which case I will switch my Fedora HLS build to use git instead, though using Hackage is more reproducible.
Personally, I feel if one day we can get HLS into Stackage that would be even better, but that is probably still far.)

@juhp juhp added status: needs triage type: bug Something isn't right: doesn't work as intended, documentation is missing/outdated, etc.. labels Dec 29, 2022
@juhp juhp changed the title upload latest deps to Hackage? upload latest plugins to Hackage? Dec 29, 2022
@wz1000
Copy link
Collaborator

wz1000 commented Dec 29, 2022 via email

@maralorn
Copy link
Contributor

I think everything is fine besides a missing update for hls-call-hierarchy-plugin. The released hls-call-hierarchy-plugin is not compatible with hls 1.9.

@juhp
Copy link
Contributor Author

juhp commented Dec 30, 2022

Okay I think maralorn is right - I was also hitting some unrelated local packaging changes I had made for 1.8...

@juhp juhp changed the title upload latest plugins to Hackage? hls-call-hierarchy-plugin-1.1.0.0 on Hackage incompatible with hls-1.9 Dec 30, 2022
@juhp
Copy link
Contributor Author

juhp commented Dec 30, 2022

Specifically Hackage hls-call-hierarchy-plugin-1.1.0.0 requires hls-plugin-api < 1.6

Error: cabal: Could not resolve dependencies:
[__0] trying: haskell-language-server-1.9.0.0 (user goal)
[__1] trying: haskell-language-server:+callhierarchy
[__2] trying: hls-call-hierarchy-plugin-1.1.0.0 (dependency of
haskell-language-server +callhierarchy)
[__3] next goal: hls-plugin-api (dependency of haskell-language-server)
[__3] rejecting: hls-plugin-api-1.6.0.0 (conflict: hls-call-hierarchy-plugin
=> hls-plugin-api^>=1.5)
[__3] rejecting: hls-plugin-api-1.5.0.0 (conflict: haskell-language-server =>
hls-plugin-api^>=1.6)
[__3] skipping: hls-plugin-api-1.4.0.0, hls-plugin-api-1.3.0.0,
hls-plugin-api-1.2.0.2, hls-plugin-api-1.2.0.1, hls-plugin-api-1.2.0.0,
hls-plugin-api-1.1.0.2, hls-plugin-api-1.1.0.1, hls-plugin-api-1.1.0.0,
hls-plugin-api-1.0.0.0, hls-plugin-api-0.7.1.0, hls-plugin-api-0.7.0.0,
hls-plugin-api-0.6.0.0, hls-plugin-api-0.5.0.1, hls-plugin-api-0.5.0.0,
hls-plugin-api-0.4.1.0, hls-plugin-api-0.4.0.0 (has the same characteristics
that caused the previous version to fail: excluded by constraint '^>=1.6' from
'haskell-language-server')
[__3] fail (backjumping, conflict set: haskell-language-server,
hls-call-hierarchy-plugin, hls-plugin-api)
After searching the rest of the dependency tree exhaustively, these were the
goals I've had most trouble fulfilling: haskell-language-server,
hls-plugin-api, hls-call-hierarchy-plugin,
haskell-language-server:callhierarchy

@juhp juhp changed the title hls-call-hierarchy-plugin-1.1.0.0 on Hackage incompatible with hls-1.9 Hackage hls-call-hierarchy-plugin-1.1.0.0 incompatible with hls-1.9 Dec 30, 2022
@spacekitteh
Copy link

Yes, please upload the latest version of the plugin to hackage.

@spacekitteh
Copy link

jbgi added a commit to IntersectMBO/cardano-node that referenced this issue Jan 4, 2023
jbgi added a commit to IntersectMBO/cardano-node that referenced this issue Jan 4, 2023
wz1000 added a commit that referenced this issue Jan 5, 2023
@wz1000
Copy link
Collaborator

wz1000 commented Jan 5, 2023

Published https://hackage.haskell.org/package/hls-call-hierarchy-plugin-1.2.0.0 and revised HLS (https://hackage.haskell.org/package/haskell-language-server-1.9.0.0/revisions/)

@cumber
Copy link

cumber commented Mar 2, 2023

@wz1000 Did that revision to depend on hls-call-hierarchy-plugin ^>= 1.2 ever make it into the actual source? It seems to have reverted to ^>= 1.1 in the hackage release of HLS 1.9.1

@tbidne
Copy link

tbidne commented Mar 7, 2023

@wz1000 Did that revision to depend on hls-call-hierarchy-plugin ^>= 1.2 ever make it into the actual source? It seems to have reverted to ^>= 1.1 in the hackage release of HLS 1.9.1

@cumber It is in HEAD but apparently did not make it into the hackage release of 1.9.1:

build-depends: hls-call-hierarchy-plugin ^>= 1.1

https://hackage.haskell.org/package/haskell-language-server-1.9.1.0/src/haskell-language-server.cabal

You can see the builds failed with the expected rejecting: hls-call-hierarchy-plugin-1.2.0.0 line.

Perhaps a hackage revision could fix this?

@cumber
Copy link

cumber commented Mar 11, 2023

@tbidne Yes, I'm aware of that because such a revision was made to HLS 1.9.0.0.

But then 1.9.1.0 was pushed to Hackage with the old version bounds, so it doesn't work again. We need another revision.

@juhp
Copy link
Contributor Author

juhp commented Mar 14, 2023

https://download.copr.fedorainfracloud.org/results/petersen/haskell-language-server/fedora-rawhide-x86_64/05634904-haskell-language-server/ ->
https://download.copr.fedorainfracloud.org/results/petersen/haskell-language-server/fedora-rawhide-x86_64/05634904-haskell-language-server/build.log.gz

[__2] next goal: hls-call-hierarchy-plugin (dependency of
haskell-language-server +callhierarchy)
[__2] rejecting: hls-call-hierarchy-plugin-1.2.0.0 (conflict:
haskell-language-server +callhierarchy => hls-call-hierarchy-plugin^>=1.1)
[__2] trying: hls-call-hierarchy-plugin-1.1.0.0
[__3] next goal: hls-plugin-api (dependency of haskell-language-server)
[__3] rejecting: hls-plugin-api-1.6.0.0 (conflict: hls-call-hierarchy-plugin
=> hls-plugin-api^>=1.5)
[__3] rejecting: hls-plugin-api-1.5.0.0 (conflict: haskell-language-server =>
hls-plugin-api^>=1.6)

It's not enabled in CI or should it be disabled by default?

@michaelpj
Copy link
Collaborator

This is still a problem.

@michaelpj michaelpj reopened this Mar 21, 2023
@michaelpj
Copy link
Collaborator

I published a revision to 1.9.1.0. Revisions are tricky!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: needs triage type: bug Something isn't right: doesn't work as intended, documentation is missing/outdated, etc..
Projects
None yet
Development

No branches or pull requests

7 participants