Update runfiles initialization v3 to reference rules_shell

[peakschris]

This line:
https://github.com/bazelbuild/rules_shell/blob/e6d845428779b1e5ebb9971451ed3c43c865fc0b/shell/runfiles/runfiles.bash#L72

Needs to be updated to:

set -uo pipefail; set +e; f=rules_shell/shell/runfiles/runfiles.bash

[fmeum]

It's correct as is, we kept it the same as before to ensure backwards compatibility and because we need to hardcode the repo name, which isn't cleanly possible for modules other than bazel_tools.

Are you experiencing any errors that you think may be caused by this?

[peakschris]

I see the other issues now: #25

I'm trying to make use of runfiles in rules_oci and struggling... I agree with the others that this is very confusing!

Should rules_oci (as a ruleset that must work with bazel 7) have:

• dependency on rules_shell
• use rules_shell/shell:toolchain_type where needed
• and depend on "@rules_shell//shell/runfiles",

Or:

• use the old bazel_tools references?

Part of the problem is that I follow the reference to rules_shell from bazel_tools and think that this is the current best practice but there is no documentation and I start to make random changes and get lost :-)

Some simple current docs or reference from here to docs would really help

[fmeum]

Sorry for the late reply. I should add better docs.

The simple answer is that it shouldn't matter, both rules_shell and bazel_tools should work and will continue to work for a while. rules_shell is more future proof though.