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: macOS build #2551

Closed
wants to merge 1 commit into from
Closed

fix: macOS build #2551

wants to merge 1 commit into from

Conversation

remoe
Copy link
Contributor

@remoe remoe commented May 31, 2024

MacOS build failed because of current CMake protobuf integration. And it had some minor compile issues.

@remoe remoe force-pushed the fix/macos-build branch from 6001f70 to 743a46e Compare June 8, 2024 08:35
@remoe
Copy link
Contributor Author

remoe commented Jun 8, 2024

Here is the error for current implementation of Presets in clang without the fix of current PR:

Default constructor of 'Presets' is implicitly deleted because field '_nodata' of const-qualified type 'const osgEarth::Config' would not be initialized

@gwaldron
Copy link
Owner

gwaldron commented Jun 10, 2024

Remo, what version of cmake do you use?

@gwaldron
Copy link
Owner

Remo, I pushed my own version of these fixes. Let me know. Thanks!

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