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

Bump Tasty to released version (1.5) #2577

Merged
merged 7 commits into from
Sep 17, 2023

Conversation

martijnbastiaan
Copy link
Member

Still TODO:

  • Write a changelog entry (see changelog/README.md)
  • Check copyright notices are up to date in edited files

@martijnbastiaan
Copy link
Member Author

martijnbastiaan commented Sep 12, 2023

@kloonbot run_ci fd4c41a

The package index is already fetched in the "Setup CI" step. Restoring
the index from caches means overwriting the up-to-date package index
with a potentially older one.
martijnbastiaan and others added 2 commits September 12, 2023 10:10
Cabal's freeze file doesn't include version hashes, just version
numbers. This means that updates to dependencies specified in
`cabal.project`'s `source-repository-package` field don't get taken into
account properly.

This issue doesn't affect Stack, as it only relies on version hashes in
the first place.
@leonschoorl
Copy link
Member

@kloonbot run_ci 6abe185

Copy link
Member

@DigitalBrains1 DigitalBrains1 left a comment

Choose a reason for hiding this comment

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

So far the rest of it looks good to me, but perhaps CI will turn up some more packages that don't build.

@leonschoorl leonschoorl force-pushed the bump-tasty branch 5 times, most recently from 07c4426 to c7c106b Compare September 12, 2023 15:54
@leonschoorl
Copy link
Member

It was good you submitted this as an external PR, there was a problem in the tests we run on external PRs only

@martijnbastiaan
Copy link
Member Author

It was good you submitted this as an external PR, there was a problem in the tests we run on external PRs only

I mostly wanted to test my Stack changes, pity the Cabal jobs broke..

@leonschoorl leonschoorl force-pushed the bump-tasty branch 2 times, most recently from 2e92600 to 7865ebd Compare September 14, 2023 13:03
@leonschoorl
Copy link
Member

leonschoorl commented Sep 14, 2023

@kloonbot run_ci 7865ebd

@leonschoorl
Copy link
Member

@DigitalBrains1 I've re-requested a review from you because I've added my latest attempt at the mmap bug workaround to this PR
🤞 it works correctly this time

@martijnbastiaan
Copy link
Member Author

@leonschoorl GiLab CI: Up the manual cache bust token => GitLab :-)

@leonschoorl
Copy link
Member

leonschoorl commented Sep 15, 2023

I mostly wanted to test my Stack changes, pity the Cabal jobs broke..

https://gitlab.com/clash-lang/clash-compiler/-/jobs/5098978057
It failed here due to network glitchiness

Copy link
Member

@DigitalBrains1 DigitalBrains1 left a comment

Choose a reason for hiding this comment

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

This LGTM with one exception we discussed in the office: clash-lib:unittests overwrites -with-rtsopts so clash-lib will also need the workaround-ghc-mmap-crash flag and dance.

…nfig

This avoids the issue having to selectively pass around GHCRTS,
because it broke the building of certain packages.

But a downside is this doesn't work for binaries that have
 -with-rtsopts set locally, because that option isn't cumulative.
The GHC 8.6.5 job was doing the
 error while loading shared libraries: libHS...so: cannot open shared object file: No such file or directory
@martijnbastiaan martijnbastiaan merged commit 540cf0e into clash-lang:master Sep 17, 2023
@martijnbastiaan martijnbastiaan deleted the bump-tasty branch September 17, 2023 16:49
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.

4 participants