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

Emit canonical labels if needed in pkg_name_from_label #447

Merged
merged 2 commits into from
Mar 12, 2025
Merged

Conversation

fmeum
Copy link
Member

@fmeum fmeum commented Jan 23, 2025

Previously resulted in errors such as:

   Traceback (most recent call last):
	File "/private/var/tmp/_bazel_fmeum/5d22c23119063f77cfcdafc5f5d52f01/external/toolchains_llvm+/toolchain/internal/configure.bzl", line 141, column 66, in llvm_config_impl
		sysroot_paths_dict, sysroot_labels_dict = _sysroot_paths_dict(
	File "/private/var/tmp/_bazel_fmeum/5d22c23119063f77cfcdafc5f5d52f01/external/toolchains_llvm+/toolchain/internal/sysroot.bzl", line 76, column 53, in sysroot_paths_dict
		path = _canonical_dir_path(str(rctx.path(label).dirname))
Error in path: Unable to load package for @@[unknown repo 'toy-toolchain++toy_sysroot+sysroot_amd64_ubuntu' requested from @@toolchains_llvm+]//:BUILD.bazel: The repository '@@[unknown repo 'toy-toolchain++toy_sysroot+sysroot_amd64_ubuntu' requested from @@toolchains_llvm+]' could not be resolved: No repository visible as '@toy-toolchain++toy_sysroot+sysroot_amd64_ubuntu' from repository '@@toolchains_llvm+

fmeum added 2 commits March 12, 2025 13:07
Previously resulted in errors such as:
```
   Traceback (most recent call last):
	File "/private/var/tmp/_bazel_fmeum/5d22c23119063f77cfcdafc5f5d52f01/external/toolchains_llvm+/toolchain/internal/configure.bzl", line 141, column 66, in llvm_config_impl
		sysroot_paths_dict, sysroot_labels_dict = _sysroot_paths_dict(
	File "/private/var/tmp/_bazel_fmeum/5d22c23119063f77cfcdafc5f5d52f01/external/toolchains_llvm+/toolchain/internal/sysroot.bzl", line 76, column 53, in sysroot_paths_dict
		path = _canonical_dir_path(str(rctx.path(label).dirname))
Error in path: Unable to load package for @@[unknown repo 'toy-toolchain++toy_sysroot+sysroot_amd64_ubuntu' requested from @@toolchains_llvm+]//:BUILD.bazel: The repository '@@[unknown repo 'toy-toolchain++toy_sysroot+sysroot_amd64_ubuntu' requested from @@toolchains_llvm+]' could not be resolved: No repository visible as '@toy-toolchain++toy_sysroot+sysroot_amd64_ubuntu' from repository '@@toolchains_llvm+
```
@fmeum
Copy link
Member Author

fmeum commented Mar 12, 2025

@jsharpe @rrbutani Friendly ping

@fmeum fmeum merged commit c0f336b into master Mar 12, 2025
36 checks passed
@fmeum fmeum deleted the fmeum-patch-1 branch March 12, 2025 13:47
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