gh-62589: extend test for grpc

[rizsotto]

• Have you followed the guidelines for contributing?
• Have you checked that there aren't other open pull requests for the same formula update/change?
• Have you built your formula locally with brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
• Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
• Does your build pass brew audit --strict <formula> (after doing brew install <formula>)?

---

@jonchang @SMillerDev can you help me out with this? (I can't test locally and no experience with ruby either.)

It's the change to extend the test coverage of grpc to use pkg-config.

[rizsotto]

Is there a better way to fix the pkg-config for openssl than this?

PKG_CONFIG_PATH=/usr/local/opt/[email protected]/lib/pkgconfig

And how to pass it to the shell_output?

[jonchang]

Maybe something like...

ENV.prepend_path "PKG_CONFIG_PATH", Formula["[email protected]"].opt_lib/"pkgconfig"

[SMillerDev]

ld: library not found for -labsl_raw_hash_set

[rizsotto]

Yep, it really looks to me that these two versions (abseil and grpc) are incompatible. What do you guys think? What shell we do to this PR?

[SMillerDev]

First we need to have a test that breaks when the versions are incompatible. Currently it just seems like the grpc devs are bad at pkg-config. That could be a bug, but it wouldn't be in homebrew.

After that we need to either have grpc fix it, or we need to rebuild on the homebrew side.

[rizsotto]

Would it make sense to roll back the abseil version bump? (Before it was working fine.) It's a step back, but Homebrew would provide a good development experience.

[SMillerDev]

We're not rolling back the version unless upstream can tell us that these versions are fundamentally incompatible. So far we've only seen one bug report with 6,342 installs in the last 30 days.

[rizsotto]

There is another one... #61866 (comment)

Anyway, if you think I can help any of the action items, let me know.

[SMillerDev]

In that case it needs to be reported and patched upstream for sure.

[BrewTestBot]

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

[rizsotto]

Is there any way that this gets merged? I think it is improvement on the test coverage.

[fxcoudert]

Is there any way that this gets merged? I think it is improvement on the test coverage.

The test is failing, so we're not going to merge it. Hopefully this is solved in future versions.