Skip to content

Attach windows binaries for ghc 8.8.4 #276

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 3, 2020 · 38 comments
Closed

Attach windows binaries for ghc 8.8.4 #276

jneira opened this issue Aug 3, 2020 · 38 comments
Labels
CI Continuous integration os: windows

Comments

@jneira
Copy link
Member

jneira commented Aug 3, 2020

As it is difficult to make ci generate those windows binaries but it is doable locally, i think we could try to attach my local windows versions.
I am gonna follow the naming conventions of the existing ones and attach them manually asap, if no one disagree.

/cc @bubba

@jneira jneira added the CI Continuous integration label Aug 3, 2020
@lukel97
Copy link
Collaborator

lukel97 commented Aug 3, 2020

That would be perfect!

@Ailrun Ailrun added os: windows status: in discussion Not actionable, because discussion is still ongoing or there's no decision yet labels Aug 3, 2020
@jneira
Copy link
Member Author

jneira commented Aug 3, 2020

I've attached binaries in the 0.2.2 release:

I would keep this open to track the manual upload of those versions, while they are not available via ci

@jneira jneira removed the status: in discussion Not actionable, because discussion is still ongoing or there's no decision yet label Aug 3, 2020
@lukel97
Copy link
Collaborator

lukel97 commented Aug 4, 2020

Is 8.8.2 completely unbuildable on windows?

@jneira
Copy link
Member Author

jneira commented Aug 4, 2020

it was last time i tried it but not sure it it worths the time to try it again: two minor versions released and 5 months since the last 8.8.2 stackage lts

@lukel97
Copy link
Collaborator

lukel97 commented Aug 9, 2020

@jneira I’ve kicked off the 0.3.0 pre-release, whenever you're able can you upload some windows binaries for ghc-8.8.3 and 8.8.4 at some point? I'll wait for them before untagging it as pre-release

@jneira
Copy link
Member Author

jneira commented Aug 9, 2020

Sure, i'll upload them asap

@jneira
Copy link
Member Author

jneira commented Aug 9, 2020

@bubba done!

@lukel97
Copy link
Collaborator

lukel97 commented Sep 3, 2020

Hi @jneira , @alanz has made the 0.4.0 release could you upload some binaries for those on 8.8.4 and 8.8.3 as well?

@jneira
Copy link
Member Author

jneira commented Sep 4, 2020

Sure, i uploaded 8.8.3, 8.8.4 and 8.10.2 last night

@alanz
Copy link
Collaborator

alanz commented Sep 4, 2020

Can we close this?

@jneira
Copy link
Member Author

jneira commented Sep 5, 2020

Well, we will need to upload manually windows versions hard to install in a ci env.
Stackage is using 8.8.4 for lts and 8.10.2 for nightly (with a windows workaround) so there is chances that users will need them.
I would continue uploading until newer versions are released and stackage is updated to use them.

@alanz
Copy link
Collaborator

alanz commented Sep 5, 2020

Oops, sorry, I thought this related to the 0.4.0 tag specifically.

@lukel97
Copy link
Collaborator

lukel97 commented Oct 3, 2020

@jneira 0.5.0 has been tagged, can you upload some binaries again whenever you're free? https://github.com/haskell/haskell-language-server/releases/tag/0.5.0

Btw, should we be also bumping hls-plugin-api to 0.5.0?

@jneira
Copy link
Member Author

jneira commented Oct 3, 2020

I did a diff between master and 0.4.1.0 and there is no differences, I will double check it just in case.

EDIT: git diff 0.5.0 hls-plugin-api-0.4.1.0 -- .\hls-plugin-api\ does show nothing

@jneira jneira changed the title Attach windows binaries for ghc versions 8.8.3 and 8.8.4 Attach windows binaries for ghc versions 8.8.3, 8.8.4 and 8.10.2 Oct 3, 2020
@jneira
Copy link
Member Author

jneira commented Oct 5, 2020

  • binaries for hls-0.5.0 already uploaded

@lukel97
Copy link
Collaborator

lukel97 commented Oct 11, 2020

@jneira can you do binaries for 0.5.1 at some point?

@jneira
Copy link
Member Author

jneira commented Oct 11, 2020

yeah, but I will not able to do it until tomorrow at night (UTC)

@lukel97
Copy link
Collaborator

lukel97 commented Oct 11, 2020

@jneira No rush, once you upload them can you uncheck the pre-release button? Thanks again as always!

@jneira
Copy link
Member Author

jneira commented Oct 12, 2020

@bubba uploaded and pre button unchecked!

@jneira
Copy link
Member Author

jneira commented Nov 17, 2020

Added missing windows binaries for 0.6.0

@jneira
Copy link
Member Author

jneira commented Dec 2, 2020

As we added 8.10.2 to cu, that version not need to be generated locally

@jneira jneira changed the title Attach windows binaries for ghc versions 8.8.3, 8.8.4 and 8.10.2 Attach windows binaries for ghc 8.8.3 and 8.8.4 Dec 2, 2020
@jneira
Copy link
Member Author

jneira commented Dec 17, 2020

To alliviate the manual generation of binaries i popose drop support for the ghc-8.8.3 binary and only relase ghc-8.8.4.
If nobody disagree i will do it in the next release. If some user explicitly ask for it we can attach it.

@jneira jneira changed the title Attach windows binaries for ghc 8.8.3 and 8.8.4 Attach windows binaries for ghc 8.8.4 Dec 17, 2020
@alanz
Copy link
Collaborator

alanz commented Dec 17, 2020

Sounds good to me

@Ailrun
Copy link
Member

Ailrun commented Dec 17, 2020

Also agreed.

@Ailrun
Copy link
Member

Ailrun commented Feb 24, 2021

@jneira Could you upload HLS-8.8.4 for Windows?

@jneira
Copy link
Member Author

jneira commented Feb 24, 2021

my laptop is on heat, i ll try to upload it this evening gmt

@jneira
Copy link
Member Author

jneira commented Feb 25, 2021

This time i am not able to install hls for ghc-8.8.4, it fails continously with segfaults building floskell.
@Ailrun Maybe we should make definitive the release and hope not many windows users continue being using ghc-8.8.4
I'll continue trying

@Ailrun
Copy link
Member

Ailrun commented Feb 25, 2021

Hm... does it work without floskell?

@jneira
Copy link
Member Author

jneira commented Feb 25, 2021

Hm... does it work without floskell?

I want to try some tricks including it to get the full build but that is a good alternative if everything else fails.
However i did not want make rest of users continue waiting and ghc-8.10.4 is a good version for windows so we always can suggest upgrade it. I cant continue trying to build until this night gmt.
The release has been made public via ghcup and vscode users still cant upgrade so...

@Ailrun
Copy link
Member

Ailrun commented Feb 25, 2021

However i did not want make rest of users continue waiting and ghc-8.10.4 is a good version for windows so we always can suggest upgrade it.

Agreed. I will make the pre-release be a real release.

P.S. done.

@jneira
Copy link
Member Author

jneira commented Feb 25, 2021

@Ailrun many thanks, i know make the release can be a little bit stressing and you have done a perfect work 😄

@Ailrun
Copy link
Member

Ailrun commented Feb 25, 2021

@jneira it's all my words, thank you for all your help for the release :)

@jneira
Copy link
Member Author

jneira commented Feb 25, 2021

Finally i've got to build it and it is already included in the release 😄

Using -j1, cabal build instead cabal install and setting floskell as local package 😅

@Ailrun
Copy link
Member

Ailrun commented Feb 25, 2021

If we can create a binary reliably with that setting, how about put things back to Github Actions?

@jneira
Copy link
Member Author

jneira commented Feb 26, 2021

mmm iirc in the CI windows environment the bug is more persistent, but maybe could work, good idea (the script would be more complex though)

@jneira
Copy link
Member Author

jneira commented Jun 13, 2021

I am installing locally 1.2.0 for ghc-8.8.4

@jneira
Copy link
Member Author

jneira commented Jun 14, 2021

This time i am not able to avoid the segfault easily, i am trying with

package floskell
    ghc-options: -O0

😟

@jneira
Copy link
Member Author

jneira commented Sep 17, 2021

No need for more manual builds thanks to #2199

@jneira jneira closed this as completed Sep 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous integration os: windows
Projects
None yet
Development

No branches or pull requests

4 participants