Fabian Meumertzheim
Fabian Meumertzheim
Looks reasonable to me, could you send a PR?
Actually, it looks like Bazel's test setup already does this: https://cs.opensource.google/bazel/bazel/+/master:tools/test/windows/tw.cc;l=543?q=%22%5C%22HOME%5C%22%22&ss=bazel%2Fbazel Do you have a reproducer for a situation in which it isn't set?
Which version of Xcode are you on? I think I saw similar error messages on Slack and this was resolved by just updating to the latest version.
@hanwen-flow Could you test this PR on your machine? I would like to make sure that it actually resolves the issue for you and doesn't just show a warning.
I manually verified that this results in `sun.jnu.encoding = UTF-8` on a Linux machine without the `en_US.ISO-8859-1` locale installed and with `LC_ALL=C` set on server startup, so I'm positive that...
@tjgq Friendly ping
@bazel-io fork 8.4.0
I don't have any better idea than selectively allowing `**kwargs` in `MODULE.bazel` files and recommending them for this use case.
One more alternative would be to accept pairs as positional arguments: `use_repo(ext, ("bar-baz", "foo"))`
~For `override_repo`, you can use `repo_name` on the dep to rename it to something supported by `override_repo`. That requires changing all other references to it in your BUILD files, but...