Skip to content

Fix nlohmann_json include on Ubuntu22#23

Merged
Maxim-Doronin merged 1 commit intodevelopfrom
md/fix_nlohmann_json_includes_u22
Aug 16, 2023
Merged

Fix nlohmann_json include on Ubuntu22#23
Maxim-Doronin merged 1 commit intodevelopfrom
md/fix_nlohmann_json_includes_u22

Conversation

@Maxim-Doronin
Copy link
Copy Markdown
Collaborator

@Maxim-Doronin Maxim-Doronin commented Aug 16, 2023

Summary

The reason is https://github.com/openvinotoolkit/openvino/blob/releases/2023/0/thirdparty/CMakeLists.txt#L604
On Ubuntu20 we have 3.7.3 in apt-get, so OV add and export its own target
On Ubuntu22 we have 3.10.5 in apt-get, so OV ignores its own nlohmann_json and doesn't export IE::nlohmann_json

Target Platform For Release Notes (Mandatory)

Aids the generation of release notes

  • VPUX3011
  • VPUX3011
  • VPUX3110
  • VPUX37XX
  • NONE (Not included in release notes)

Classification of this Pull Request

  • Maintenance
  • BUG
  • Feature

Related PRs

(Please add links to related PRs (if you have such PRs) and a small note why you depend on it)

  • ()

Related tickets

(Please list tickets which the PR closes if you have any)

  • E#####

Code Review Survey (Copy and Complete in your code review)

  • number_minutes_spent_on_review[0]
  • number_p1_defects_found[0]
  • number_p2_defects_found[0]
  • number_p3_defects_found[0]
  • number_p4_defects_found[0]

@Maxim-Doronin Maxim-Doronin requested a review from a team as a code owner August 16, 2023 14:36
Copy link
Copy Markdown
Contributor

@OrestChura OrestChura left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Maxim-Doronin Maxim-Doronin merged commit 914dd10 into develop Aug 16, 2023
@Maxim-Doronin Maxim-Doronin deleted the md/fix_nlohmann_json_includes_u22 branch August 16, 2023 15:11
Maxim-Doronin added a commit to Maxim-Doronin/npu_compiler that referenced this pull request Aug 18, 2025
Maxim-Doronin added a commit that referenced this pull request Aug 21, 2025
* UD2025.32 Update file batch #1

* UD2025.32 Update file batch #2

* UD2025.32 Update file batch #3

* UD2025.32 Update file batch #4

* UD2025.32 Update file batch #5

* UD2025.32 Update file batch #6

* UD2025.32 Update file batch #7

* UD2025.32 Update file batch #8

* UD2025.32 Update file batch #9

* UD2025.32 Update file batch #10

* UD2025.32 Update file batch #11

* UD2025.32 Update file batch #12

* UD2025.32 Update file batch #13

* UD2025.32 Update file batch #14

* UD2025.32 Update file batch #15

* UD2025.32 Update file batch #16

* UD2025.32 Update file batch #17

* UD2025.32 Update file batch #18

* UD2025.32 Update file batch #19

* UD2025.32 Update file batch #20

* UD2025.32 Update file batch #21

* UD2025.32 Update file batch #22

* UD2025.32 Update file batch #23

* UD2025.32 Update file batch #24

* UD2025.32 Update file batch #25

* UD2025.32 Update file batch #26

* UD2025.32 Update file batch #27

* UD2025.32 Update file batch #28

* UD2025.32 Update file batch #29

* UD2025.32 Update file batch #30

* UD2025.32 Update file batch #31

* UD2025.32 Update file batch #32

* UD2025.32 Update file batch #33

* UD2025.32 Update file batch #34

* UD2025.32 Update file batch #35

* UD2025.32 Update file batch #36

* UD2025.32 Update file batch #37

* UD2025.32 Update file batch #38

* UD2025.32 Update file batch #39

* UD2025.32 Update file batch #40

* UD2025.32 Update ELF

* WA for GCC warning

* Revert driver update

* Test fixes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants