* [bug#53582] [PATCH 0/2] Update obs to 27.1.3 and use new package style. @ 2022-01-27 12:10 Andrew Tropin 2022-01-28 20:29 ` Leo Famulari 0 siblings, 1 reply; 4+ messages in thread From: Andrew Tropin @ 2022-01-27 12:10 UTC (permalink / raw) To: 53582 [-- Warning: decoded text below may be mangled, UTF-8 assumed --] [-- Attachment #1.1: 0001-gnu-obs-Update-to-27.1.3.patch --] [-- Type: text/x-patch, Size: 1190 bytes --] From c985af51a431656ad5e9389b9f8f132e7ac193b7 Mon Sep 17 00:00:00 2001 From: Andrew Tropin <andrew@trop.in> Date: Thu, 27 Jan 2022 13:09:07 +0300 Subject: [PATCH 1/2] gnu: obs: Update to 27.1.3. * gnu/packages/video.scm (obs): Update to 27.1.3. --- gnu/packages/video.scm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm index 1b26418eea..3ab34812f2 100644 --- a/gnu/packages/video.scm +++ b/gnu/packages/video.scm @@ -3322,7 +3322,7 @@ (define-public v4l-utils (define-public obs (package (name "obs") - (version "27.0.1") + (version "27.1.3") (source (origin (method git-fetch) (uri (git-reference @@ -3332,7 +3332,7 @@ (define-public obs (file-name (git-file-name name version)) (sha256 (base32 - "04fzsr9yizmxy0r7z2706crvnsnybpnv5kgfn77znknxxjacfhkn")) + "1ndiarr3d6qihymaigf34jjml0lrgbj640fnpnffz2ysj7276q0j")) (patches (search-patches "obs-modules-location.patch")))) (build-system cmake-build-system) -- 2.34.0 [-- Warning: decoded text below may be mangled, UTF-8 assumed --] [-- Attachment #1.2: 0002-gnu-obs-Use-new-package-style.patch --] [-- Type: text/x-patch, Size: 2345 bytes --] From eb09c0cf30606564e3794b7a0afe409e0b48a9a1 Mon Sep 17 00:00:00 2001 From: Andrew Tropin <andrew@trop.in> Date: Thu, 27 Jan 2022 14:54:22 +0300 Subject: [PATCH 2/2] gnu: obs: Use new package style. * gnu/packages/video.scm (obs): Use new package style. --- gnu/packages/video.scm | 28 ++++++++++++++-------------- 1 file changed, 14 insertions(+), 14 deletions(-) diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm index 3ab34812f2..932fb9165d 100644 --- a/gnu/packages/video.scm +++ b/gnu/packages/video.scm @@ -3337,21 +3337,21 @@ (define-public obs (search-patches "obs-modules-location.patch")))) (build-system cmake-build-system) (arguments - `(#:configure-flags - (list (string-append "-DOBS_VERSION_OVERRIDE=" ,version) - "-DENABLE_UNIT_TESTS=TRUE" - ;; Browser plugin requires cef, but it is not packaged yet. - ;; <https://bitbucket.org/chromiumembedded/cef/src/master/> - "-DBUILD_BROWSER=FALSE") + (list + #:configure-flags + #~(list (string-append "-DOBS_VERSION_OVERRIDE=" #$version) + "-DENABLE_UNIT_TESTS=TRUE" + ;; Browser plugin requires cef, but it is not packaged yet. + ;; <https://bitbucket.org/chromiumembedded/cef/src/master/> + "-DBUILD_BROWSER=FALSE") #:phases - (modify-phases %standard-phases - (add-after 'install 'wrap-executable - (lambda* (#:key outputs #:allow-other-keys) - (let ((out (assoc-ref outputs "out")) - (plugin-path (getenv "QT_PLUGIN_PATH"))) - (wrap-program (string-append out "/bin/obs") - `("QT_PLUGIN_PATH" ":" prefix (,plugin-path)))) - #t))))) + #~(modify-phases %standard-phases + (add-after 'install 'wrap-executable + (lambda* (#:key outputs #:allow-other-keys) + (let ((out (assoc-ref outputs "out")) + (plugin-path (getenv "QT_PLUGIN_PATH"))) + (wrap-program (string-append out "/bin/obs") + `("QT_PLUGIN_PATH" ":" prefix (,plugin-path))))))))) (native-search-paths (list (search-path-specification (variable "OBS_PLUGINS_DIRECTORY") -- 2.34.0 [-- Attachment #1.3: Type: text/plain, Size: 223 bytes --] Andrew Tropin (2): gnu: obs: Update to 27.1.3. gnu: obs: Use new package style. gnu/packages/video.scm | 32 ++++++++++++++++---------------- 1 file changed, 16 insertions(+), 16 deletions(-) -- 2.34.0 [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 853 bytes --] ^ permalink raw reply related [flat|nested] 4+ messages in thread
* [bug#53582] [PATCH 0/2] Update obs to 27.1.3 and use new package style. 2022-01-27 12:10 [bug#53582] [PATCH 0/2] Update obs to 27.1.3 and use new package style Andrew Tropin @ 2022-01-28 20:29 ` Leo Famulari 2022-01-31 16:48 ` Andrew Tropin 0 siblings, 1 reply; 4+ messages in thread From: Leo Famulari @ 2022-01-28 20:29 UTC (permalink / raw) To: Andrew Tropin; +Cc: 53582 On Thu, Jan 27, 2022 at 03:10:16PM +0300, Andrew Tropin wrote: > From c985af51a431656ad5e9389b9f8f132e7ac193b7 Mon Sep 17 00:00:00 2001 > From: Andrew Tropin <andrew@trop.in> > Date: Thu, 27 Jan 2022 13:09:07 +0300 > Subject: [PATCH 1/2] gnu: obs: Update to 27.1.3. > > * gnu/packages/video.scm (obs): Update to 27.1.3. Thanks! I pushed this patch as 10d1125f71205f094a6030c51c498b0f8dab5f73 > From eb09c0cf30606564e3794b7a0afe409e0b48a9a1 Mon Sep 17 00:00:00 2001 > From: Andrew Tropin <andrew@trop.in> > Date: Thu, 27 Jan 2022 14:54:22 +0300 > Subject: [PATCH 2/2] gnu: obs: Use new package style. > > * gnu/packages/video.scm (obs): Use new package style. I did not push this patch, because it only changes some elements to the new style. It still uses the old style of inputs (with labels) and it also continues to use labels in the arguments, for example for "out". Instead of (assoc-ref outputs "out"), you should use the gexp #$output. Can you send a revised patch to address those issues? ^ permalink raw reply [flat|nested] 4+ messages in thread
* [bug#53582] [PATCH 0/2] Update obs to 27.1.3 and use new package style. 2022-01-28 20:29 ` Leo Famulari @ 2022-01-31 16:48 ` Andrew Tropin 2022-01-31 19:03 ` bug#53582: " Leo Famulari 0 siblings, 1 reply; 4+ messages in thread From: Andrew Tropin @ 2022-01-31 16:48 UTC (permalink / raw) To: Leo Famulari; +Cc: 53582 [-- Attachment #1.1: Type: text/plain, Size: 1189 bytes --] On 2022-01-28 15:29, Leo Famulari wrote: > On Thu, Jan 27, 2022 at 03:10:16PM +0300, Andrew Tropin wrote: >> From c985af51a431656ad5e9389b9f8f132e7ac193b7 Mon Sep 17 00:00:00 2001 >> From: Andrew Tropin <andrew@trop.in> >> Date: Thu, 27 Jan 2022 13:09:07 +0300 >> Subject: [PATCH 1/2] gnu: obs: Update to 27.1.3. >> >> * gnu/packages/video.scm (obs): Update to 27.1.3. > > Thanks! I pushed this patch as 10d1125f71205f094a6030c51c498b0f8dab5f73 > >> From eb09c0cf30606564e3794b7a0afe409e0b48a9a1 Mon Sep 17 00:00:00 2001 >> From: Andrew Tropin <andrew@trop.in> >> Date: Thu, 27 Jan 2022 14:54:22 +0300 >> Subject: [PATCH 2/2] gnu: obs: Use new package style. >> >> * gnu/packages/video.scm (obs): Use new package style. > > I did not push this patch, because it only changes some elements to the > new style. > > It still uses the old style of inputs (with labels) and it also > continues to use labels in the arguments, for example for "out". > > Instead of (assoc-ref outputs "out"), you should use the gexp #$output. > > Can you send a revised patch to address those issues? Hi, thank you for merging, attaching updated version of the second patch: [-- Warning: decoded text below may be mangled, UTF-8 assumed --] [-- Attachment #1.2: v2-0001-gnu-obs-Use-new-package-style.patch --] [-- Type: text/x-patch, Size: 3779 bytes --] From ce003ac1cf2561eff6cba80985afe532bacd836d Mon Sep 17 00:00:00 2001 From: Andrew Tropin <andrew@trop.in> Date: Thu, 27 Jan 2022 14:54:22 +0300 Subject: [PATCH v2] gnu: obs: Use new package style. * gnu/packages/video.scm (obs): Use new package style. --- gnu/packages/video.scm | 76 +++++++++++++++++++++--------------------- 1 file changed, 38 insertions(+), 38 deletions(-) diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm index 94562d52ae..de9fb42904 100644 --- a/gnu/packages/video.scm +++ b/gnu/packages/video.scm @@ -3360,21 +3360,20 @@ (define-public obs (search-patches "obs-modules-location.patch")))) (build-system cmake-build-system) (arguments - `(#:configure-flags - (list (string-append "-DOBS_VERSION_OVERRIDE=" ,version) - "-DENABLE_UNIT_TESTS=TRUE" - ;; Browser plugin requires cef, but it is not packaged yet. - ;; <https://bitbucket.org/chromiumembedded/cef/src/master/> - "-DBUILD_BROWSER=FALSE") + (list + #:configure-flags + #~(list (string-append "-DOBS_VERSION_OVERRIDE=" #$version) + "-DENABLE_UNIT_TESTS=TRUE" + ;; Browser plugin requires cef, but it is not packaged yet. + ;; <https://bitbucket.org/chromiumembedded/cef/src/master/> + "-DBUILD_BROWSER=FALSE") #:phases - (modify-phases %standard-phases - (add-after 'install 'wrap-executable - (lambda* (#:key outputs #:allow-other-keys) - (let ((out (assoc-ref outputs "out")) - (plugin-path (getenv "QT_PLUGIN_PATH"))) - (wrap-program (string-append out "/bin/obs") - `("QT_PLUGIN_PATH" ":" prefix (,plugin-path)))) - #t))))) + #~(modify-phases %standard-phases + (add-after 'install 'wrap-executable + (lambda* _ + (let ((plugin-path (getenv "QT_PLUGIN_PATH"))) + (wrap-program (string-append #$output "/bin/obs") + `("QT_PLUGIN_PATH" ":" prefix (,plugin-path))))))))) (native-search-paths (list (search-path-specification (variable "OBS_PLUGINS_DIRECTORY") @@ -3387,30 +3386,31 @@ (define-public obs (native-inputs (list cmocka pkg-config)) (inputs - `(("alsa-lib" ,alsa-lib) - ("curl" ,curl) - ("eudev" ,eudev) - ("ffmpeg" ,ffmpeg) - ("fontconfig" ,fontconfig) - ("freetype" ,freetype) - ("glib" ,glib) - ("jack" ,jack-1) - ("jansson" ,jansson) - ("libx264" ,libx264) - ("libxcomposite" ,libxcomposite) - ("mbedtls" ,mbedtls-apache) - ("mesa" ,mesa) - ("pipewire" ,pipewire-0.3) - ("pulseaudio" ,pulseaudio) - ("qtbase" ,qtbase-5) - ("qtsvg" ,qtsvg) - ("qtx11extras" ,qtx11extras) - ("qtwayland" ,qtwayland) - ("speexdsp" ,speexdsp) - ("v4l-utils" ,v4l-utils) - ("wayland" ,wayland) - ("wayland-protocols" ,wayland-protocols) - ("zlib" ,zlib))) + (list + alsa-lib + curl + eudev + ffmpeg + fontconfig + freetype + glib + jack-1 + jansson + libx264 + libxcomposite + mbedtls-apache + mesa + pipewire-0.3 + pulseaudio + qtbase-5 + qtsvg + qtx11extras + qtwayland + speexdsp + v4l-utils + wayland + wayland-protocols + zlib)) (synopsis "Live streaming software") (description "Open Broadcaster Software provides a graphical interface for video recording and live streaming. OBS supports capturing audio and video -- 2.34.0 [-- Attachment #1.3: Type: text/plain, Size: 39 bytes --] -- Best regards, Andrew Tropin [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 853 bytes --] ^ permalink raw reply related [flat|nested] 4+ messages in thread
* bug#53582: [PATCH 0/2] Update obs to 27.1.3 and use new package style. 2022-01-31 16:48 ` Andrew Tropin @ 2022-01-31 19:03 ` Leo Famulari 0 siblings, 0 replies; 4+ messages in thread From: Leo Famulari @ 2022-01-31 19:03 UTC (permalink / raw) To: Andrew Tropin; +Cc: 53582-done [-- Attachment #1: Type: text/plain, Size: 485 bytes --] On Mon, Jan 31, 2022 at 07:48:50PM +0300, Andrew Tropin wrote: > Hi, thank you for merging, attaching updated version of the second > patch: > > From ce003ac1cf2561eff6cba80985afe532bacd836d Mon Sep 17 00:00:00 2001 > From: Andrew Tropin <andrew@trop.in> > Date: Thu, 27 Jan 2022 14:54:22 +0300 > Subject: [PATCH v2] gnu: obs: Use new package style. > > * gnu/packages/video.scm (obs): Use new package style. Thanks! Pushed as b3a2c79fdc4ec3794639808c1161870341f175a6 [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2022-01-31 19:18 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2022-01-27 12:10 [bug#53582] [PATCH 0/2] Update obs to 27.1.3 and use new package style Andrew Tropin 2022-01-28 20:29 ` Leo Famulari 2022-01-31 16:48 ` Andrew Tropin 2022-01-31 19:03 ` bug#53582: " Leo Famulari
Code repositories for project(s) associated with this public inbox https://git.savannah.gnu.org/cgit/guix.git This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).