CMake build for Linux and Windows #339
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jmrandol
approved these changes
Apr 30, 2020
kaydoh
pushed a commit
to kaydoh/six-library
that referenced
this pull request
Feb 9, 2022
11e1260 Merge pull request ngageoint#339 from mdaus/fileStream-cleanup 7c34d0a style fix 4ea8a1c Fixing line endings 02eb685 Fix clean up in fileStreamOS destructors 37b85e7 Merge pull request ngageoint#338 from mdaus/fix_bad_unittest_condition 08e98cb Fix incorrect range check. Seconds are floating-point values, so, e.g., 59.23 should be valid. 480de23 Merge pull request ngageoint#337 from mdaus/jenkins-debug d257f02 Reformat README e3a0671 Document error in README c1f065c Remove Python2-specific function 5721dd1 Merge pull request ngageoint#336 from mdaus/getNumpyArraySize 02b1900 Address memory leak 2257341 Proper signature for main 9cab375 Add getNumElements to numpyutils 9cc1dae Merge pull request ngageoint#331 from mdaus/scratch_memory_unittest_fix 0a0b800 Merge pull request ngageoint#335 from mdaus/const_data_to_numpy_array 63be556 Allow converting a const buffer to a numpy array 473f206 Merge pull request ngageoint#332 from mdaus/pass_swig_home 37fb901 Merge pull request ngageoint#333 from mdaus/8_bit_integer_to_string ae40302 mDayOfWeek for Saturday should be 7, not 0 fbe556c Use proper offset when converting to tm_wday 723d901 Treat 8-bit integers as numbers rather than characters when converting to string 9a3ef58 Throw if attempting to release key not in ScratchMemory eda690e Push back element before getting iterator 321af0c Allow specifying custom swig install 6e28a86 Add <algorithm> header for std::find 5ea6906 Fix incorrect checking in unittest git-subtree-dir: externals/coda-oss git-subtree-split: 11e126064f6891ca9f74e44347ae010a6cd1ae43
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
These changes add CMake build capabilities for Linux and Windows alongside the existing waf build system.
Github Actions workflows were added to build and run unit tests on pushes to the repo.
Preliminary support for Conan builds is also included. When building via Conan, the externals folder is ignored and these dependencies are imported from coda-oss and nitro Conan packages.
The build exports all CMake targets to a file within the install directory (lib/cmake/six-libraryConfig.cmake). This file can be used to import them into downstream project CMake builds.