Skip to content

Conversation

@Wyverald
Copy link
Member

@Wyverald Wyverald commented Apr 5, 2024

This new flag defaults to false. When set to true, native repo rules cannot be used in WORKSPACE; their Starlark counterparts must be used. Native repo rules already can't be used in Bzlmod.

Work towards #18285.

@Wyverald Wyverald requested review from aranguyen and fmeum and removed request for a team, ahumesky, aranguyen, lberki and ted-xie April 5, 2024 23:16
@github-actions github-actions bot added team-Configurability platforms, toolchains, cquery, select(), config transitions team-ExternalDeps External dependency handling, remote repositiories, WORKSPACE file. awaiting-review PR is awaiting review from an assigned reviewer labels Apr 5, 2024
@Wyverald Wyverald force-pushed the wyv-nonative-flag branch from a949407 to 834ac67 Compare April 5, 2024 23:28
@Wyverald Wyverald marked this pull request as draft April 5, 2024 23:33
@Wyverald Wyverald force-pushed the wyv-nonative-flag branch from 834ac67 to 2faca5c Compare April 6, 2024 01:09
@Wyverald Wyverald marked this pull request as ready for review April 6, 2024 04:00
@Wyverald Wyverald force-pushed the wyv-nonative-flag branch 2 times, most recently from af1766a to ab7bad2 Compare April 9, 2024 09:10
@Wyverald Wyverald changed the title Add new flag --enable_native_repo_rules Add new flag --incompatible_disable_native_repo_rules Apr 9, 2024
@Wyverald Wyverald force-pushed the wyv-nonative-flag branch from ab7bad2 to e189c49 Compare April 16, 2024 13:52
@Wyverald Wyverald added awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally and removed awaiting-review PR is awaiting review from an assigned reviewer labels Apr 16, 2024
@Wyverald Wyverald force-pushed the wyv-nonative-flag branch 2 times, most recently from 46b984a to 7958e88 Compare April 19, 2024 11:25
This new flag defaults to `false`. When set to `true`, native repo rules cannot be used in WORKSPACE; their Starlark counterparts must be used. Native repo rules already can't be used in Bzlmod.

Work towards #18285.
@Wyverald Wyverald force-pushed the wyv-nonative-flag branch from 7958e88 to 000d1a0 Compare April 19, 2024 13:38
@Wyverald Wyverald deleted the wyv-nonative-flag branch April 21, 2024 00:50
Wyverald added a commit that referenced this pull request Apr 25, 2024
This new flag defaults to `false`. When set to `true`, native repo rules cannot be used in WORKSPACE; their Starlark counterparts must be used. Native repo rules already can't be used in Bzlmod.

Work towards #18285.

Closes #21913.

PiperOrigin-RevId: 626410163
Change-Id: I4981cd8d99fc87d0d0b66991e563941478b40f12
@sgowroji sgowroji removed the awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally label Apr 26, 2024
Wyverald added a commit that referenced this pull request Apr 29, 2024
This new flag defaults to `false`. When set to `true`, native repo rules cannot be used in WORKSPACE; their Starlark counterparts must be used. Native repo rules already can't be used in Bzlmod.

Work towards #18285.

Closes #21913.

PiperOrigin-RevId: 626410163
Change-Id: I4981cd8d99fc87d0d0b66991e563941478b40f12
Wyverald added a commit that referenced this pull request Apr 30, 2024
This new flag defaults to `false`. When set to `true`, native repo rules cannot be used in WORKSPACE; their Starlark counterparts must be used. Native repo rules already can't be used in Bzlmod.

Work towards #18285.

Closes #21913.

PiperOrigin-RevId: 626410163
Change-Id: I4981cd8d99fc87d0d0b66991e563941478b40f12
Kila2 pushed a commit to Kila2/bazel that referenced this pull request May 13, 2024
This new flag defaults to `false`. When set to `true`, native repo rules cannot be used in WORKSPACE; their Starlark counterparts must be used. Native repo rules already can't be used in Bzlmod.

Work towards bazelbuild#18285.

Closes bazelbuild#21913.

PiperOrigin-RevId: 626410163
Change-Id: I4981cd8d99fc87d0d0b66991e563941478b40f12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

team-Configurability platforms, toolchains, cquery, select(), config transitions team-ExternalDeps External dependency handling, remote repositiories, WORKSPACE file.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants