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

WIP - OSX cmake dmg #11364

Closed
wants to merge 1 commit into from
Closed

WIP - OSX cmake dmg #11364

wants to merge 1 commit into from

Conversation

DonLakeFlyer
Copy link
Contributor

So this is what I have been testing with.

@DonLakeFlyer DonLakeFlyer requested a review from HTRamsey April 23, 2024 17:40
if(MACOS)
find_library(GSTREAMER_FRAMEWORK
NAMES GStreamer
HINTS ${GSTREAMER_ROOT})
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Because we are using a Framework for gstreamer you use find_library to find it. cmake supports finding frameworks in find_library.

if (GSTREAMER_FRAMEWORK-NOTFOUND)
message(WARNING "Gstreamer Framework Not Found")
else()
set(GST_FOUND TRUE)
Copy link
Contributor Author

@DonLakeFlyer DonLakeFlyer Apr 23, 2024

Choose a reason for hiding this comment

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

Since the find_library uses a different mechanism to find the framework the GST_FOUND variable is not set automatically. So I set it manually here. I must be doing this wrong somehow.

Copy link
Contributor Author

@DonLakeFlyer DonLakeFlyer Apr 23, 2024

Choose a reason for hiding this comment

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

In this specific CMakelists.txt it does see that GST_FOUND was set to TRUE further down the file.

@@ -35,6 +35,7 @@ if(GST_FOUND)
Settings
)

message(FATAL_ERROR "GST_FOUND: ${GST_FOUND}")
Copy link
Contributor Author

Choose a reason for hiding this comment

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

But once it gets here it doesn't see that GST_FOUND is set to TRUE.

@@ -32,6 +32,7 @@
#include "GStreamer.h"
#include "VideoSettings.h"
#else
#error Boom
Copy link
Contributor Author

Choose a reason for hiding this comment

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

And then I use this to verify that the DEFINE is set correctly which it is not.

@DonLakeFlyer
Copy link
Contributor Author

@HTRamsey I added comments to the changes to show what I'm running into. Probably something simple related to my cmake stupidity.

@DonLakeFlyer DonLakeFlyer marked this pull request as draft April 23, 2024 18:02
@HTRamsey HTRamsey deleted the MacCmakeDMG branch September 20, 2024 08:26
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.

1 participant