GNU bug report logs -
#45707
[PATCH] gnu: Add obs-v4l2sink.
Previous Next
Reported by: Evan Straw <evan.straw99 <at> gmail.com>
Date: Wed, 6 Jan 2021 23:37:01 UTC
Severity: normal
Tags: patch
Done: Ludovic Courtès <ludo <at> gnu.org>
Bug is archived. No further changes may be made.
To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 45707 in the body.
You can then email your comments to 45707 AT debbugs.gnu.org in the normal way.
Toggle the display of automated, internal messages from the tracker.
Report forwarded
to
guix-patches <at> gnu.org
:
bug#45707
; Package
guix-patches
.
(Wed, 06 Jan 2021 23:37:02 GMT)
Full text and
rfc822 format available.
Acknowledgement sent
to
Evan Straw <evan.straw99 <at> gmail.com>
:
New bug report received and forwarded. Copy sent to
guix-patches <at> gnu.org
.
(Wed, 06 Jan 2021 23:37:02 GMT)
Full text and
rfc822 format available.
Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):
[0001-gnu-Add-obs-v4l2sink.patch (text/x-patch, inline)]
From 0aa683242ae591d0ddac8b4b23163805dc4414c9 Mon Sep 17 00:00:00 2001
From: Evan Straw <evan.straw99 <at> gmail.com>
Date: Wed, 6 Jan 2021 15:33:25 -0800
Subject: [PATCH] gnu: Add obs-v4l2sink.
To: guix-patches <at> gnu.org
* gnu/packages/video.scm (obs-v4l2sink): New variable.
---
gnu/packages/video.scm | 39 +++++++++++++++++++++++++++++++++++++++
1 file changed, 39 insertions(+)
diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index 2b1b111e97..6d64d03e44 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -3133,6 +3133,45 @@ and JACK.")
(home-page "https://obsproject.com")
(license license:gpl2+)))
+(define-public obs-v4l2sink
+ (package
+ (name "obs-v4l2sink")
+ (version "0.1.0")
+ (source (origin
+ (method git-fetch)
+ (uri (git-reference (url "https://github.com/CatxFish/obs-v4l2sink")
+ (commit version)))
+ (file-name (string-append name "-" version))
+ (sha256
+ (base32
+ "0l4lavaywih5lzwgxcbnvdrxhpvkrmh56li06s3aryikngxwsk3z"))))
+ (build-system cmake-build-system)
+ (arguments `(#:tests? #f ;; obs-v4l2sink does not have tests
+ #:phases
+ (modify-phases %standard-phases
+ ;; The CMakeLists.txt file for obs-v4l2sink attempts to use
+ ;; its own method to search for libobs which uses a lot of
+ ;; hardcoded paths and ultimately fails under Guix. OBS
+ ;; already comes with its own CMake configuration, so
+ ;; remove the line that uses the hardcoded method and just
+ ;; let CMake locate OBS.
+ (add-before 'configure 'fix-libobs
+ (lambda* _
+ (substitute* "CMakeLists.txt"
+ (("^.*FindLibObs.*$") ""))
+ #t)))))
+ (inputs `(("qtbase" ,qtbase)
+ ("obs" ,obs)))
+ (home-page "https://github.com/CatxFish/obs-v4l2sink")
+ (synopsis "OBS Studio output plugin for Video4Linux2 device")
+ (description "An @dfn{Open Broadcaster Software} (OBS) Studio plugin that
+provides output capabilities to a @dfn{Video4Linux2} (v4l2) device. It can be
+used with @code{v4l2loopback} to achieve cross-program video transfer between
+OBS Studio and third party software supporting Video4Linux2, e.g. to present
+an OBS session in proprietary browser-based conferencing systems by selecting
+the OBS session as a webcam.")
+ (license license:gpl2+)))
+
(define-public libvdpau
(package
(name "libvdpau")
--
2.25.1
[signature.asc (application/pgp-signature, inline)]
Information forwarded
to
guix-patches <at> gnu.org
:
bug#45707
; Package
guix-patches
.
(Wed, 06 Jan 2021 23:43:02 GMT)
Full text and
rfc822 format available.
Message #8 received at 45707 <at> debbugs.gnu.org (full text, mbox):
Evan Straw <evan.straw99 <at> gmail.com> writes:
> +OBS Studio and third party software supporting Video4Linux2, e.g. to present
> +an OBS session in proprietary browser-based conferencing systems by selecting
> +the OBS session as a webcam.")
Seems like a proprietary software endorsement. I guess it works also
with libre browser-based conferencing systems like BBB, nextcloud talk
or jitsi, so I'd remove proprietary from the description.
Information forwarded
to
guix-patches <at> gnu.org
:
bug#45707
; Package
guix-patches
.
(Wed, 06 Jan 2021 23:46:01 GMT)
Full text and
rfc822 format available.
Message #11 received at 45707 <at> debbugs.gnu.org (full text, mbox):
Yeah, I was also a little unsure about that part as well. I will remove
that from the description. Should have another patch to send in a
minute.
-- Evan
Information forwarded
to
guix-patches <at> gnu.org
:
bug#45707
; Package
guix-patches
.
(Wed, 06 Jan 2021 23:49:02 GMT)
Full text and
rfc822 format available.
Message #14 received at 45707 <at> debbugs.gnu.org (full text, mbox):
[0001-gnu-Add-obs-v4l2sink.patch (text/x-patch, inline)]
From 7bcaffb15f1f6697310a4c4bacb8dea1eca2a4e8 Mon Sep 17 00:00:00 2001
From: Evan Straw <evan.straw99 <at> gmail.com>
Date: Wed, 6 Jan 2021 15:33:25 -0800
Subject: [PATCH] gnu: Add obs-v4l2sink.
* gnu/packages/video.scm (obs-v4l2sink): New variable.
---
gnu/packages/video.scm | 39 +++++++++++++++++++++++++++++++++++++++
1 file changed, 39 insertions(+)
diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index 2b1b111e97..194699dd23 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -3133,6 +3133,45 @@ and JACK.")
(home-page "https://obsproject.com")
(license license:gpl2+)))
+(define-public obs-v4l2sink
+ (package
+ (name "obs-v4l2sink")
+ (version "0.1.0")
+ (source (origin
+ (method git-fetch)
+ (uri (git-reference (url "https://github.com/CatxFish/obs-v4l2sink")
+ (commit version)))
+ (file-name (string-append name "-" version))
+ (sha256
+ (base32
+ "0l4lavaywih5lzwgxcbnvdrxhpvkrmh56li06s3aryikngxwsk3z"))))
+ (build-system cmake-build-system)
+ (arguments `(#:tests? #f ;; obs-v4l2sink does not have tests
+ #:phases
+ (modify-phases %standard-phases
+ ;; The CMakeLists.txt file for obs-v4l2sink attempts to use
+ ;; its own method to search for libobs which uses a lot of
+ ;; hardcoded paths and ultimately fails under Guix. OBS
+ ;; already comes with its own CMake configuration, so
+ ;; remove the line that uses the hardcoded method and just
+ ;; let CMake locate OBS.
+ (add-before 'configure 'fix-libobs
+ (lambda* _
+ (substitute* "CMakeLists.txt"
+ (("^.*FindLibObs.*$") ""))
+ #t)))))
+ (inputs `(("qtbase" ,qtbase)
+ ("obs" ,obs)))
+ (home-page "https://github.com/CatxFish/obs-v4l2sink")
+ (synopsis "OBS Studio output plugin for Video4Linux2 device")
+ (description "An @dfn{Open Broadcaster Software} (OBS) Studio plugin that
+provides output capabilities to a @dfn{Video4Linux2} (v4l2) device. It can be
+used with @code{v4l2loopback} to achieve cross-program video transfer between
+OBS Studio and third party software supporting Video4Linux2, e.g. to present
+an OBS session in browser-based conferencing systems by selecting the OBS
+session as a webcam.")
+ (license license:gpl2+)))
+
(define-public libvdpau
(package
(name "libvdpau")
--
2.25.1
[signature.asc (application/pgp-signature, inline)]
Information forwarded
to
guix-patches <at> gnu.org
:
bug#45707
; Package
guix-patches
.
(Thu, 07 Jan 2021 00:12:02 GMT)
Full text and
rfc822 format available.
Message #17 received at 45707 <at> debbugs.gnu.org (full text, mbox):
Thanks for the update.
Evan Straw <evan.straw99 <at> gmail.com> writes:
> + (method git-fetch)
> + (uri (git-reference (url "https://github.com/CatxFish/obs-v4l2sink")
> + (commit version)))
I'm not 100% sure (I usually don't do patch review) but I think the
indentation is wrong (line should break after git-reference).
> + (inputs `(("qtbase" ,qtbase)
> + ("obs" ,obs)))
I think it's the same after inputs..
> + (description "An @dfn{Open Broadcaster Software} (OBS) Studio plugin that
> +provides output capabilities to a @dfn{Video4Linux2} (v4l2) device. It can be
> +used with @code{v4l2loopback} to achieve cross-program video transfer between
I think that instead of definitions, those are acronyms, so it should be
@acronym{OBS, Open Broadcaster Software} and @acronym{Video 4 Linux 2,
vl42}.
I'd say that you should wait for somebody that knows better than me
before applying my suggestions.
It builds and lint is ok, but obs does not work on my laptop (video card
too old) so I cannot test that it run.
Nicolo
Information forwarded
to
guix-patches <at> gnu.org
:
bug#45707
; Package
guix-patches
.
(Thu, 07 Jan 2021 04:38:02 GMT)
Full text and
rfc822 format available.
Message #20 received at 45707 <at> debbugs.gnu.org (full text, mbox):
[Message part 1 (text/plain, inline)]
On further investigation, I'm not even sure this package will work correctly because of the way OBS handles plugins. To install plugins, OBS requires that you add their .so files to a plugins directory under the OBS share directory, which under Guix should be immutable. As far as I'm aware, there doesn't seem to be a way to get OBS to use a different directory or use an environment variable containing search paths. Not sure if there is a way to even get plugins for OBS to work at all under Guix if this is the case.
Apologies for creating this patch without having really done this research first. If there's really no way to get this to work under Guix, then feel free to just close this ticket.
--Evan
Jan 6, 2021 3:47:55 PM Evan Straw <evan.straw99 <at> gmail.com>:
>
[signature.asc (application/pgp-signature, inline)]
Information forwarded
to
guix-patches <at> gnu.org
:
bug#45707
; Package
guix-patches
.
(Thu, 07 Jan 2021 08:20:02 GMT)
Full text and
rfc822 format available.
Message #23 received at 45707 <at> debbugs.gnu.org (full text, mbox):
Hi Evan,
You can make it work.
In order to provide plugins for obs via $XDG_CONFIG_HOME (~/.config) you need to provide the following structure:
"$XDG_CONFIG_HOME/obs-studio/plugins/<module-name>/bin/<arch>/"
As it implemented in obs-studio (~/obs-studio/UI/window-basic-main.cpp)
--8<---------------cut here---------------start------------->8---
static void AddExtraModulePaths()
{
char base_module_dir[512];
#if defined(_WIN32) || defined(__APPLE__)
int ret = GetProgramDataPath(base_module_dir, sizeof(base_module_dir),
"obs-studio/plugins/%module%");
#else
int ret = GetConfigPath(base_module_dir, sizeof(base_module_dir),
"obs-studio/plugins/%module%");
#endif
if (ret <= 0)
return;
string path = base_module_dir;
#if defined(__APPLE__)
obs_add_module_path((path + "/bin").c_str(), (path + "/data").c_str());
BPtr<char> config_bin =
os_get_config_path_ptr("obs-studio/plugins/%module%/bin");
BPtr<char> config_data =
os_get_config_path_ptr("obs-studio/plugins/%module%/data");
obs_add_module_path(config_bin, config_data);
#elif ARCH_BITS == 64
obs_add_module_path((path + "/bin/64bit").c_str(),
(path + "/data").c_str());
#else
obs_add_module_path((path + "/bin/32bit").c_str(),
(path + "/data").c_str());
#endif
}
--8<---------------cut here---------------end--------------->8---
So what I am currently have is this:
--8<---------------cut here---------------start------------->8---
λ find ~/.config/obs-studio/plugins
/home/levenson/.config/obs-studio/plugins
/home/levenson/.config/obs-studio/plugins/v4l2sink
/home/levenson/.config/obs-studio/plugins/v4l2sink/bin
/home/levenson/.config/obs-studio/plugins/v4l2sink/bin/64bit
/home/levenson/.config/obs-studio/plugins/v4l2sink/bin/64bit/v4l2sink.so
--8<---------------cut here---------------end--------------->8---
where the library is a symlink
v4l2sink.so -> /home/levenson/.guix-profile/lib/obs-plugins/v4l2sink.so
--
Alexey
Information forwarded
to
guix-patches <at> gnu.org
:
bug#45707
; Package
guix-patches
.
(Wed, 13 Jan 2021 14:36:01 GMT)
Full text and
rfc822 format available.
Message #26 received at 45707 <at> debbugs.gnu.org (full text, mbox):
Hi,
Alexey Abramov <levenson <at> mmer.org> skribis:
> In order to provide plugins for obs via $XDG_CONFIG_HOME (~/.config) you need to provide the following structure:
>
> "$XDG_CONFIG_HOME/obs-studio/plugins/<module-name>/bin/<arch>/"
>
> As it implemented in obs-studio (~/obs-studio/UI/window-basic-main.cpp)
>
> static void AddExtraModulePaths()
> {
> char base_module_dir[512];
> #if defined(_WIN32) || defined(__APPLE__)
> int ret = GetProgramDataPath(base_module_dir, sizeof(base_module_dir),
> "obs-studio/plugins/%module%");
> #else
> int ret = GetConfigPath(base_module_dir, sizeof(base_module_dir),
> "obs-studio/plugins/%module%");
> #endif
What about patching this function so that it honors, say, an
‘OBS_PLUGIN_PATH’ environment variable?
We’d put plugins under $prefix/lib/obs-studio/plugins and define a
‘search-path-specification’ for it.
My 2¢,
Ludo’.
Information forwarded
to
guix-patches <at> gnu.org
:
bug#45707
; Package
guix-patches
.
(Fri, 15 Jan 2021 16:49:01 GMT)
Full text and
rfc822 format available.
Message #29 received at 45707 <at> debbugs.gnu.org (full text, mbox):
[Message part 1 (text/plain, inline)]
Hi,
Thanks Ludo. With this patch, obs will check OBS_PLUGINS_PATH environment variable for a *default* location of modules.
Plugins might also have some data. I am not sure if it is safe to use relative path thought. Would it be better to use OBS_PLUGINS_DATA_PATH var?
--
Alexey
[0001-gnu-obs-Use-an-environment-variable-for-plugins-loca.patch (text/x-patch, attachment)]
Information forwarded
to
guix-patches <at> gnu.org
:
bug#45707
; Package
guix-patches
.
(Fri, 15 Jan 2021 17:11:01 GMT)
Full text and
rfc822 format available.
Message #32 received at 45707 <at> debbugs.gnu.org (full text, mbox):
Hello Evan!
Since OBS version 26.1 without any plugins there is a new button
“Start Virtual Camera” in the bottom right.
Is this v4l2sink plugin really still useful? If I tested corrrectly,
it is not.
Making OBS plugins usable by search path is good though.
Regards,
Florian
Information forwarded
to
guix-patches <at> gnu.org
:
bug#45707
; Package
guix-patches
.
(Fri, 15 Jan 2021 19:26:02 GMT)
Full text and
rfc822 format available.
Message #35 received at 45707 <at> debbugs.gnu.org (full text, mbox):
[Message part 1 (text/plain, inline)]
Hi Florian,
"pelzflorian (Florian Pelz)" <pelzflorian <at> pelzflorian.de> writes:
> Since OBS version 26.1 without any plugins there is a new button
> “Start Virtual Camera” in the bottom right.
I was actually not aware of this, I guess I did not have the required
kernel module installed so it didn't show up for me. With that said, I
suppose that this package is indeed not very useful anymore since this
functionality is built into OBS. Apologies, this was some poor research
on my part. Still, maybe it's good to have started the discussion of how
to package OBS plugins for Guix in the future.
-- Evan
[signature.asc (application/pgp-signature, inline)]
Information forwarded
to
guix-patches <at> gnu.org
:
bug#45707
; Package
guix-patches
.
(Sat, 16 Jan 2021 21:48:01 GMT)
Full text and
rfc822 format available.
Message #38 received at 45707 <at> debbugs.gnu.org (full text, mbox):
Hi,
Alexey Abramov <levenson <at> mmer.org> skribis:
> Thanks Ludo. With this patch, obs will check OBS_PLUGINS_PATH environment variable for a *default* location of modules.
OK.
> Plugins might also have some data. I am not sure if it is safe to use relative path thought. Would it be better to use OBS_PLUGINS_DATA_PATH var?
I don’t know, how does this patch change the situation wrt. finding a
plugin’s data?
> From 9ade5607aab510cc88561efb7c0b08567b5e19fe Mon Sep 17 00:00:00 2001
> From: Alexey Abramov <levenson <at> mmer.org>
> Date: Fri, 15 Jan 2021 17:26:37 +0100
> Subject: [PATCH] gnu: obs: Use an environment variable for plugins location.
>
> * gnu/local.mk (dist_patch_DATA): Add it.
> * gnu/packages/video.scm (obs): Use it.
> * gnu/packages/patches/obs-override-default-modules-location.patch: Patch it.
Some minor issues:
> +++ b/gnu/packages/patches/obs-override-default-modules-location.patch
‘guix lint’ might complain about the file name length. :-)
Perhaps remove ‘override-default-’ from the file name.
> @@ -0,0 +1,32 @@
> +From 96e3c05043eb6b682b105e9080b45efeee33dcd0 Mon Sep 17 00:00:00 2001
> +From: Alexey Abramov <levenson <at> mmer.org>
> +Date: Fri, 15 Jan 2021 17:07:17 +0100
> +Subject: [PATCH] Use an environment variable for a default modules location
> +
> +---
> + libobs/obs-nix.c | 10 ++++++++--
If it comes from another distro, please indicate it clearly at the top
of the file.
> ++ char *obs_plugins_path = getenv("OBS_PLUGINS_PATH");
> ++ if (obs_plugins_path) {
> ++ struct dstr obs_plugins_data_path;
> ++ dstr_init_copy(&obs_plugins_data_path, obs_plugins_path);
> ++ dstr_cat(&obs_plugins_data_path, "/../../share/obs/obs-plugins/%module%");
Shouldn’t it be lib/obs-plugins, as your ‘search-path-specification’
indicates?
> + (native-search-paths
> + (list (search-path-specification
> + (variable "OBS_PLUGINS_PATH")
> + (separator #f) ;single entry
> + (files '("lib/obs-plugins")))))
How about renaming the variable to ‘OBS_PLUGINS_DIRECTORY’ given that
it designates a single directory rather than a search path?
Could you send an updated patch?
Thanks!
Ludo’.
Information forwarded
to
guix-patches <at> gnu.org
:
bug#45707
; Package
guix-patches
.
(Wed, 20 Jan 2021 08:29:02 GMT)
Full text and
rfc822 format available.
Message #41 received at 45707 <at> debbugs.gnu.org (full text, mbox):
[Message part 1 (text/plain, inline)]
Ludovic Courtès <ludo <at> gnu.org> writes:
> Hi,
>
> Alexey Abramov <levenson <at> mmer.org> skribis:
>
>> Thanks Ludo. With this patch, obs will check OBS_PLUGINS_PATH environment variable for a *default* location of modules.
>
> OK.
>
>> Plugins might also have some data. I am not sure if it is safe to
>> use relative path thought. Would it be better to use
>> OBS_PLUGINS_DATA_PATH var?
>
> I don’t know, how does this patch change the situation wrt. finding a
> plugin’s data?
They have locals in their data directory at least. And probably something else, yes.
>
>> From 9ade5607aab510cc88561efb7c0b08567b5e19fe Mon Sep 17 00:00:00 2001
>> From: Alexey Abramov <levenson <at> mmer.org>
>> Date: Fri, 15 Jan 2021 17:26:37 +0100
>> Subject: [PATCH] gnu: obs: Use an environment variable for plugins location.
>>
>> * gnu/local.mk (dist_patch_DATA): Add it.
>> * gnu/packages/video.scm (obs): Use it.
>> * gnu/packages/patches/obs-override-default-modules-location.patch: Patch it.
>
> Some minor issues:
>
>> +++ b/gnu/packages/patches/obs-override-default-modules-location.patch
>
> ‘guix lint’ might complain about the file name length. :-)
> Perhaps remove ‘override-default-’ from the file name.
>
Done. Passed.
>> @@ -0,0 +1,32 @@
>> +From 96e3c05043eb6b682b105e9080b45efeee33dcd0 Mon Sep 17 00:00:00 2001
>> +From: Alexey Abramov <levenson <at> mmer.org>
>> +Date: Fri, 15 Jan 2021 17:07:17 +0100
>> +Subject: [PATCH] Use an environment variable for a default modules location
>> +
>> +---
>> + libobs/obs-nix.c | 10 ++++++++--
>
> If it comes from another distro, please indicate it clearly at the top
> of the file.
>
>> ++ char *obs_plugins_path = getenv("OBS_PLUGINS_PATH");
>> ++ if (obs_plugins_path) {
>> ++ struct dstr obs_plugins_data_path;
>> ++ dstr_init_copy(&obs_plugins_data_path, obs_plugins_path);
>> ++ dstr_cat(&obs_plugins_data_path, "/../../share/obs/obs-plugins/%module%");
>
> Shouldn’t it be lib/obs-plugins, as your ‘search-path-specification’
> indicates?
That is a data directory. With the new patch it will be taken from
OBS_PLUGINS_DATA_DIRECTORY. %module% part is required for data.
>
>> + (native-search-paths
>> + (list (search-path-specification
>> + (variable "OBS_PLUGINS_PATH")
>> + (separator #f) ;single entry
>> + (files '("lib/obs-plugins")))))
>
> How about renaming the variable to ‘OBS_PLUGINS_DIRECTORY’ given that
> it designates a single directory rather than a search path?
Done.
>
> Could you send an updated patch?
Attached. I also tested the obs patch with this [1] and this [2] and
haven't seen any erros. obs loads plugins successfully.
--8<---------------cut here---------------start------------->8---
info: ---------------------------------
info: Loaded Modules:
info: v4l2sink.so <------ this
info: text-freetype2.so
info: spectralizer.so <------ this
info: rtmp-services.so
info: obs-x264.so
info: obs-transitions.so
info: obs-outputs.so
info: obs-filters.so
info: obs-ffmpeg.so
info: linux-v4l2.so
info: linux-pulseaudio.so
info: linux-jack.so
info: linux-decklink.so
info: linux-capture.so
info: linux-alsa.so
info: libwlrobs.so <------ this
info: image-source.so
info: frontend-tools.so
info: decklink-ouput-ui.so
info: decklink-captions.so
info: ---------------------------------
~ master* ⇡
λ find /gnu/store/3s84wmh9pqcryfzx0rrri8sqff7fqlqn-obs-wlrobs-1.0/
/gnu/store/3s84wmh9pqcryfzx0rrri8sqff7fqlqn-obs-wlrobs-1.0/
/gnu/store/3s84wmh9pqcryfzx0rrri8sqff7fqlqn-obs-wlrobs-1.0/lib
/gnu/store/3s84wmh9pqcryfzx0rrri8sqff7fqlqn-obs-wlrobs-1.0/lib/obs-plugins
/gnu/store/3s84wmh9pqcryfzx0rrri8sqff7fqlqn-obs-wlrobs-1.0/lib/obs-plugins/libwlrobs.so
~ master* ⇡
λ find /gnu/store/rmz58nv5qcwr1pm38nx7aicyfq4jbp4a-obs-spectralizer-1.3.3/
/gnu/store/rmz58nv5qcwr1pm38nx7aicyfq4jbp4a-obs-spectralizer-1.3.3/
/gnu/store/rmz58nv5qcwr1pm38nx7aicyfq4jbp4a-obs-spectralizer-1.3.3/share
/gnu/store/rmz58nv5qcwr1pm38nx7aicyfq4jbp4a-obs-spectralizer-1.3.3/share/doc
/gnu/store/rmz58nv5qcwr1pm38nx7aicyfq4jbp4a-obs-spectralizer-1.3.3/share/doc/obs-spectralizer-1.3.3
/gnu/store/rmz58nv5qcwr1pm38nx7aicyfq4jbp4a-obs-spectralizer-1.3.3/share/doc/obs-spectralizer-1.3.3/COPYING.txt
/gnu/store/rmz58nv5qcwr1pm38nx7aicyfq4jbp4a-obs-spectralizer-1.3.3/share/obs
/gnu/store/rmz58nv5qcwr1pm38nx7aicyfq4jbp4a-obs-spectralizer-1.3.3/share/obs/obs-plugins
/gnu/store/rmz58nv5qcwr1pm38nx7aicyfq4jbp4a-obs-spectralizer-1.3.3/share/obs/obs-plugins/spectralizer
/gnu/store/rmz58nv5qcwr1pm38nx7aicyfq4jbp4a-obs-spectralizer-1.3.3/share/obs/obs-plugins/spectralizer/locale
/gnu/store/rmz58nv5qcwr1pm38nx7aicyfq4jbp4a-obs-spectralizer-1.3.3/share/obs/obs-plugins/spectralizer/locale/ko-KR.ini
/gnu/store/rmz58nv5qcwr1pm38nx7aicyfq4jbp4a-obs-spectralizer-1.3.3/share/obs/obs-plugins/spectralizer/locale/ru-RU.ini
/gnu/store/rmz58nv5qcwr1pm38nx7aicyfq4jbp4a-obs-spectralizer-1.3.3/share/obs/obs-plugins/spectralizer/locale/en-US.ini
/gnu/store/rmz58nv5qcwr1pm38nx7aicyfq4jbp4a-obs-spectralizer-1.3.3/lib
/gnu/store/rmz58nv5qcwr1pm38nx7aicyfq4jbp4a-obs-spectralizer-1.3.3/lib/obs-plugins
/gnu/store/rmz58nv5qcwr1pm38nx7aicyfq4jbp4a-obs-spectralizer-1.3.3/lib/obs-plugins/spectralizer.so
--8<---------------cut here---------------end--------------->8---
Footnotes:
[1] https://issues.guix.gnu.org/45960
[2] https://issues.guix.gnu.org/45961
--
Alexey
[0001-gnu-obs-Use-an-environment-variable-for-plugins-loca.patch (text/x-patch, attachment)]
Reply sent
to
Ludovic Courtès <ludo <at> gnu.org>
:
You have taken responsibility.
(Tue, 02 Feb 2021 09:32:02 GMT)
Full text and
rfc822 format available.
Notification sent
to
Evan Straw <evan.straw99 <at> gmail.com>
:
bug acknowledged by developer.
(Tue, 02 Feb 2021 09:32:02 GMT)
Full text and
rfc822 format available.
Message #46 received at 45707-done <at> debbugs.gnu.org (full text, mbox):
Hi Alexey,
Alexey Abramov <levenson <at> mmer.org> skribis:
>>From f67bf04b57f45f0ca11cb6b902d22f7c4d44e005 Mon Sep 17 00:00:00 2001
> From: Alexey Abramov <levenson <at> mmer.org>
> Date: Fri, 15 Jan 2021 17:26:37 +0100
> Subject: [PATCH] gnu: obs: Use an environment variable for plugins location.
>
> * gnu/local.mk (dist_patch_DATA): Add it.
> * gnu/packages/video.scm (obs): Use it.
> * gnu/packages/patches/obs-modules-location.patch: Patch it.
Applied. Thanks, and apologies for the delay!
Ludo’.
Information forwarded
to
guix-patches <at> gnu.org
:
bug#45707
; Package
guix-patches
.
(Tue, 02 Feb 2021 20:01:01 GMT)
Full text and
rfc822 format available.
Message #49 received at 45707 <at> debbugs.gnu.org (full text, mbox):
Hi everyone!
I have been doing the similar work related to obs load path and already
contibuted it to upstream [fn:1]. The patch was accepted and obs
currently support OBS_PLUGINS_PATH and OBS_PLUGINS_DATA_PATH out of the
box.
I've sent a patch with update to OBS version [fn:2] to get those changes
available in guix. Unluckily I was not aware of this thread before
Alexey replied to [fn:2] and also forgot to ask not to apply this patch.
As the changes [fn:1] is already in upstream and do the similar thing,
can we revert this patch and apply [fn:2] instead, please?
Sorry for missing this thread, forgetting to notify you earlier and all
the inconvenience.
* Footnotes
[fn:1] https://github.com/obsproject/obs-studio/pull/4067
[fn:2] http://issues.guix.gnu.org/46113
--
Best regards,
Andrew Tropin
Information forwarded
to
guix-patches <at> gnu.org
:
bug#45707
; Package
guix-patches
.
(Wed, 03 Feb 2021 14:41:02 GMT)
Full text and
rfc822 format available.
Message #52 received at 45707-done <at> debbugs.gnu.org (full text, mbox):
> Should we wait for the next OBS release instead of packaging an
> arbitrary commit?
I'm not in a hurry, but as I explained earlier in this thread [fn:2] it seems
relatively safe for me to use this commit and getting back later to
26.1.3 or 26.2.0 or whatever next release will be.
> I am not insist on keeping [1], but I do think that is more cleaner
> solution.
In terms of implememntation I like that [fn:1] prevents double loading
of plugins, by excluding obs installation dir from "load-path". However
double loading of the same plugin doesn't seem to break anything. Also,
OBS_PLUGINS_DIRECTORY variable name maybe a little better than
OBS_PLUGINS_PATH as it contains only one path.
The problem is that now there are two almost identical mechanisms (one
in upstream and one via patch [fn:1]), which can bring some maintanance
problems in the future.
There are two good option in my opinion:
- contribute patch from [fn:1] to upstream (reverting OBS_PLUGINS_PATH)
- revert [fn:1] and use OBS_PLUGINS_PATH from upstream
If Alexey ready to contibute OBS_PLUGINS_DIRECTORY patch to obs
(reverting OBS_PLUGINS_PATH), I would be glad to support it. Otherwise,
I would prefer to revert [fn:1] and apply this one. To prevent
maintanance problems in the future.
* Footnotes
[fn:2] http://issues.guix.gnu.org/46113
[fn:1] https://issues.guix.gnu.org/45707
--
Best regards,
Andrew Tropin
Information forwarded
to
guix-patches <at> gnu.org
:
bug#45707
; Package
guix-patches
.
(Wed, 03 Feb 2021 21:12:02 GMT)
Full text and
rfc822 format available.
Message #55 received at 45707-done <at> debbugs.gnu.org (full text, mbox):
Hi!
Andrew Tropin <andrew <at> trop.in> skribis:
> The problem is that now there are two almost identical mechanisms (one
> in upstream and one via patch [fn:1]), which can bring some maintanance
> problems in the future.
>
> There are two good option in my opinion:
> - contribute patch from [fn:1] to upstream (reverting OBS_PLUGINS_PATH)
> - revert [fn:1] and use OBS_PLUGINS_PATH from upstream
>
> If Alexey ready to contibute OBS_PLUGINS_DIRECTORY patch to obs
> (reverting OBS_PLUGINS_PATH), I would be glad to support it. Otherwise,
> I would prefer to revert [fn:1] and apply this one. To prevent
> maintanance problems in the future.
I think it’s nicer to follow upstream in general, but I also don’t mind
using our OBS_PLUGINS_DIRECTORY patch until we upgrade to the next
release.
So, Andrew and Alexey: lemme know and I’ll apply what you consider best!
Ludo’.
Information forwarded
to
guix-patches <at> gnu.org
:
bug#45707
; Package
guix-patches
.
(Thu, 04 Feb 2021 10:48:02 GMT)
Full text and
rfc822 format available.
Message #58 received at 45707-done <at> debbugs.gnu.org (full text, mbox):
Hi Andrew,
Andrew Tropin <andrew <at> trop.in> writes:
>> Should we wait for the next OBS release instead of packaging an
>> arbitrary commit?
>
> I'm not in a hurry, but as I explained earlier in this thread [fn:2] it seems
> relatively safe for me to use this commit and getting back later to
> 26.1.3 or 26.2.0 or whatever next release will be.
>
>> I am not insist on keeping [1], but I do think that is more cleaner
>> solution.
>
> In terms of implememntation I like that [fn:1] prevents double loading
> of plugins, by excluding obs installation dir from "load-path". However
> double loading of the same plugin doesn't seem to break anything. Also,
> OBS_PLUGINS_DIRECTORY variable name maybe a little better than
> OBS_PLUGINS_PATH as it contains only one path.
>
> The problem is that now there are two almost identical mechanisms (one
> in upstream and one via patch [fn:1]), which can bring some maintanance
> problems in the future.
>
> There are two good option in my opinion:
> - contribute patch from [fn:1] to upstream (reverting OBS_PLUGINS_PATH)
I would go with this one. But the thing is that [fn:1] is specific and
makes sense only for guix. At least from my point of view. So I doubt
that the upstream accept it. But you can try for sure.
> - revert [fn:1] and use OBS_PLUGINS_PATH from upstream
>
> If Alexey ready to contibute OBS_PLUGINS_DIRECTORY patch to obs
> (reverting OBS_PLUGINS_PATH), I would be glad to support it. Otherwise,
> I would prefer to revert [fn:1] and apply this one. To prevent
> maintanance problems in the future.
As I am not a regular obs user, I am afraid I won't be able to find time
for this soon. Feel free to use/delete/revert [fn:1]. It is your call.
> * Footnotes
>
> [fn:2] http://issues.guix.gnu.org/46113
>
> [fn:1] https://issues.guix.gnu.org/45707
>
> --
> Best regards,
> Andrew Tropin
--
Alexey
Information forwarded
to
guix-patches <at> gnu.org
:
bug#45707
; Package
guix-patches
.
(Thu, 04 Feb 2021 12:50:02 GMT)
Full text and
rfc822 format available.
Message #61 received at 45707-done <at> debbugs.gnu.org (full text, mbox):
> > There are two good option in my opinion:
> > - contribute patch from [fn:1] to upstream (reverting OBS_PLUGINS_PATH)
> I would go with this one. But the thing is that [fn:1] is specific and
> makes sense only for guix. At least from my point of view. So I doubt
> that the upstream accept it. But you can try for sure.
> As I am not a regular obs user, I am afraid I won't be able to find
> time for this soon. Feel free to use/delete/revert [fn:1]. It is your
> call.
Thank you for the response!
Probably I won't find time to contribute a patch from [fn:1] to obs any
time soon too. So I will wait for next obs release and will update
package definition to use upstream solution once a new version is
released.
Thank you for your work, sorry for inconvenience)
--
Best regards,
Andrew Tropin
bug archived.
Request was from
Debbugs Internal Request <help-debbugs <at> gnu.org>
to
internal_control <at> debbugs.gnu.org
.
(Fri, 05 Mar 2021 12:24:04 GMT)
Full text and
rfc822 format available.
This bug report was last modified 3 years and 50 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.