Skip to content

cmake: Find TBB dependency in Config.cmake.in when building static lib#3557

Merged
lgritz merged 1 commit into
AcademySoftwareFoundation:masterfrom
p12tic:cmake-fix-dependencies-static-tbb
Sep 18, 2022
Merged

cmake: Find TBB dependency in Config.cmake.in when building static lib#3557
lgritz merged 1 commit into
AcademySoftwareFoundation:masterfrom
p12tic:cmake-fix-dependencies-static-tbb

Conversation

@p12tic
Copy link
Copy Markdown
Contributor

@p12tic p12tic commented Sep 17, 2022

TBB::tbb is added as a library dependency via TARGET_NAME_IF_EXISTS. Therefore if an executable links OpenImageIO static library without the executable itself depending on TBB, then TBB will not be linked in and result in link time errors.

Checklist:

  • I have read the contribution guidelines.
  • If this is more extensive than a small change to existing code, I
    have previously submitted a Contributor License Agreement
    (individual, and if there is any way my
    employers might think my programming belongs to them, then also
    corporate).
  • [not needed] I have updated the documentation, if applicable.
  • [not needed] I have ensured that the change is tested somewhere in the testsuite
    (adding new test cases if necessary).
  • My code follows the prevailing code style of this project.

TBB::tbb is added as a library dependency via TARGET_NAME_IF_EXISTS.
Therefore if an executable links OpenImageIO static library without the
executable itself depending on TBB, then TBB will not be linked in and
result in link time errors.
@p12tic p12tic changed the title cmake: Find TBB dependency in Config.cmake.in when building static cmake: Find TBB dependency in Config.cmake.in when building static lib Sep 17, 2022
@lgritz lgritz merged commit 96f420b into AcademySoftwareFoundation:master Sep 18, 2022
lgritz pushed a commit to lgritz/OpenImageIO that referenced this pull request Sep 18, 2022
…cademySoftwareFoundation#3557)

TBB::tbb is added as a library dependency via TARGET_NAME_IF_EXISTS.
Therefore if an executable links OpenImageIO static library without the
executable itself depending on TBB, then TBB will not be linked in and
result in link time errors.
@p12tic p12tic deleted the cmake-fix-dependencies-static-tbb branch September 20, 2022 09:46
lgritz added a commit to lgritz/OpenImageIO that referenced this pull request Sep 23, 2022
Changes since RC1:

  - ImageCache/TextureSystem both have added a `getattributetype()`
    method, which retrieves just the type of a named attribute. AcademySoftwareFoundation#3559
    (2.4.4.0)  NOTE: THIS IS AN ABI BREAKING CHANGE
  - Python: Fix the ability to `getattribute()` of int64 and uint64
    metadata or attributes. AcademySoftwareFoundation#3555 (2.4.4.0)
  - Build: Improvements to the generated cmake config files when
    building static libraries. AcademySoftwareFoundation#3552 AcademySoftwareFoundation#3557 (2.4.4.0)
  - Support for gcc 12.1. AcademySoftwareFoundation#3480 (2.4.2.1) AcademySoftwareFoundation#3551 (2.4.4.0)
  - Support building with clang 15.0. AcademySoftwareFoundation#3563 (2.4.4.0)
  - Fix cross-compiling on Android failing due to `-latomic` check. AcademySoftwareFoundation#3560
    (2.4.4.0)
  - Fix building on iOS. AcademySoftwareFoundation#3562 (2.4.4.0)
lgritz added a commit to lgritz/OpenImageIO that referenced this pull request Sep 23, 2022
Changes since RC1:

  - ImageCache/TextureSystem both have added a `getattributetype()`
    method, which retrieves just the type of a named attribute. AcademySoftwareFoundation#3559
    (2.4.4.0)  NOTE: THIS IS AN ABI BREAKING CHANGE
  - Python: Fix the ability to `getattribute()` of int64 and uint64
    metadata or attributes. AcademySoftwareFoundation#3555 (2.4.4.0)
  - Build: Improvements to the generated cmake config files when
    building static libraries. AcademySoftwareFoundation#3552 AcademySoftwareFoundation#3557 (2.4.4.0)
  - Support for gcc 12.1. AcademySoftwareFoundation#3480 (2.4.2.1) AcademySoftwareFoundation#3551 (2.4.4.0)
  - Support building with clang 15.0. AcademySoftwareFoundation#3563 (2.4.4.0)
  - Fix cross-compiling on Android failing due to `-latomic` check. AcademySoftwareFoundation#3560
    (2.4.4.0)
  - Fix building on iOS. AcademySoftwareFoundation#3562 (2.4.4.0)

Note that we bumped the version to from 2.4.3.x 2.4.4.0 to reflect the
ABI break introduced by the last minute by the IC/TS addition of a new
virtual function. ABI will not change again for 2.4.
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