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

Add rules_haskell module version 0.17 #1029

Merged
merged 13 commits into from
Nov 6, 2023
Merged

Conversation

avdv
Copy link
Contributor

@avdv avdv commented Oct 19, 2023

This PR adds the newly released rules_haskell version 0.17 to the BCR. See https://github.com/tweag/rules_haskell/releases/tag/v0.17

@avdv avdv force-pushed the rules_haskell-0.17 branch 2 times, most recently from 8538827 to cb15708 Compare October 19, 2023 14:11
@avdv avdv changed the title Mark tools/add_module.py executable Add rules_haskell module version 0.17 Oct 20, 2023
@avdv avdv marked this pull request as ready for review October 20, 2023 07:03
@alexeagle alexeagle added the presubmit-auto-run Presubmit jobs will be triggered for new changes automatically without reviewer's approval label Oct 27, 2023
@alexeagle
Copy link
Contributor

CI says

You should use a release archive URL in the format of https://github.com/<ORGANIZATION>/<REPO>/releases/download/<version>/<name>.tar.gz to ensure the archive checksum stability.

I'll add a tag here to bypass that check, but it's worth asking if you could easily include a release artifact in GH release.

@alexeagle alexeagle added the skip-url-stability-check Skip the URL stability check for the PR label Oct 27, 2023
@avdv
Copy link
Contributor Author

avdv commented Oct 30, 2023

CI says

You should use a release archive URL in the format of https://github.com/<ORGANIZATION>/<REPO>/releases/download/<version>/<name>.tar.gz to ensure the archive checksum stability.

I'll add a tag here to bypass that check, but it's worth asking if you could easily include a release artifact in GH release.

Thank you, I'll see that we change our release pipeline to create an artifact for releases.

@avdv avdv force-pushed the rules_haskell-0.17 branch 7 times, most recently from 0e274d1 to 0f73014 Compare November 2, 2023 07:34
@avdv
Copy link
Contributor Author

avdv commented Nov 2, 2023

Currently, this fails on Windows (https://buildkite.com/bazel/bcr-presubmit/builds/2394#018b8ef4-da85-4da0-ba4b-aa4dcd90c618):

ERROR: �[0mC:/b/n6445vdd/external/zlib~1.2.13/BUILD.bazel:5:11: Compiling gzwrite.c [for tool] failed: undeclared inclusion(s) in rule '@zlib~1.2.13//:zlib':
�_bk;t=1698910631219�this rule is missing dependency declarations for the following files included by 'gzwrite.c':
�_bk;t=1698910631219�  'C:/b/f3c3gu33/execroot/_main/external/rules_haskell~0.17~haskell_toolchains~rules_haskell_ghc_windows_amd64/mingw/x86_64-w64-mingw32/include/stdio.h'
�_bk;t=1698910631219�  'C:/b/f3c3gu33/execroot/_main/external/rules_haskell~0.17~haskell_toolchains~rules_haskell_ghc_windows_amd64/mingw/x86_64-w64-mingw32/include/corecrt_stdio_config.h'
�_bk;t=1698910631219�  'C:/b/f3c3gu33/execroot/_main/external/rules_haskell~0.17~haskell_toolchains~rules_haskell_ghc_windows_amd64/mingw/x86_64-w64-mingw32/include/corecrt.h'
�_bk;t=1698910631219�  'C:/b/f3c3gu33/execroot/_main/external/rules_haskell~0.17~haskell_toolchains~rules_haskell_ghc_windows_amd64/mingw/x86_64-w64-mingw32/include/_mingw.h'
�_bk;t=1698910631219�  'C:/b/f3c3gu33/execroot/_main/external/rules_haskell~0.17~haskell_toolchains~rules_haskell_ghc_windows_amd64/mingw/x86_64-w64-mingw32/include/_mingw_mac.h'
�_bk;t=1698910631219�  'C:/b/f3c3gu33/execroot/_main/external/rules_haskell~0.17~haskell_toolchains~rules_haskell_ghc_windows_amd64/mingw/x86_64-w64-mingw32/include/_mingw_secapi.h'
�_bk;t=1698910631219�  'C:/b/f3c3gu33/execroot/_main/external/rules_haskell~0.17~haskell_toolchains~rules_haskell_ghc_windows_amd64/mingw/x86_64-w64-mingw32/include/vadefs.h'
�_bk;t=1698910631219�  'C:/b/f3c3gu33/execroot/_main/external/rules_haskell~0.17~haskell_toolchains~rules_haskell_ghc_windows_amd64/mingw/x86_64-w64-mingw32/include/sdks/_mingw_ddk.h'
�_bk;t=1698910631219�  'C:/b/f3c3gu33/execroot/_main/external/rules_haskell~0.17~haskell_toolchains~rules_haskell_ghc_windows_amd64/mingw/x86_64-w64-mingw32/include/_mingw_off_t.h'
�_bk;t=1698910631219�  'C:/b/f3c3gu33/execroot/_main/external/rules_haskell~0.17~haskell_toolchains~rules_haskell_ghc_windows_amd64/mingw/x86_64-w64-mingw32/include/swprintf.inl'
�_bk;t=1698910631219�  'C:/b/f3c3gu33/execroot/_main/external/rules_haskell~0.17~haskell_toolchains~rules_haskell_ghc_windows_amd64/mingw/x86_64-w64-mingw32/include/sec_api/stdio_s.h'
�_bk;t=1698910631219�  'C:/b/f3c3gu33/execroot/_main/external/rules_haskell~0.17~haskell_toolchains~rules_haskell_ghc_windows_amd64/mingw/lib/gcc/x86_64-w64-mingw32/10.2.0/include/stddef.h'
�_bk;t=1698910631219�  'C:/b/f3c3gu33/execroot/_main/external/rules_haskell~0.17~haskell_toolchains~rules_haskell_ghc_windows_amd64/mingw/x86_64-w64-mingw32/include/stddef.h'
�_bk;t=1698910631219�  'C:/b/f3c3gu33/execroot/_main/external/rules_haskell~0.17~haskell_toolchains~rules_haskell_ghc_windows_amd64/mingw/x86_64-w64-mingw32/include/crtdefs.h'
�_bk;t=1698910631219�  'C:/b/f3c3gu33/execroot/_main/external/rules_haskell~0.17~haskell_toolchains~rules_haskell_ghc_windows_amd64/mingw/lib/gcc/x86_64-w64-mingw32/10.2.0/include-fixed/limits.h'
�_bk;t=1698910631219�  'C:/b/f3c3gu33/execroot/_main/external/rules_haskell~0.17~haskell_toolchains~rules_haskell_ghc_windows_amd64/mingw/lib/gcc/x86_64-w64-mingw32/10.2.0/include-fixed/syslimits.h'
�_bk;t=1698910631219�  'C:/b/f3c3gu33/execroot/_main/external/rules_haskell~0.17~haskell_toolchains~rules_haskell_ghc_windows_amd64/mingw/x86_64-w64-mingw32/include/limits.h'
�_bk;t=1698910631219�  'C:/b/f3c3gu33/execroot/_main/external/rules_haskell~0.17~haskell_toolchains~rules_haskell_ghc_windows_amd64/mingw/x86_64-w64-mingw32/include/sys/types.h'
�_bk;t=1698910631219�  'C:/b/f3c3gu33/execroot/_main/external/rules_haskell~0.17~haskell_toolchains~rules_haskell_ghc_windows_amd64/mingw/lib/gcc/x86_64-w64-mingw32/10.2.0/include/stdarg.h'
�_bk;t=1698910631219�  'C:/b/f3c3gu33/execroot/_main/external/rules_haskell~0.17~haskell_toolchains~rules_haskell_ghc_windows_amd64/mingw/x86_64-w64-mingw32/include/stdarg.h'
�_bk;t=1698910631219�  'C:/b/f3c3gu33/execroot/_main/external/rules_haskell~0.17~haskell_toolchains~rules_haskell_ghc_windows_amd64/mingw/x86_64-w64-mingw32/include/_mingw_stdarg.h'
�_bk;t=1698910631219�  'C:/b/f3c3gu33/execroot/_main/external/rules_haskell~0.17~haskell_toolchains~rules_haskell_ghc_windows_amd64/mingw/x86_64-w64-mingw32/include/string.h'
�_bk;t=1698910631219�  'C:/b/f3c3gu33/execroot/_main/external/rules_haskell~0.17~haskell_toolchains~rules_haskell_ghc_windows_amd64/mingw/x86_64-w64-mingw32/include/sec_api/string_s.h'
�_bk;t=1698910631219�  'C:/b/f3c3gu33/execroot/_main/external/rules_haskell~0.17~haskell_toolchains~rules_haskell_ghc_windows_amd64/mingw/x86_64-w64-mingw32/include/stdlib.h'
�_bk;t=1698910631219�  'C:/b/f3c3gu33/execroot/_main/external/rules_haskell~0.17~haskell_toolchains~rules_haskell_ghc_windows_amd64/mingw/x86_64-w64-mingw32/include/corecrt_wstdlib.h'
�_bk;t=1698910631219�  'C:/b/f3c3gu33/execroot/_main/external/rules_haskell~0.17~haskell_toolchains~rules_haskell_ghc_windows_amd64/mingw/x86_64-w64-mingw32/include/sec_api/stdlib_s.h'
�_bk;t=1698910631219�  'C:/b/f3c3gu33/execroot/_main/external/rules_haskell~0.17~haskell_toolchains~rules_haskell_ghc_windows_amd64/mingw/x86_64-w64-mingw32/include/malloc.h'
�_bk;t=1698910631219�  'C:/b/f3c3gu33/execroot/_main/external/rules_haskell~0.17~haskell_toolchains~rules_haskell_ghc_windows_amd64/mingw/x86_64-w64-mingw32/include/fcntl.h'
�_bk;t=1698910631219�  'C:/b/f3c3gu33/execroot/_main/external/rules_haskell~0.17~haskell_toolchains~rules_haskell_ghc_windows_amd64/mingw/x86_64-w64-mingw32/include/io.h'
�_bk;t=1698910631219�  'C:/b/f3c3gu33/execroot/_main/external/rules_haskell~0.17~haskell_toolchains~rules_haskell_ghc_windows_amd64/mingw/x86_64-w64-mingw32/include/errno.h'

It already worked once before (https://buildkite.com/bazel/bcr-presubmit/builds/2382#018b8a78-c378-4a67-8fb4-9c3d69d998de) -- for the same commit 0f73014 😕

What gives? Why do I get different results for the same commit?

@avdv avdv force-pushed the rules_haskell-0.17 branch 2 times, most recently from f6cfce0 to 1ca24a1 Compare November 6, 2023 13:19
Register the cc toolchain that comes with the GHC bindist.
Our test module depends on rules_nixpkgs' modules which are not available yet.
It is currently failing with:
```
this rule is missing dependency declarations for the following files included by 'gzwrite.c':
  'C:/b/f3c3gu33/execroot/_main/external/rules_haskell~0.17~haskell_toolchains~rules_haskell_ghc_windows_amd64/mingw/x86_64-w64-mingw32/include/stdio.h'
  'C:/b/f3c3gu33/execroot/_main/external/rules_haskell~0.17~haskell_toolchains~rules_haskell_ghc_windows_amd64/mingw/x86_64-w64-mingw32/include/corecrt_stdio_config.h'
  'C:/b/f3c3gu33/execroot/_main/external/rules_haskell~0.17~haskell_toolchains~rules_haskell_ghc_windows_amd64/mingw/x86_64-w64-mingw32/include/corecrt.h'
...
```
@avdv
Copy link
Contributor Author

avdv commented Nov 6, 2023

I could not resolve the problem on Windows, so I have disabled that platform for now. CI is now 🟢

Could anyone have a look please?

@fmeum fmeum merged commit aa95069 into bazelbuild:main Nov 6, 2023
7 checks passed
@avdv avdv deleted the rules_haskell-0.17 branch November 6, 2023 15:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
presubmit-auto-run Presubmit jobs will be triggered for new changes automatically without reviewer's approval skip-url-stability-check Skip the URL stability check for the PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants