googleapis: modernize more & bump protobuf

[SpaceIm]

• trick to avoid having protobuf both in requirements & build requirements (if native build for 1 & 2 profiles)
• use rm_safe
• use self.dependencies
• use tool_requires
• reorder methods by order of execution to give a better understanding of global recipe logic

---

• I've read the contributing guidelines.
• I've used a recent Conan client version close to the currently deployed.
• I've tried at least one configuration locally with the conan-center hook activated.

[ghost]

I detected other pull requests that are modifying googleapis/all recipe:

• googleapis: fix issues with shared=True in Conan 2.0 #16422
• googleapis: upgrade protobuf version to 3.21.9 #16034

This message is automatically generated by https://github.com/ericLemanissier/conan-center-conflicting-prs so don't hesitate to report issues/improvements there.

[jcar87]

From my own local testing, I'm pretty sure that googleapis simply does not build with Protobuf 3.21.9 on macOS, due to a clash between a macro defined in the system's syslimits.h and code generated by protoc.

It's unclear why it went from working (3.21.4) to not working (3.21.9), but it could be simply be caused as a side effect of the order of implicitly included headers changing.

/Users/jenkins/w/prod/BuildSingleReference@7/.conan/data/googleapis/cci.20220531/_/_/build/d52f4199324621c40024ef7b88e1b3bafd1d4af3/build/Release/google/storagetransfer/v1/transfer_types.pb.h:3520:24: error: expected member name or ';' after declaration specifiers
  static constexpr GID GID_MAX =
  ~~~~~~~~~~~~~~~~~~~~ ^
/Applications/conan/xcode/11.0/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.15.sdk/usr/include/sys/syslimits.h:78:28: note: expanded from macro 'GID_MAX'
#define GID_MAX            2147483647U  /* max value for a gid_t (2^31-2) */
                           ^
1 error generated.

this may be fixed in newer versions of protobuf:
https://github.com/protocolbuffers/protobuf/blob/v22.0/src/google/protobuf/port_def.inc#L971-L972

[prince-chrismc]

There's merge conflicts and the downstream dep was fixed should be good to move forward

[SpaceIm]

/opt/conan/binutils/bin/ld: CMakeFiles/test_package.dir/test_package.cpp.o: undefined reference to symbol '_ZNK6google8protobuf7Message11DebugStringB5cxx11Ev'
/opt/conan/binutils/bin/ld: /home/conan/w/prod-v2/BuildSingleReference/p/protoe1ed88265f4f3/p/lib/libprotobuf.so.32: error adding symbols: DSO missing from command line

well, transitive_libs is missing

[prince-chrismc]

This is can be removed with the new portable binaries

[prince-chrismc]

Super minor nits

[conan-center-bot]

Conan v1 pipeline ✔️

All green in build 11 (924409fe093821628d5c7df024a1073c1c690f5a):

• googleapis/cci.20210730@:
  All packages built successfully! (All logs)

• googleapis/cci.20220711@:
  All packages built successfully! (All logs)

• googleapis/cci.20221108@:
  All packages built successfully! (All logs)

• googleapis/cci.20211122@:
  All packages built successfully! (All logs)

• googleapis/cci.20220531@:
  All packages built successfully! (All logs)

---

Conan v2 pipeline (informative, not required for merge) ✔️

Note: Conan v2 builds are informative and they are not required for the PR to be merged.

All green in build 9 (924409fe093821628d5c7df024a1073c1c690f5a):

• googleapis/cci.20220711@:
  All packages built successfully! (All logs)

• googleapis/cci.20221108@:
  All packages built successfully! (All logs)

• googleapis/cci.20211122@:
  All packages built successfully! (All logs)

• googleapis/cci.20220531@:
  All packages built successfully! (All logs)

• googleapis/cci.20210730@:
  All packages built successfully! (All logs)

[prince-chrismc]

#16034 has the same changes so we will have merge conflicts :(