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

Fix test compatibility with tar 0.5.* #312

Merged
merged 1 commit into from
Apr 2, 2024

Conversation

felixonmars
Copy link
Contributor

The GenEntry method was added in tar-0.6.0.0 and thus building hackage-security with tar 0.5.* fails with the following errors:

[8 of 8] Compiling Main             ( tests/TestSuite.hs, dist/build/TestSuite/TestSuite-tmp/Main.dyn_o )

tests/TestSuite.hs:267:22: error:
    Not in scope: type constructor or class ‘Tar.GenEntry’
    Perhaps you meant ‘Tar.Entry’ (imported from Codec.Archive.Tar.Entry)
    Module ‘Codec.Archive.Tar.Entry’ does not export ‘GenEntry’.
    |
267 |     testEntries1 :: [Tar.GenEntry Tar.TarPath linkTarget]
    |                      ^^^^^^^^^^^^

The `GenEntry` method was added in tar-0.6.0.0 and thus building hackage-security with tar 0.5.* fails with the following errors:

```
[8 of 8] Compiling Main             ( tests/TestSuite.hs, dist/build/TestSuite/TestSuite-tmp/Main.dyn_o )

tests/TestSuite.hs:267:22: error:
    Not in scope: type constructor or class ‘Tar.GenEntry’
    Perhaps you meant ‘Tar.Entry’ (imported from Codec.Archive.Tar.Entry)
    Module ‘Codec.Archive.Tar.Entry’ does not export ‘GenEntry’.
    |
267 |     testEntries1 :: [Tar.GenEntry Tar.TarPath linkTarget]
    |                      ^^^^^^^^^^^^
```
@andreasabel
Copy link
Member

Mmh, maybe we need to diversify our CI to test with older package versions. E.g. use stack and some LTS snapshots.

@andreasabel andreasabel self-assigned this Apr 2, 2024
@andreasabel andreasabel added re: testsuite Concerning the included testsuites build failure Something does not build labels Apr 2, 2024
@andreasabel andreasabel added this to the 0.6.2.6 milestone Apr 2, 2024
andreasabel added a commit that referenced this pull request Apr 2, 2024
@andreasabel andreasabel mentioned this pull request Apr 2, 2024
@andreasabel andreasabel merged commit b8322a8 into haskell:master Apr 2, 2024
18 checks passed
@andreasabel
Copy link
Member

Do you need a release @felixonmars ?

@felixonmars felixonmars deleted the patch-2 branch April 2, 2024 15:27
@felixonmars
Copy link
Contributor Author

I have patched the Arch build, but it would be helpful for others I think :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build failure Something does not build re: testsuite Concerning the included testsuites
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants