[protobuf] Revert "improve discovery of protoc executable in build context"

[uilianries]

The PR #20089 is causing side-effects listed in the issue #21737 and is blocking #22244

fixes #21737

[github-actions]

🤖 Beep Boop! This pull request is making changes to 'recipes/protobuf//'.

👋 @Hopobcn you might be interested. 😉

[ghost]

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

• Add protobuf 3.15.8 #22527
• protobuf: add v4.25.3, v5.26.0 #21622
• Adding protobuf/3.5.1 #20627
• Fixed issue #18927 #18929
• protobuf: add version 4.23.1 #16098

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

[SpaceIm]

Please no, openvino recipe is likely broken since it's missing protobuf in build requirements in case of native build. It was working by luck. Issues reported in #21737 are unclear and missing proper issue template informations to investigate.

[ilya-lavrenov]

Please no, openvino recipe is likely broken since it's missing protobuf in build requirements in case of native build. It was working by luck. Issues reported in #21737 are unclear and missing proper issue template informations to investigate.

Thanks, let's check #22289

[SpaceIm]

Could you try exactly what is suggested in #22244 (comment) please?

• VirtualBuildEnv since there are build requirements (implicit with conan v2 client, but not with conan v1 client)
• Do not add protobuf & flatbuffers to build requirements if 1 profile (due to potential issues in conan v1 in case of dependencies of these libs overridden by users).
• VirtualRunEnv in build scope if 1 profile

[conan-center-bot]

Conan v1 pipeline ✔️

All green in build 2 (0b0735ea549ee77bacccc869055a89c8f122254a):

• protobuf/3.21.12:
  All packages built successfully! (All logs)

• protobuf/3.21.9:
  All packages built successfully! (All logs)

• protobuf/3.20.0:
  All packages built successfully! (All logs)

• protobuf/3.18.3:
  All packages built successfully! (All logs)

• protobuf/3.19.6:
  All packages built successfully! (All logs)

• protobuf/3.17.1:
  All packages built successfully! (All logs)

• protobuf/3.20.3:
  All packages built successfully! (All logs)

---

Conan v2 pipeline ✔️

Note: Conan v2 builds are now mandatory. Please read our discussion about it.

All green in build 2 (0b0735ea549ee77bacccc869055a89c8f122254a):

• protobuf/3.21.12:
  All packages built successfully! (All logs)

• protobuf/3.21.9:
  All packages built successfully! (All logs)

• protobuf/3.20.0:
  All packages built successfully! (All logs)

• protobuf/3.20.3:
  All packages built successfully! (All logs)

• protobuf/3.19.6:
  All packages built successfully! (All logs)

• protobuf/3.17.1:
  All packages built successfully! (All logs)

• protobuf/3.18.3:
  All packages built successfully! (All logs)

[ilya-lavrenov]

Looks like it can be closed

[uilianries]

Closing this issue as the PR #22244 is no longer blocked after updating protobuf usage.