flathub / com.obsproject.Studio

This repository is no longer used to build OBS. Issues should be reported at https://github.com/obsproject/obs-studio
https://github.com/obsproject/obs-studio
29 stars 19 forks source link

Update com.obsproject.Studio.BlackMagicLibs.json #26

Closed sat3ll closed 5 years ago

sat3ll commented 5 years ago

Update to Desktop Video 11.1

flathubbot commented 5 years ago

Started test build 2084

flathubbot commented 5 years ago

Build 2084 failed

Erick555 commented 5 years ago

@sat3ll you need to bump shared-modules submodule to get udev fix.

sat3ll commented 5 years ago

@sat3ll you need to bump shared-modules submodule to get udev fix.

That should go to a different PR. This PR only deals with the BlackMagic extension.

Erick555 commented 5 years ago

@sat3ll no, you won't be able to build anything unless you make update I mentioned. Of course you can update it in separate PR if you wish.

sat3ll commented 5 years ago

@sat3ll no, you won't be able to build anything unless you make update I mentioned.

It doesn't build with the buildbot because this extension requires you to manually fetch the tar.gz file. It will build locally as instructed by https://github.com/flathub/com.obsproject.Studio/wiki/BlackMagic-Support

Erick555 commented 5 years ago

It doesn't build with the buildbot because this extension requires you to manually fetch the tar.gz file. It will build locally as instructed by https://github.com/flathub/com.obsproject.Studio/wiki/BlackMagic-Support.

@sat3ll I'm afraid you didn't read the actual buildbot logs. It failed before it even touched your extension. It will fail if you even didn't change anything. It won't build neither locally or remotely unless you update submodules. This is my point which I have hard time to sell here :smile:.

Erick555 commented 5 years ago

I created PR which will solve the issue.

sat3ll commented 5 years ago

Strange, I got it to build this morning:

$ flatpak-builder builddir --force-clean com.obsproject.Studio.BlackMagicLibs.json --user --install
Downloading sources
Initializing build dir
Committing stage init to cache
Starting build of com.obsproject.Studio.BlackMagicLibs
========================================================================
Building module blackmagic in /tmp/com.obsproject.Studio/.flatpak-builder/build/blackmagic-1
========================================================================
Running: tar -xf other/x86_64/desktopvideo-11.1a11-x86_64.tar.gz
Running: install -Dm644 --target-directory=$FLATPAK_DEST desktopvideo-11.1a11-x86_64/usr/lib/{*.so,blackmagic/DesktopVideo/libc++*.so.1}
Committing stage build-blackmagic to cache
Cleaning up
Committing stage cleanup to cache
Finishing app
Please review the exported files and the metadata
Committing stage finish to cache
Exporting com.obsproject.Studio.BlackMagicLibs to repo
Commit: 87def687acb739ecf40143718c1e244656a56040c604cd0c1dc2bb029ee4a072
Metadata Total: 5
Metadata Written: 2
Content Total: 7
Content Written: 0
Content Bytes Written: 0 (0 bytes)
Installing runtime/com.obsproject.Studio.BlackMagicLibs/x86_64/stable
Pruning cache
$

But I suppose there's no harm in updating the shared-modules. I'll leave that part to you since you've already opened a PR.

Erick555 commented 5 years ago

I guess you didn't run flatpak update before building it. Relevant changes landed just yesterday.

flathubbot commented 5 years ago

Started test build 2220

flathubbot commented 5 years ago

Build 2220 successful To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/2126/com.obsproject.Studio.flatpakref
TingPing commented 5 years ago

This also needs the runtime to be bumped to 5.13 now.