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

ghc-8.10.1 #54005

Closed
wants to merge 20 commits into from
Closed

ghc-8.10.1 #54005

wants to merge 20 commits into from

Conversation

lukel97
Copy link
Contributor

@lukel97 lukel97 commented Apr 29, 2020

@fishtreesugar
Copy link
Contributor

Thanks!

@lukel97
Copy link
Contributor Author

lukel97 commented Apr 30, 2020

I think we'll need to make a whitelist exception for ghc@8.8 here https://github.com/Homebrew/brew/blob/72f7ef3d02dce3eb9056493044fcc530d7a83d3f/Library/Homebrew/rubocops/urls.rb#L24-L44

@lukel97
Copy link
Contributor Author

lukel97 commented Apr 30, 2020

Should the CI be taking this long?

@Bo98
Copy link
Member

Bo98 commented Apr 30, 2020

GHC pull requests usually take 16-19 hours to build the two GHC formulae and go through and test building every GHC dependent (49 formulae).

@lukel97
Copy link
Contributor Author

lukel97 commented May 1, 2020

Holy smokes. Is there a way to run it locally? I’m not able to see the logs from GitHub Actions and I’m sure there’s bound to be some failures

@Bo98
Copy link
Member

Bo98 commented May 1, 2020

Is there a way to run it locally?

If you have a lot of time to spare, I guess you can do brew install -s ghc ghc@8.8 $(brew uses --recursive --include-build ghc) but you might as well wait on the CI.

I’m not able to see the logs from GitHub Actions

Logs can be seen when the CI run is complete.

@lukel97 lukel97 force-pushed the ghc-8.10.1 branch 2 times, most recently from 56d23ec to 20a9b5e Compare May 2, 2020 16:29
@lukel97
Copy link
Contributor Author

lukel97 commented May 2, 2020

@Bo98 I've wrangled down most of the failures. Two of the failures are auditing failures, from cmt and mighttpd2.
Cmt's failure is that it's using a pre-release release on GitHub, but it's been using same version since the formula was initially added in January. mighttpd2 is failing because the website is unreachable (I've pinged the author).
I'm not sure if I'll be able to fix either of these within this PR, will it still be possible to merge this?

lukel97 added a commit to lukel97/komposition that referenced this pull request May 2, 2020
protolude-0.3 introduces some breaking changes
Part of an effort for Homebrew/homebrew-core#54005
@lukel97
Copy link
Contributor Author

lukel97 commented May 3, 2020

I think the cmt auditing failure should be the only one left now, ready for review

@bayandin
Copy link
Member

bayandin commented May 5, 2020

A patch for furmark doesn't apply:

==> Downloading https://github.com/diku-dk/futhark/commit/b862de37f1e2cd7e7b561b723d01da9c8d248881.diff?full_index=1
cp -p /Users/brew/Library/Caches/Homebrew/downloads/cee2bef6ac248d41bfa19eb8ef82dafb906efcd6a2bbc29d40425920013a89ae--b862de37f1e2cd7e7b561b723d01da9c8d248881.diff /private/tmp/futhark--patch-20200505-11593-1yz0nx5/b862de37f1e2cd7e7b561b723d01da9c8d248881.diff
Already downloaded: /Users/brew/Library/Caches/Homebrew/downloads/cee2bef6ac248d41bfa19eb8ef82dafb906efcd6a2bbc29d40425920013a89ae--b862de37f1e2cd7e7b561b723d01da9c8d248881.diff
==> Verifying cee2bef6ac248d41bfa19eb8ef82dafb906efcd6a2bbc29d40425920013a89ae--b862de37f1e2cd7e7b561b723d01da9c8d248881.diff checksum
==> Patching
==> Applying b862de37f1e2cd7e7b561b723d01da9c8d248881.diff
patch -g 0 -f -p1 -i /private/tmp/futhark--patch-20200505-11593-1yz0nx5/b862de37f1e2cd7e7b561b723d01da9c8d248881.diff
patching file src/Futhark/Pass/ExtractKernels/Interchange.hs
Hunk #1 FAILED at 102.
1 out of 1 hunk FAILED -- saving rejects to file src/Futhark/Pass/ExtractKernels/Interchange.hs.rej
Error: Failure while executing; `patch -g 0 -f -p1 -i /private/tmp/futhark--patch-20200505-11593-1yz0nx5/b862de37f1e2cd7e7b561b723d01da9c8d248881.diff` exited with 1.

@lukel97
Copy link
Contributor Author

lukel97 commented May 5, 2020

@bayandin looks like futhark was updated on master to a version that includes the patch. Hopefully this is it now

@bayandin
Copy link
Member

bayandin commented May 6, 2020

Looks good now.
There're only audit complains

brew audit --skip-style cmt --online
brew audit --skip-style mighttpd2 --online

@lukel97
Copy link
Contributor Author

lukel97 commented May 6, 2020

@bayandin Yeah, and I think these are out of our control unfortunately (or at least, out of scope for ghc)

@SMillerDev
Copy link
Member

Yeah, both are pre-existing issues. I say it's good to go!

@bayandin
Copy link
Member

bayandin commented May 7, 2020

I'm merging it manually since it required to rearrange commits a bit.

Well done @bubba! Thanks a lot for your contribution 🎉

@bayandin
Copy link
Member

bayandin commented May 7, 2020

It took about 5 hours just to upload all the bottles!
But everything is merged finally.

Thanks again @fishtreesugar, @bubba, and @Bo98!

@bayandin bayandin closed this May 7, 2020
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.

None yet

5 participants