Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add CMake support for OpenNURBS #29

Closed
wants to merge 18 commits into from
Closed

Add CMake support for OpenNURBS #29

wants to merge 18 commits into from

Conversation

aminya
Copy link
Contributor

@aminya aminya commented Dec 3, 2021 •
edited
Loading

Cherry-picked from #28

  • This pull request adds CMake support for OpenNURBS. This significantly simplifies the building and integration of OpenNURBS. The CMake files consider multiple platforms and all the dependencies.
  • It downloads the dirent header file for Windows/Clang so that OpenNURBS can be built

Compilers:

  • Windows using:
    • MSVC
    • Clang
    • shared library with MSVC
  • Linux using:
    • GCC
    • Clang
  • MacOS using:
    • Apple-Clang
    • Clang

@aminya
feat: add CMake support for OpenNURBS
9b2a76b
@aminya
fix: add ON_COMPILER_CLANG for clang compiler
bb66c2f
@aminya
feat: add cross_dirent
3a62403 
This adds the dirent library for windows + clang.
https://github.com/tronkko/dirent
@aminya
fix: add uuid.h requirement for the clang compiler on windows
a7578f5
@aminya
fix: handle opengl dependency
69bb070 
Update CMakeLists.txt
@aminya
fix: exclude opennurbs_unicode_cp932 and 949 on non-windows
d480ea0
@aminya
fix: fix zlib build in the shared mode
6726299
@aminya
fix: fix building with clang on Windows
a873463
@aminya
fix: specify the output directory for zlib
ccf5c62
@aminya
fix: use lower case opennurbs and upper case for options
87399b6
@aminya
fix: fix opennurbs_INCLUDE_DIR
5eceae3
@aminya
fix: fix building as a shared library with MSVC
4edcab1
@aminya
fix: fix building on macos by finding uuid
f195c3f
@aminya
fix: add the C++ standard if not already specified
803eeb9
@aminya
fix: download dirent for windows on demand
8ef6075
@aminya
fix: only include android_uuid on linux and android
b5c39e0
@aminya
docs: add -DCMAKE_FIND_FRAMEWORK=LAST for macos
270bc90
@aminya
docs: add arm build documentation
998fdaa
This was referenced Dec 3, 2021
Closed
Closed
@xarthurx xarthurx mentioned this pull request Aug 25, 2022
Closed
@sbaer sbaer closed this Feb 24, 2023
@sbaer
Copy link
Member

sbaer commented Feb 24, 2023

This PR was closed. The latest code in opennurbs 8.x now contains CMakeLists.txt files for opennurbs and it's child projects

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@aminya @sbaer