Skip to content

Add scripts for releases and final 1.7 tweaks #2850

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

Merged
merged 7 commits into from
Apr 26, 2022
Merged

Conversation

wz1000
Copy link
Collaborator

@wz1000 wz1000 commented Apr 23, 2022

  1. Disable 8.8.4 build in windows gitlab CI
    The job fails for inscrutable reason, binaries will have to be manually compiled in a VM.
  2. Bump hls-test-utils to 1.3
    Also need to bump the minor versions of all packages which were not bumped since 1.6.1.1
    and had changes to their testsuite.
  3. Add release scripts to download, hash, sign and upload releases to downloads.haskell.org

wz1000 added 2 commits April 23, 2022 16:38
The job fails for inscrutable reason, binaries will have to be manually compiled in a VM.
Also need to bump the minor versions of all packages which were not bumped since 1.6.1.1
and had changes to their testsuite.
Copy link
Collaborator

@pepeiborra pepeiborra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

Copy link
Collaborator

@fendor fendor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you for the insane amount of time you are putting in!

Copy link
Collaborator

@eddiemundo eddiemundo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, thank you!

@wz1000 wz1000 force-pushed the wip/1.7-release-scripts branch from 41bdac5 to 30435cb Compare April 25, 2022 07:39
@wz1000 wz1000 force-pushed the wip/1.7-release-scripts branch from 30435cb to b1e084e Compare April 26, 2022 05:56
@wz1000 wz1000 requested a review from hasufell as a code owner April 26, 2022 05:56
Copy link
Member

@hasufell hasufell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I only skimmed over the python/bash scripts. Since they're not mandatory, I guess it's fine.

@@ -10,7 +10,7 @@ ls -la out/
TARBALL_PREFIX="haskell-language-server"
case "${TARBALL_EXT}" in
zip)
HLS_VERSION="$("$CI_PROJECT_DIR/out/haskell-language-server-wrapper" --numeric-version)"
HLS_VERSION="$("$CI_PROJECT_DIR/out/haskell-language-server-8.10.7" --numeric-version)"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why this change? This will break CI once 8.10.7 is deprecated, no?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If hls-wrapper is compiled with 9.2.2 then the job fails because it can't find libgcc_s_seh-1.dll

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, this should have been fixed upstream weeks ago.

@@ -66,7 +66,7 @@ define set_rpath
endef

hls: bindist/ghcs
for ghc in $(shell [ -e "bindist/ghcs-`uname`" ] && cat "bindist/ghcs-`uname`" || cat "bindist/ghcs") ; do \
for ghc in $(shell [ -e "bindist/ghcs-`uname -o`" ] && cat "bindist/ghcs-`uname -o`" || cat "bindist/ghcs") ; do \
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried it on Windows and FreeBSD, and it seems like the only way to detect mingw without a version number so I think it is OK.

@@ -0,0 +1,154 @@
#!/usr/bin/env bash
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note that /usr/bin/env is not guaranteed by posix either, contrary to popular belief of nix users.

@wz1000 wz1000 enabled auto-merge April 26, 2022 12:18
@wz1000 wz1000 merged commit b5a37f7 into master Apr 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants