Skip to content
This repository was archived by the owner on Mar 15, 2021. It is now read-only.

Conversation

@DaveWK
Copy link

@DaveWK DaveWK commented May 3, 2020

note: I know this one is up: #33 but I think i was able to trim down the build time considerably by reworking the cmake a bit..

…g obs-studio

There's a lot of deps I seemed to be mising to compile obs-studio

Interstingly the "dev" package for Fedora does contain the ObsPluginHelpers.cmake however the Ubuntu one does not

I noticed the config cmake for libObs seemed to have any values or things you'd need out of it so the "include dir" statement isn't needed

You can still pass in a var to get the old behaviour and leverage any possibly shared libs like ffmped with the LIBOBS_CMAKE_MODULES var

I was able to successfuly get this working with Fedora 32

…g obs-studio

There's a lot of deps I seemed to be mising to compile obs-studio

Interstingly the "dev" package for Fedora does contain the ObsPluginHelpers.cmake however the Ubuntu one does not

I noticed the config cmake for libObs seemed to have any values or things you'd need out of it so the "include dir" statement isn't needed

You can still pass in a var to get the old behaviour and leverage any possibly shared libs like ffmped with the LIBOBS_CMAKE_MODULES var

I was able to successfuly get this working with Fedora 32
@raylu
Copy link

raylu commented May 4, 2020

on my Debian system, I get

-- Detecting CXX compile features - done
-- Found Libobs: /usr/lib/x86_64-linux-gnu/libobs.so  
CMake Error at external/FindLibObs.cmake:76 (include):
  include could not find load file:

    /LibObsConfig.cmake
Call Stack (most recent call first):
  CMakeLists.txt:5 (include)


-- Configuring incomplete, errors occurred!

rossmeier@e6102b5 gets me past this error

README.md Outdated
cd obs-v4l2sink
mkdir build && cd build
cmake -DLIBOBS_INCLUDE_DIR="../../obs-studio/libobs" -DCMAKE_INSTALL_PREFIX=/usr ..
cmake ..
Copy link

@scottlinux scottlinux May 5, 2020

Choose a reason for hiding this comment

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

I had to specify the make install prefix as well like:

-DCMAKE_INSTALL_PREFIX=/usr

Otherwise the make install puts the plugin in /usr/local/lib/obs-plugins/v4l2sink.so which obs does not see on Fedora.

obs on Fedora appears to expect:

/usr/lib64/obs-plugins/v4l2sink.so

Copy link
Author

Choose a reason for hiding this comment

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

798adea

added it back in ^^

@scottlinux
Copy link

@lukester1975 gave steps to build on Fedora 32 without the need of a PR:

#42 (comment)

@DaveWK
Copy link
Author

DaveWK commented May 22, 2020

(Reposted/deleted from other thread) I think those instrructions are using dependencies off the system without specifying them for obs-studio, which additionally was why I wanted to not compile obs-studio to build the plugin, as it adds extra packages and steps to make it. if I install the bin for obs-studio from a package manager I dont need to install as many deps or wait for obs-studio to compile just to get the static libs.

@DaveWK
Copy link
Author

DaveWK commented May 22, 2020

@raylu -- are you sure you installed libobs-dev? I think this is the right place for debian and ubuntu? https://packages.ubuntu.com/bionic/amd64/libobs-dev/filelist

@DaveWK
Copy link
Author

DaveWK commented May 23, 2020

Added some dockerfiles to prove that it's working on both debian and fedora now; there is an odd bug in obs-studio's packaging where the ubuntu/debian version doesn't contain obs-frontend-api.h (but the Fedora package does) -- a fix was merged a few days ago: obsproject/obs-studio#2635

@DaveWK
Copy link
Author

DaveWK commented May 23, 2020

I am not super familiar with if there's some trickery that I can do to make cmake use relative paths for the install makefile, but it's almost at the point where it seems like using CPack and making an rpm/deb might be a simpler solution to package it all in a docker container..

@DaveWK DaveWK closed this Feb 5, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants