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

cmake: linux: sync with upstream plugin template #459

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

AdelKS
Copy link

@AdelKS AdelKS commented Dec 30, 2024

Hey,

When installing on linux, currently the project has things hard-coded in cmake to install to

  • usr/dist/obs-multi-rtmp/data/locale for the translation files
  • usr/dist/obs-multi-rtmp/bin/64bit/obs-multi-rtmp.so for the shared library

Yet I want things to go to these two folders instead

  • usr/share/obs/obs-plugins/obs-multi-rtmp/locale/
  • usr/lib64/obs-plugins/obs-multi-rtmp.so

Unfortunately I cannot do anything about it except patching the cmake files: we can sync with the upstream code for the cmake files, and that gives variables we can use to tweak where files go:

  • CMAKE_INSTALL_BINDIR
  • CMAKE_INSTALL_LIBDIR
  • CMAKE_INSTALL_DATAROOTDIR

When installing on linux, currently the project has things
hard-coded in cmake to install to

- usr/dist/obs-multi-rtmp/data/locale for the translation files
- usr/dist/obs-multi-rtmp/bin/64bit/obs-multi-rtmp.so for
  the shared library

Yet we want things to go to these two folders instead

- usr/share/obs/obs-plugins/obs-multi-rtmp/locale/
- usr/lib64/obs-plugins/obs-multi-rtmp.so

Use upstream template code [1] that enables using these vars
to decide where things get installed

- CMAKE_INSTALL_BINDIR
- CMAKE_INSTALL_LIBDIR
- CMAKE_INSTALL_DATAROOTDIR

[1] https://github.com/obsproject/obs-plugintemplate
@AdelKS AdelKS force-pushed the linux-template-sync branch from 7bcde62 to 27dedd5 Compare January 2, 2025 21:22
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