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

Preliminary changes for bzlmod #1891

Merged
merged 8 commits into from
May 11, 2023
Merged

Preliminary changes for bzlmod #1891

merged 8 commits into from
May 11, 2023

Conversation

ylecornec
Copy link
Member

This PR contains some fixes and changes needed to support bzlmod.
Most notably:

  • It updates rules_python to use its runfiles library which now support bzlmod (instead of the deprecated one from bazel_tools).
  • It uses the new names of the rules_nixpkgs repositories inside the rules_haskell one. This is because repositories like rules_nixpkgs_core and rules_nixpkgs_python are modules which are meant to be used in bzlmod, but io_tweag_rules_nixpkgs is not.
  • Make use of the runfiles library in IntegrationTesting.hs, to not hardcode the names of external repository folders (which change with bzlmod).

This is necessary with bzlmod because `io_tweag_rules_nixpkgs` is not a module (as opposed to rules_nixpkgs_core`, `rules_nixpkgs_python`, etc).
In this version, the runfiles library supports bzlmod
The runfiles library from bazel_tools is deprecated in favor of rules_python
This handle the case where external repositories use the canonical name in bzlmod.
@ylecornec ylecornec requested a review from avdv as a code owner May 4, 2023 08:27
@ylecornec ylecornec mentioned this pull request May 4, 2023
14 tasks
Copy link
Member

@avdv avdv left a comment

Choose a reason for hiding this comment

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

Nice, thank you!

@ylecornec ylecornec added the merge-queue merge on green CI label May 10, 2023
@mergify mergify bot merged commit c91013e into master May 11, 2023
@mergify mergify bot deleted the ylecornec/prepare_bzlmod_2 branch May 11, 2023 07:54
@mergify mergify bot removed the merge-queue merge on green CI label May 11, 2023
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.

2 participants