Upgrade rules_cc to 7bd0e49cfb2237086df89dca9e6ba7a5a5dfb36d#2712
Upgrade rules_cc to 7bd0e49cfb2237086df89dca9e6ba7a5a5dfb36d#2712achew22 wants to merge 1 commit intobazel-contrib:masterfrom
Conversation
This is a part of bazel-contrib/bazel-gazelle#760. To update the copy in the bazel mirror: ``` cd $(mktemp -d) wget https://github.com/bazelbuild/rules_cc/archive/7bd0e49cfb2237086df89dca9e6ba7a5a5dfb36d.zip gsutil cp 02becfef8bc97bda4f9bb64e153f1b0671aec4ba.zip gs://mirror.bazel.build/github.com/bazelbuild/rules_cc/archive/02becfef8bc97bda4f9bb64e153f1b0671aec4ba.zip ``` Fixed: bazel-contrib#2711
|
Interestingly, it looks like the test is failing because of a minimum version issue. The missing keyword is flagged by |
|
Is it important that this update happens now? I'll update everything for the |
Sorry to make that unclear. It's totally okay to hold off on this. I was merely trying to
No rush. Thanks for being so prompt and wonderful 🙂 |
|
Cool, will hold off for now then. Thanks for working through this and for being patient. I know multi-stage changes over a long time are hard to deal with. |
|
With |
|
@UebelAndre Yes. I'll close this PR since it's superseded by #2742. |
…ntrib#2712) The validating `py_test` generated by `compile_pip_requirements` chokes when the source `requirements.txt` is stored read-only, such as when managed by the Perforce Helix Core SCM. Though `dependency_resolver` makes a temporary copy of this file, it does so w/ `shutil.copy` which preserves the original read-only file mode. To address this, this commit replaces `shutil.copy` with a `shutil.copyfileobj` such that the temporary file is created w/ permissions according to the user's umask. Resolves (bazel-contrib#2608). --------- Co-authored-by: Ignas Anikevicius <240938+aignas@users.noreply.github.com>
3 participants
This is a part of bazel-contrib/bazel-gazelle#760.
To update the copy in the bazel mirror:
Fixed: #2711