Skip to content
This repository has been archived by the owner on Jul 1, 2024. It is now read-only.

Linux and MacOS Standalone Builds #13

Closed
aliksend opened this issue Apr 25, 2018 · 3 comments
Closed

Linux and MacOS Standalone Builds #13

aliksend opened this issue Apr 25, 2018 · 3 comments
Labels
enhancement Enhancements

Comments

@aliksend
Copy link

Hi

Thank you for your library.
How difficult is it to compile it for OS X and linux?
I tried to compile it on my mac and got error:

CMake Error:
  Error evaluating generator expression:

    $<TARGET_PDB_FILE:obs-stream-effects>

  TARGET_PDB_FILE is not supported by the target linker.

I think that some platform-specific build configuration required like in obs-studio CMakeLists.txt. But is configuration will be enough or there is some windows-specific code in your library that makes cross-platform more complex?

Thank you
@Xaymar
Copy link
Owner

Xaymar commented Apr 26, 2018 •
edited
Loading

As far as I know, there is no code that makes it incompatible with other platforms. Standalone builds however currently require Windows, as you have already discovered. So in order to build it on Linux and Mac, you need to include it as a plugin in obs-studio.

Xaymar added a commit that referenced this issue Apr 26, 2018
@Xaymar
cmake: Improve support for Linux and MacOS
9f06d5a 
Standalone builds should not assume that everything is Windows and in the future hopefully work out of the box for Linux and MacOS. This is required if CI should also work for Linux and MacOS.

Related: #13
@Xaymar Xaymar changed the title Version for mac/linux Linux and MacOS Standalone Builds Apr 26, 2018
@aliksend
Copy link
Author

As I understood you will make builds for mac and linux. Thanks, I will wait for it.

@catb0t catb0t mentioned this issue Jan 13, 2019
Closed
Xaymar added a commit that referenced this issue Jan 14, 2019
@Xaymar
project: Improve compiler compatibility
108b972 
This change does the following:
- Remove `extern "C"` around includes that do not necessarily need them.
- Remove min, max and clamp macro (unused, legacy stuff)
- Replace <....h> include with the equivalent <c...> include

See issue #13 and #27.
@Xaymar Xaymar closed this as completed in e0e2d9f Jan 14, 2019
@Xaymar Xaymar mentioned this issue Jan 14, 2019
Closed
@Xaymar
Copy link
Owner

Xaymar commented Apr 26, 2021

After three years, StreamFX is now natively available on both platforms.

Repository owner locked as resolved and limited conversation to collaborators Apr 26, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
enhancement Enhancements
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Xaymar @aliksend