-
Notifications
You must be signed in to change notification settings - Fork 354
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
Add RE2 release 2023-06-02
.
#675
Conversation
@junyer In case you have a chance to influence this: It would be great if upstream could publish release artifacts as the current source URL is not guaranteed to provide a consistent hash over time. |
Looking at the failed checks, I see that I totally forgot to configure dependencies on Abseil and GoogleTest. (Benchmark isn't in BCR, so I will have to exclude one of our targets.) |
We thought so too, but it turned out that this was just miscommunication on GitHub's part and these links also aren't stable (see the rest of bazel-contrib/SIG-rules-authors#11 for all the details). The only stable artifacts are manually created release artifacts. |
Unfortunately, I don't understand why the checks are now barfing on |
I wonder whether https://code.googlesource.com/re2 can guarantee stability. 🤔 |
I just downloaded the
Herp derp. I messed up the patch. #shamecube |
FYI, I made |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CI is still failing due to a reference to google/benchmark.
Mea culpa, I need to omit |
I formatted |
My bad, forgot that the BCR doesn't test modules as root modules. If you want to run the tests, you will need to remove the |
No worries. Done. :) |
RE2 recently started doing this to unbreak the Bazel CI, but I'm not sure what to do for the BCR CI... |
Oh, I can just copycat what Abseil are doing here. And refactor to use |
Good to go! :D |
Ping, @meteorcloudy? :)