Skip to content

CMAKE Nitro Adjustments#627

Merged
holmesv3 merged 6 commits intomdaus:mainfrom
mhocking2020:hotifx/partial-install
Feb 23, 2026
Merged

CMAKE Nitro Adjustments#627
holmesv3 merged 6 commits intomdaus:mainfrom
mhocking2020:hotifx/partial-install

Conversation

@mhocking2020
Copy link
Contributor

This changeset makes a variety of adjustments to the Nitro CMAKE with the goal of allowing it to be built in a more flexible environment. Most of these changes are due to Nitro not handling partial installs well in its current state.
Changes:

  • Made the C++ version be set only if it's the base level project. This way, a parent project can build Nitro with C++17
  • Make TRE installation and a few other installations optional so they do not fail the install process if they are not targeted
  • Added some TREs that were missing from the CMAKE
  • Make a macro target that installs all the TREs

939ecced8 Sync CODA-OSS with Projects that use expanded version (#796)
93e1231 Merge pull request #793 from mdaus/support/1.x
8af433c Merge pull request #792 from holmesv3/support/1.x
2cfe7e8 Merge branch 'support/1.x' into support/1.x
1fdb594 Add note about version scheme in README, setup CMake version info for find_package
df7f960 Add cmake module include
787418a Add note about version scheme in README, setup CMake version info for find_package
b2a0cba Add explicit implementations of polynomial velocity and acceleration (#791)
6538491 Some Updates (#790)
f4b0a71 Make fixed polys more user friendly (#788)
036b17d Add nlohmann json driver module and initial type impls (#787)
06aaa4d Parallel file reads through io::FileInputStreamOS (#784)
7aec150 Changes the default value for 'install_pydir' (#781)
46a9764 Restore ci to working order (#783)
3c5e173 attempt allow build of pull requests (#782)
026e8dd Allow derived classes that offer additional functionality (such as locking) (#780)

git-subtree-dir: externals/coda-oss
git-subtree-split: 939ecced81946804808dedbbd8db178af7d12134
@mhocking2020
Copy link
Contributor Author

Noting the differences aside from the coda-oss changes:
j2k/CmakeLists.txt - Optional prevents files not present due to a partial build failing the install command
jpeg/CmakeLists.txt - Optional prevents files not present due to a partial build failing the install command
nitf/CMakeLists.txt - Add in TREs not currently included in the CMake Build and bundle them into a single target. Also make them optional such that they don't fail partial installs
CMakelists.txt: Make it such that Nitro has to be the base level project to impose a C++ 14 requirement

@holmesv3 holmesv3 merged commit 28e3cb5 into mdaus:main Feb 23, 2026
3 checks passed
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