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

Fix Linux build & some warnings #6438

Merged
merged 4 commits into from
Aug 18, 2024
Merged

Conversation

Noisyfox
Copy link
Collaborator

Fixes Linux Debug build.
Fixes some "loop variable creates a copy from type" and "loop variable binds to a temporary constructed from type" warnings so for loops should be slightly faster.

…f deps on Linux.

This prevents downloading the same source packages twice, and avoid downloading again after deleting the build dir.
…ble binds to a temporary constructed from type"
@henrivdr
Copy link
Contributor

Snap. I was busy making the same fixes 🤓
You beat me to it because I'm super rusty on C++ these days.
I had those asserts in Emboss commented out, just to get it compiled.

The ../deps outside the code tree puzzled me.

@henrivdr
Copy link
Contributor

I would love to know how Orca worked the (un)sorted assert which was introduced 3 months ago.

Copy link
Owner

@SoftFever SoftFever left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM
Thanks!

@SoftFever SoftFever merged commit f136f04 into SoftFever:main Aug 18, 2024
15 checks passed
@Noisyfox Noisyfox deleted the bugfox/fix-linux branch August 18, 2024 06:22
henrivdr pushed a commit to henrivdr/OrcaSlicer that referenced this pull request Aug 18, 2024
* Fix linux deps debug build

* Use the same DL_CACHE for release build when building debug version of deps on Linux.
This prevents downloading the same source packages twice, and avoid downloading again after deleting the build dir.

* Fix debug build

* Fix warnings "loop variable creates a copy from type" and "loop variable binds to a temporary constructed from type"
Ocraftyone added a commit to Ocraftyone/OrcaSlicer-Ocraftyone that referenced this pull request Sep 13, 2024
The root cause for not being able to find the debug version of Qhull was solved in SoftFever#6438
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.

3 participants