opentelemetry-cpp: fix CMake target include dirs

[algorithmiker]

• opentelemetry-cpp: 1.20.0 -> 1.22.0, opentelemetry-proto: 1.5.0 -> 1.7.0 #440194 re-introduced the bug opentelemetry-cpp cmake find_package fails with non-existent path #297443.
• This cherry-picks 2b99603 again¹, which was effectively reverted in that PR.

This broke any library depending on opentelemetry-cpp; but nixpkgs-review didn't catch this because there are no in-tree deps yet (pending #435928).

Things done

• Built on platform:
  • x86_64-linux
  • aarch64-linux
  • x86_64-darwin
  • aarch64-darwin
• Tested, as applicable:
  • NixOS tests in nixos/tests.
  • Package tests at passthru.tests.
  • Tests in lib/tests or pkgs/test for functions and "core" functionality.
• Ran nixpkgs-review on this PR. See nixpkgs-review usage.
• Tested basic functionality of all binary files, usually in ./result/bin/.
• Nixpkgs Release Notes
  • Package update: when the change is major or breaking.
• NixOS Release Notes
  • Module addition: when adding a new NixOS module.
  • Module update: when the change is significant.
• Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

---

Add a 👍 reaction to pull requests you find important.

Footnotes

1. or rather the slightly modified version of it from 390485. ↩

[pbsbot]

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 442584
Commit: 8fd6d01f7e0b8938b7af84456c79243263488108

---

x86_64-linux

✅ 2 packages built:

• opentelemetry-cpp
• opentelemetry-cpp.dev

[pbsds]

Thanks for noticing and following up on this! I was a bit too quick when merging that, assuming it was a dropped postPatch which no longer applied.

[panicgh]

Thanks for fixing. I explained the mistake in the discussion at #440194 (comment).

[algorithmiker]

That was very fast, thank you!