-
Notifications
You must be signed in to change notification settings - Fork 518
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
[FR]: @bazel/runfiles needs to handle repo mapping #3714
Comments
@fmeum I know you visited a lot of languages in the ecosystem for the other runfiles fixes. I don't recall, did you have a look here as well? |
The lookup procedure itself looks sound. It's really just the repo mapping logic that is missing. My JS is pretty rusty, so I would prefer someone else to implement this, but I can review the PR. The Python library can serve as a reference here. The challenging part would be determining the current repo. I don't know how one would get the path of the current source file in JS. Minification and bundling could be problematic in this context. Can source maps be consumed at runtime? |
@ewhauser is this something you're interested in pairing with Fabian on? |
This issue has been automatically marked as stale because it has had no recent activity. It will be closed if no further activity occurs in 30 days. Note as of rules_nodejs v6 the rules_nodejs repository contains only the core nodejs toolchain and |
Can someone please add a label to turn off the stale bot for this issue? This blocks use of bzlmod for anyone using |
This issue has been automatically marked as stale because it has had no recent activity. It will be closed if no further activity occurs in 30 days. Note as of rules_nodejs v6 the rules_nodejs repository contains only the core nodejs toolchain and |
sigh Not stale. |
I'm tempted to move the implementation of the runfiles library to rules_js at this point in the process of fixing it for bzlmod. rules_nodejs now only supplies the Node.js toolchain and no longer deals with the runtime js_binary / js_test implementation. I'm not sure we even have the right permissions to publish to the |
What is the current behavior?
The @bazel/runfiles library fails to find runfiles specified with apparent repository names when Bzlmod is enabled.
Describe the feature
Equivalent to bazelbuild/rules_go#3347 / bazelbuild/rules_python#998.
The text was updated successfully, but these errors were encountered: