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

Use HLint 3.6.1 #666

Closed
wants to merge 1 commit into from
Closed

Use HLint 3.6.1 #666

wants to merge 1 commit into from

Conversation

phadej
Copy link
Collaborator

@phadej phadej commented Jul 7, 2023

No description provided.

@RyanGlScott
Copy link
Contributor

One thing that I discovered when trying out the hlint-3.6.1 bindist is that dynamically links against the glibc version provided by Ubuntu 22.04 (Jammy). This will not work if you run hlint-3.6.1 on Ubuntu 20.04 (Bionic), which is the default setting for haskell-ci. An example of this failing:

hlint: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.28' not found (required by hlint)
hlint: /lib/x86_64-linux-gnu/libm.so.6: version `GLIBC_2.29' not found (required by hlint)

I'm not sure if there is a good way to handle this on the haskell-ci side. Perhaps a disclaimer in the documentation somewhere?

@phadej
Copy link
Collaborator Author

phadej commented Jul 7, 2023

Thanks for bringing that up!

I'm not sure if there is a good way to handle this on the haskell-ci side.

There isn't.

Perhaps a disclaimer in the documentation somewhere?

Nobody reads docs, until there is problem, then they google and will find this issue.


I opened an upstream issue: ndmitchell/hlint#1533, and will withhold merging this PR until the upstream issue is resolved in a way or another.

@phadej phadej added the blocked label Aug 3, 2023
@marinelli
Copy link

marinelli commented Sep 18, 2023

It's possible to specify, for a particular GitHub Job, a different image [1]. So you could use the ubuntu:20.04 image by default, but for the linting jobs the ubuntu:22.04 one.
Anyway, I agree that a statically linked binary is a much more resilient solution.

[1] https://docs.github.com/en/actions/using-jobs/running-jobs-in-a-container

@RyanGlScott
Copy link
Contributor

This is likely obsolete after #717.

@phadej phadej closed this Apr 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants