Skip to content
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

Merged
merged 1 commit into from
Jun 1, 2023
Merged

Add RE2 release 2023-06-02. #675

merged 1 commit into from
Jun 1, 2023

Conversation

junyer
Copy link
Contributor

@junyer junyer commented May 31, 2023

Ping, @meteorcloudy? :)

@fmeum fmeum added the skip-url-stability-check Skip the URL stability check for the PR label May 31, 2023
@fmeum
Copy link
Contributor

fmeum commented May 31, 2023

@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.

@junyer
Copy link
Contributor Author

junyer commented May 31, 2023

@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.

My understanding was that the issue had been resolved as per #69. Is that not actually the case? :(

@junyer
Copy link
Contributor Author

junyer commented May 31, 2023

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.)

@fmeum
Copy link
Contributor

fmeum commented May 31, 2023

My understanding was that the issue had been resolved as per #69. Is that not actually the case? :(

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.

@junyer
Copy link
Contributor Author

junyer commented May 31, 2023

Unfortunately, I don't understand why the checks are now barfing on MODULE.bazel. Could someone please let me know?

@junyer
Copy link
Contributor Author

junyer commented May 31, 2023

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.

I wonder whether https://code.googlesource.com/re2 can guarantee stability. 🤔

@junyer junyer changed the title Add RE2 release 2023-06-01. Add RE2 release 2023-06-02. Jun 1, 2023
@junyer
Copy link
Contributor Author

junyer commented Jun 1, 2023

I wonder whether https://code.googlesource.com/re2 can guarantee stability. 🤔

I just downloaded the .tar.gz and .zip files and uploaded them straight back. The skip-url-stability-check label can be removed now. :)

Unfortunately, I don't understand why the checks are now barfing on MODULE.bazel. Could someone please let me know?

Herp derp. I messed up the patch. #shamecube

@fmeum fmeum removed the skip-url-stability-check Skip the URL stability check for the PR label Jun 1, 2023
@junyer
Copy link
Contributor Author

junyer commented Jun 1, 2023

FYI, I made compatibility_level match the current SONAME.

@meteorcloudy meteorcloudy added the presubmit-auto-run Presubmit jobs will be triggered for new changes automatically without reviewer's approval label Jun 1, 2023
Copy link
Contributor

@fmeum fmeum left a 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.

modules/re2/2023-06-02/MODULE.bazel Outdated Show resolved Hide resolved
@junyer
Copy link
Contributor Author

junyer commented Jun 1, 2023

CI is still failing due to a reference to google/benchmark.

Mea culpa, I need to omit //:benchmark too.

@junyer
Copy link
Contributor Author

junyer commented Jun 1, 2023

I formatted MODULE.bazel and that rearranged things a bit more than I was expecting, but c'est la vie.

@fmeum
Copy link
Contributor

fmeum commented Jun 1, 2023

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 dev_dependency param on the googletest dep.

@junyer
Copy link
Contributor Author

junyer commented Jun 1, 2023

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 dev_dependency param on the googletest dep.

No worries. Done. :)

@junyer
Copy link
Contributor Author

junyer commented Jun 1, 2023

RE2 recently started doing this to unbreak the Bazel CI, but I'm not sure what to do for the BCR CI...

@junyer
Copy link
Contributor Author

junyer commented Jun 1, 2023

Oh, I can just copycat what Abseil are doing here. And refactor to use matrix while I'm at it. :)

@junyer
Copy link
Contributor Author

junyer commented Jun 1, 2023

Good to go! :D

@fmeum fmeum merged commit e32cddb into bazelbuild:main Jun 1, 2023
alexeagle pushed a commit to aspect-build/bazel-central-registry that referenced this pull request Jun 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
presubmit-auto-run Presubmit jobs will be triggered for new changes automatically without reviewer's approval
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants