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.

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

Update QHull and BuildMacOS.sh to support debug and XCode builds #3173

Open
wants to merge 61 commits into
base: master
Choose a base branch
from

Conversation

nanoplane
Copy link
Sponsor

updated Qhull.cmake to add patch to the imported Qhull CMakeLists.txt so that it creates correctly named debug libraries.

This allows working debug libraries to be built under MacOS.

A second change in BuildMacOS.sh allows Xcode builds.

LuckyTurtleDev and others added 30 commits July 19, 2022 20:12
fix createrelease as the mac arm can't be compiled on github anymore
Added a piece of code into handle_legacy to check for aliases.
'.mf3 files won't open 2.4.58.3' supermerill#2939
…one click instead of two (recovering the old behavior).
Mitigation: remove shallow angles that seems to make it appear.
As the previous remove_colinear was on a fixed dist (taht became too small on long segments)
I created a new one that work from an angle.
supermerill#2971
And fix for the visualization of the spiral start.
enricoturri1966 and others added 22 commits August 10, 2022 23:44
…rientation or the scale of a part of a multipart object
…xtra frame the first time an imgui window is shown
Fixed division by zero on zero length extrusions.
Added bunch of asserts to check for divisions by zero.
 - contiguous: even more aligned
 - scattered: old random: random per island per layer
 - random : new random: every loop has its own random seam.
Also, can now use {tool_name} and other filaments & extruder settings in filament start/end
supermerill#2934
Tips for updating on the RPI4.
updated Qhull.cmake to add patch to the imported Qhull CMakeLists.txt so that it creates correctly named debug libraries.
@nanoplane nanoplane changed the title updated BuildMacOS.sh to support XCode builds Update QHull and BuildMacOS.sh to support debug and XCode builds Sep 1, 2022
@nanoplane nanoplane marked this pull request as draft September 1, 2022 23:53
@nanoplane
Copy link
Sponsor Author

The Qhull CMakeLists.txt directives in the original change the name of Debug Libraries resulting in library names having an "_dd" suffix added rather than just a "d" suffix added. The patch comments out the change and always just defines the base names with no suffix, allowing the overall cmake rule to apply.

The second change modifies BuildMacOS.sh to support building an Xcode project using : BuildMacOS.sh -c

@nanoplane nanoplane marked this pull request as ready for review September 2, 2022 00:03
supermerill pushed a commit that referenced this pull request Apr 25, 2023
updated Qhull.cmake to add patch to the imported Qhull CMakeLists.txt so that it creates correctly named debug libraries.
#3173
@supermerill
Copy link
Owner

supermerill commented Apr 25, 2023

merged, will check if it's okay with the auto build.

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.