Skip to content

Include ignore_root_user_error in repository_rule keys - #835

Merged
f0rmiga merged 2 commits into
bazel-contrib:mainfrom
keith:ks/include-ignore_root_user_error-in-repository_rule-keys
Oct 5, 2022
Merged

Include ignore_root_user_error in repository_rule keys#835
f0rmiga merged 2 commits into
bazel-contrib:mainfrom
keith:ks/include-ignore_root_user_error-in-repository_rule-keys

Conversation

@keith

@keith keith commented Sep 23, 2022

Copy link
Copy Markdown
Member

Otherwise you can see this warning:

DEBUG: Rule 'python3_9_aarch64-apple-darwin' indicated that a canonical reproducible form can be obtained by dropping arguments ["ignore_root_user_error"]
DEBUG: Repository python3_9_aarch64-apple-darwin instantiated at:
  /Users/ksmiley/dev/lyft/ios4/WORKSPACE:113:27: in <toplevel>
  /private/var/tmp/_bazel_ksmiley/c921e5b09fe02de914188528345349ce/external/rules_python/python/repositories.bzl:366:26: in python_register_toolchains
Repository rule python_repository defined at:
  /private/var/tmp/_bazel_ksmiley/c921e5b09fe02de914188528345349ce/external/rules_python/python/repositories.bzl:269:36: in <toplevel>

Otherwise you can see this warning:

```
DEBUG: Rule 'python3_9_aarch64-apple-darwin' indicated that a canonical reproducible form can be obtained by dropping arguments ["ignore_root_user_error"]
DEBUG: Repository python3_9_aarch64-apple-darwin instantiated at:
  /Users/ksmiley/dev/lyft/ios4/WORKSPACE:113:27: in <toplevel>
  /private/var/tmp/_bazel_ksmiley/c921e5b09fe02de914188528345349ce/external/rules_python/python/repositories.bzl:366:26: in python_register_toolchains
Repository rule python_repository defined at:
  /private/var/tmp/_bazel_ksmiley/c921e5b09fe02de914188528345349ce/external/rules_python/python/repositories.bzl:269:36: in <toplevel>
```
@f0rmiga
f0rmiga merged commit 44d41ee into bazel-contrib:main Oct 5, 2022
@keith
keith deleted the ks/include-ignore_root_user_error-in-repository_rule-keys branch October 5, 2022 23:41
@keith

keith commented Oct 5, 2022

Copy link
Copy Markdown
Member Author

thanks!

@f0rmiga

f0rmiga commented Oct 5, 2022

Copy link
Copy Markdown
Member

@keith you are welcome!

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