Skip to content

Hackage ci fails to build lsp-types-1.3.0.0 #349

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

Closed
jneira opened this issue Aug 1, 2021 · 9 comments · Fixed by #350 or #351
Closed

Hackage ci fails to build lsp-types-1.3.0.0 #349

jneira opened this issue Aug 1, 2021 · 9 comments · Fixed by #350 or #351
Labels
type: bug type: build Continuous integration and building

Comments

@jneira
Copy link
Member

jneira commented Aug 1, 2021

From #348 (comment) by @bkomuves


Both the Hackage CI and the Matrix Builder (I'm not sure if these are separate entities or not?) fails to build lsp-types-1.3.0.0:

which in turn also means that there is no browsable documentation on Hackage.

Since it builds locally (with ghc 8.6.5 in my case), I would guess that the reason is maybe that some dependencies have a not-strict-enough version bound?

@jneira jneira added type: bug type: build Continuous integration and building labels Aug 1, 2021
@jneira
Copy link
Member Author

jneira commented Aug 1, 2021

Cant reproduce the build error locally in windows with ghc-8.10.5 neither

@bkomuves
Copy link

bkomuves commented Aug 2, 2021

#263 may be related (the build error messages look very similar, and the apparent reason behind it looks believable too)

@jneira
Copy link
Member Author

jneira commented Aug 2, 2021

weird it is not reproduced locally, the project does not set the hackage index in cabal.project 🤔
i am trying with ghc-8.10.2, the used one by hackage ci

@jneira
Copy link
Member Author

jneira commented Aug 2, 2021

It is picking dependent-sum-0.7.1.0 locally instead the used one in hackage ci dependent-sum-0.6.2.0 cause we are using a custom dependent-sum version in cabal.project

lsp/cabal.project

Lines 28 to 33 in dec07c6

source-repository-package
type: git
location: https://github.com/anka-213/dependent-sum
tag: 8cf4c7fbc3bfa2be475a17bb7c94a1e1e9a830b5
subdir: dependent-sum-template
-- https://github.com/obsidiansystems/dependent-sum/pull/57
🤦

@jneira
Copy link
Member Author

jneira commented Aug 2, 2021

working on fixing it

hamishmack added a commit to input-output-hk/haskell.nix that referenced this issue Aug 5, 2021
hamishmack added a commit to input-output-hk/haskell.nix that referenced this issue Aug 6, 2021
@jneira
Copy link
Member Author

jneira commented Aug 6, 2021

This is still not fixed and it is not possible to make a revision adding new libraries, so a new minor version is needed

@jneira jneira reopened this Aug 6, 2021
@jneira
Copy link
Member Author

jneira commented Aug 6, 2021

Opened issue upstream obsidiansystems/dependent-sum#61, to being able to remove the inclusion of dependent-sum in the .cabal file (it is a transitive dependency and we should not have it there)

@jneira
Copy link
Member Author

jneira commented Aug 6, 2021

Published https://hackage.haskell.org/package/lsp-types-1.3.0.1, i hope this one will work 🤞

@jneira
Copy link
Member Author

jneira commented Aug 7, 2021

@bkomuves the hackage build for lsp-types-1.3.0.1 has been succesful: https://hackage.haskell.org/package/lsp-types-1.3.0.1/reports/1

booniepepper pushed a commit to booniepepper/haskell.nix that referenced this issue Feb 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug type: build Continuous integration and building
Projects
None yet
2 participants