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

Build Ubuntu package #53

Merged
merged 2 commits into from
Apr 6, 2022
Merged

Build Ubuntu package #53

merged 2 commits into from
Apr 6, 2022

Conversation

norihiro
Copy link
Contributor

@norihiro norihiro commented Jan 10, 2022

Description

This PR implements the packaging flow for Ubuntu.

  • ONNX runtime is installed into /usr/lib/obs-virtualbg/ by following obs-backgroundremoval in AUR. Even if another package uses a different version of ONNX runtime, it should not conflict.
  • Debian package is created by checkinstall. The code came from the previous obs-plugintemplate
    • License is modified to MIT since the plugin is released under MIT license.
    • The maintainer property is not implemented.
  • In CMakeLists.txt, CMAKE_INSTALL_FULL_LIBDIR is not expanded to /usr/lib, I implemented another variable OBS_PLUGIN_DESTINATION that can be specified from cmake's command line.

How it has been tested

OS: Ubuntu 20.04.3 LTS.
I ran this flow on Github Actions with my account, downloaded the package and installed to my Ubuntu machine on VirtualBox. And, I installed the package and confirmed the background becomes transparent.

Close #52.

@hazrpg
Copy link

hazrpg commented Jan 16, 2022

I can confirm that the plugin does install from this, but the background didn't become transparent - that could just be the quality of my mic - or the ML that it might be based on. But I'm glad that the package installed though! Blend just showed my webcam as normal (no transparent background), and mask was just all black so I couldn't even mask it.

@kounoike
Copy link
Owner

kounoike commented Apr 6, 2022

@norihiro Sorry for later reply. Thanks to your contributing!
I prefer CPack, so I had to rewrite it.

@kounoike kounoike merged commit d657e0c into kounoike:main Apr 6, 2022
@kounoike kounoike added feature Implements new feature minor labels Apr 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Implements new feature minor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Linux support?
3 participants