-
Notifications
You must be signed in to change notification settings - Fork 433
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
Pubish runfiles package to crates.io #1592
Comments
I'm a bit doubtful about successfully using this from a cargo context: from the sources it looks like this is very specifically designed for using in bazel environments: https://github.com/bazelbuild/rules_rust/blob/main/tools/runfiles/runfiles.rs#L1. As a risk: if we publish it to crates.io, this may create unreasonable expectations that it works in cargo context. Could you say more about how this unnecessarily forces to choose between bazel and cargo? Naively, if I had a project and wanted it to support both cargo and bazel and needed the runfiles functionality for the bazel bits, I'd try and guard the bazel-specific bits that require the runfiles under a source config / conditional compilation or something like that. |
While it's true that the runfiles library should change only very rarely, it actually will have to change to accommodate Bzlmod (see bazelbuild/bazel#16124 and the linked proposal). Whatever the outcome of this discussion is, I would urge you to wait until Bazel 6 has been released and the required changes have been implemented. |
Seems I still think this should be a published crate. |
I think another benefit is to have it accessible on docs.rs, although pkg.go.dev works differently. |
Would it be possible to publish the [runfiles|https://github.com/bazelbuild/rules_rust/blob/main/tools/runfiles/runfiles.rs] package to https://crates.io/ so it doesn't become an unnecessary forcing forcing function between the use of Bazel and Cargo? The library very rarely changes so I think it'd be fine to publish something to maintain interoperability.
The text was updated successfully, but these errors were encountered: