Package: guix-patches;
Reported by: aurtzy <aurtzy <at> gmail.com>
Date: Thu, 3 Aug 2023 06:55:02 UTC
Severity: normal
Tags: moreinfo, patch
Done: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
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 65031 in the body.
You can then email your comments to 65031 AT debbugs.gnu.org in the normal way.
Toggle the display of automated, internal messages from the tracker.
View this report as an mbox folder, status mbox, maintainer mbox
guix-patches <at> gnu.org
:bug#65031
; Package guix-patches
.
(Thu, 03 Aug 2023 06:55:02 GMT) Full text and rfc822 format available.aurtzy <aurtzy <at> gmail.com>
:guix-patches <at> gnu.org
.
(Thu, 03 Aug 2023 06:55:02 GMT) Full text and rfc822 format available.Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):
From: aurtzy <aurtzy <at> gmail.com> To: guix-patches <at> gnu.org Cc: aurtzy <aurtzy <at> gmail.com> Subject: [PATCH] gnu: Add syncplay. Date: Thu, 3 Aug 2023 00:35:28 -0400
* gnu/packages/networking.scm (syncplay): New variable. --- gnu/packages/networking.scm | 52 +++++++++++++++++++++++++++++++++++++ 1 file changed, 52 insertions(+) diff --git a/gnu/packages/networking.scm b/gnu/packages/networking.scm index 488d015107..10eb19e980 100644 --- a/gnu/packages/networking.scm +++ b/gnu/packages/networking.scm @@ -60,6 +60,7 @@ ;;; Copyright © 2023 Bruno Victal <mirai <at> makinata.eu> ;;; Copyright © 2023 Yovan Naumovski <yovan <at> gorski.stream> ;;; Copyright © 2023 Zheng Junjie <873216071 <at> qq.com> +;;; Copyright © 2023 aurtzy <aurtzy <at> gmail.com> ;;; ;;; This file is part of GNU Guix. ;;; @@ -4644,3 +4645,54 @@ (define-public dropwatch recording packets that are dropped by the kernel. It provides the commands @command{dropwatch} and @command{dwdump}.") (license license:gpl2+))) + +(define-public syncplay + (package + (name "syncplay") + (version "1.7.0") + (source (origin + (method git-fetch) + (uri (git-reference + (url "https://github.com/Syncplay/syncplay.git") + (commit (string-append "v" version)))) + (file-name (git-file-name name version)) + (sha256 + (base32 + "061kpnb48lad8rr8v58xac33mwpbrixfbhn7d0xa63zpxg43bvsd")))) + (build-system gnu-build-system) + (arguments + (list #:imported-modules `(,@%gnu-build-system-modules (guix build + qt-utils) + (guix build utils)) + #:modules '((guix build gnu-build-system) + (guix build qt-utils) + (guix build utils)) + #:make-flags #~`("DESTDIR=" ,(string-append "PREFIX=" + (assoc-ref %outputs + "out"))) + #:phases #~(modify-phases %standard-phases + (delete 'configure) + (delete 'build) + (delete 'check) + (add-after 'install 'wrap-qt + (lambda* (#:key inputs #:allow-other-keys) + (wrap-qt-program "syncplay" + #:output #$output + #:inputs inputs)))))) + (inputs (list bash-minimal qtwayland-5)) + (propagated-inputs (list python + python-service-identity + python-twisted + python-pyside-2 + python-certifi + python-idna)) + (home-page "https://syncplay.pl") + (synopsis "Client/server to synchronize media playback on many computers") + (description + "Syncplay is a solution to synchronize video playback across multiple +instances of media players over the Internet. When one person pauses/unpauses +playback or skips to a position in the video, this is replicated across all +media players connected to the same server and in the same \"room\" (viewing +session). A built-in text chat for discussing the synced media is also +included for convenience.") + (license license:asl2.0))) -- 2.41.0
guix-patches <at> gnu.org
:bug#65031
; Package guix-patches
.
(Tue, 05 Sep 2023 14:55:02 GMT) Full text and rfc822 format available.Message #8 received at 65031 <at> debbugs.gnu.org (full text, mbox):
From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com> To: aurtzy <aurtzy <at> gmail.com> Cc: 65031 <at> debbugs.gnu.org Subject: Re: bug#65031: [PATCH] gnu: Add syncplay. Date: Tue, 05 Sep 2023 10:54:43 -0400
Hi, aurtzy <aurtzy <at> gmail.com> writes: > * gnu/packages/networking.scm (syncplay): New variable. > --- > gnu/packages/networking.scm | 52 +++++++++++++++++++++++++++++++++++++ > 1 file changed, 52 insertions(+) > > diff --git a/gnu/packages/networking.scm b/gnu/packages/networking.scm > index 488d015107..10eb19e980 100644 > --- a/gnu/packages/networking.scm > +++ b/gnu/packages/networking.scm > @@ -60,6 +60,7 @@ > ;;; Copyright © 2023 Bruno Victal <mirai <at> makinata.eu> > ;;; Copyright © 2023 Yovan Naumovski <yovan <at> gorski.stream> > ;;; Copyright © 2023 Zheng Junjie <873216071 <at> qq.com> > +;;; Copyright © 2023 aurtzy <aurtzy <at> gmail.com> > ;;; > ;;; This file is part of GNU Guix. > ;;; > @@ -4644,3 +4645,54 @@ (define-public dropwatch > recording packets that are dropped by the kernel. It provides the commands > @command{dropwatch} and @command{dwdump}.") > (license license:gpl2+))) > + > +(define-public syncplay > + (package > + (name "syncplay") > + (version "1.7.0") > + (source (origin > + (method git-fetch) > + (uri (git-reference > + (url "https://github.com/Syncplay/syncplay.git") > + (commit (string-append "v" version)))) > + (file-name (git-file-name name version)) > + (sha256 > + (base32 > + "061kpnb48lad8rr8v58xac33mwpbrixfbhn7d0xa63zpxg43bvsd")))) > + (build-system gnu-build-system) > + (arguments > + (list #:imported-modules `(,@%gnu-build-system-modules (guix build > + qt-utils) > + (guix build utils)) > + #:modules '((guix build gnu-build-system) > + (guix build qt-utils) > + (guix build utils)) > + #:make-flags #~`("DESTDIR=" ,(string-append "PREFIX=" > + (assoc-ref %outputs > + "out"))) > + #:phases #~(modify-phases %standard-phases > + (delete 'configure) > + (delete 'build) > + (delete 'check) > + (add-after 'install 'wrap-qt > + (lambda* (#:key inputs #:allow-other-keys) > + (wrap-qt-program "syncplay" > + #:output #$output > + #:inputs inputs)))))) > + (inputs (list bash-minimal qtwayland-5)) Is it compatible with Qt 6? To try it, replace qtwayland-5 with qtwayland, and specify a #:qtbase qtbase argument. > + (propagated-inputs (list python > + python-service-identity > + python-twisted > + python-pyside-2 > + python-certifi > + python-idna)) These should be inputs, not propagated-inputs, unless it can be used as a library? Also, please sort the inputs alphabetically. > + (home-page "https://syncplay.pl") > + (synopsis "Client/server to synchronize media playback on many computers") > + (description > + "Syncplay is a solution to synchronize video playback across multiple > +instances of media players over the Internet. When one person pauses/unpauses > +playback or skips to a position in the video, this is replicated across all > +media players connected to the same server and in the same \"room\" (viewing > +session). A built-in text chat for discussing the synced media is also > +included for convenience.") > + (license license:asl2.0))) The rest LGTM. Could you please send a v2? -- Thanks, Maxim
Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
to control <at> debbugs.gnu.org
.
(Tue, 05 Sep 2023 14:55:02 GMT) Full text and rfc822 format available.guix-patches <at> gnu.org
:bug#65031
; Package guix-patches
.
(Thu, 21 Sep 2023 22:53:01 GMT) Full text and rfc822 format available.Message #13 received at 65031 <at> debbugs.gnu.org (full text, mbox):
From: aurtzy <aurtzy <at> gmail.com> To: 65031 <at> debbugs.gnu.org Cc: aurtzy <aurtzy <at> gmail.com> Subject: [PATCH v2] gnu: Add syncplay. Date: Thu, 21 Sep 2023 15:37:06 -0400
* gnu/packages/networking.scm (syncplay): New variable. * gnu/packages/patches/syncplay-fix-typeerror.patch: New file. --- > Is it compatible with Qt 6? To try it, replace qtwayland-5 with > qtwayland, and specify a #:qtbase qtbase argument. Looks like it is! #:qtbase doesn't seem to be supported by gnu-build-system or python-build-system (switched to the latter for this v2; elaborated further below) and I had to make a few other adjustments to get it working, so I may need advice if I've missed anything here. A patch has been added to fix a Qt 6 issue causing Syncplay to crash. > These should be inputs, not propagated-inputs, unless it can be used as > a library? Also, please sort the inputs alphabetically. Fixed; not a library. It appears I was using an inappropriate build system for this package - I originally made them propagated because gnu-build-system wasn't doing whatever wrapping python-build-system does to make the inputs available, so this v2 uses python-build-system instead now. Best, aurtzy gnu/packages/networking.scm | 54 +++++++++++++++++++ .../patches/syncplay-fix-typeerror.patch | 25 +++++++++ 2 files changed, 79 insertions(+) create mode 100644 gnu/packages/patches/syncplay-fix-typeerror.patch diff --git a/gnu/packages/networking.scm b/gnu/packages/networking.scm index c1cab1fe0d..a7fde61ea8 100644 --- a/gnu/packages/networking.scm +++ b/gnu/packages/networking.scm @@ -60,6 +60,7 @@ ;;; Copyright © 2023 Bruno Victal <mirai <at> makinata.eu> ;;; Copyright © 2023 Yovan Naumovski <yovan <at> gorski.stream> ;;; Copyright © 2023 Zheng Junjie <873216071 <at> qq.com> +;;; Copyright © 2023 aurtzy <aurtzy <at> gmail.com> ;;; ;;; This file is part of GNU Guix. ;;; @@ -4700,3 +4701,56 @@ (define-public dropwatch recording packets that are dropped by the kernel. It provides the commands @command{dropwatch} and @command{dwdump}.") (license license:gpl2+))) + +(define-public syncplay + (package + (name "syncplay") + (version "1.7.0") + (source (origin + (method git-fetch) + (uri (git-reference + (url "https://github.com/Syncplay/syncplay.git") + (commit (string-append "v" version)))) + (file-name (git-file-name name version)) + (sha256 + (base32 + "061kpnb48lad8rr8v58xac33mwpbrixfbhn7d0xa63zpxg43bvsd")) + (patches (search-patches "syncplay-fix-typeerror.patch")))) + (build-system python-build-system) + (arguments + (list #:imported-modules `(,@%python-build-system-modules + (guix build qt-utils) + (guix build utils)) + #:modules '((guix build python-build-system) + (guix build qt-utils) + (guix build utils)) + #:phases #~(modify-phases %standard-phases + (delete 'check) + (replace 'install + (lambda _ + (invoke "make" "install" "DESTDIR=" + (string-append "PREFIX=" + (assoc-ref %outputs "out"))))) + (add-after 'install 'wrap-qt + (lambda* (#:key inputs #:allow-other-keys) + (wrap-qt-program "syncplay" + #:output #$output + #:inputs inputs + #:qt-major-version "6")))))) + (native-inputs (list python-pyside-6)) + (inputs (list bash-minimal + python-certifi + python-idna + python-service-identity + python-twisted + qtwayland)) + (home-page "https://syncplay.pl") + (synopsis "Client/server to synchronize media playback on many computers") + (description + "Syncplay is a solution to synchronize video playback across multiple +instances of media players over the Internet. When one person pauses/unpauses +playback or skips to a position in the video, this is replicated across all +media players connected to the same server and in the same \"room\" (viewing +session). A built-in text chat for discussing the synced media is also +included for convenience.") + (license license:asl2.0))) diff --git a/gnu/packages/patches/syncplay-fix-typeerror.patch b/gnu/packages/patches/syncplay-fix-typeerror.patch new file mode 100644 index 0000000000..6b1168a0e9 --- /dev/null +++ b/gnu/packages/patches/syncplay-fix-typeerror.patch @@ -0,0 +1,25 @@ +From b62b038cdf58c54205987dfc52ebf228505ad03b Mon Sep 17 00:00:00 2001 +From: Etoh <etoh <at> syncplay.pl> +Date: Tue, 11 Apr 2023 20:22:52 +0100 +Subject: [PATCH] Fix ArchLinux GUI TypeError (#600) + +--- + syncplay/ui/gui.py | 5 ++++- + 1 file changed, 4 insertions(+), 1 deletion(-) + +diff --git a/syncplay/ui/gui.py b/syncplay/ui/gui.py +index 7c100542..ca0f5d4a 100755 +--- a/syncplay/ui/gui.py ++++ b/syncplay/ui/gui.py +@@ -2106,7 +2106,10 @@ def __init__(self, passedBar=None): + if isMacOS(): + self.setWindowFlags(self.windowFlags()) + else: +- self.setWindowFlags(self.windowFlags() & Qt.AA_DontUseNativeMenuBar) ++ try: ++ self.setWindowFlags(self.windowFlags() & Qt.AA_DontUseNativeMenuBar) ++ except TypeError: ++ self.setWindowFlags(self.windowFlags()) + self.setWindowTitle("Syncplay v" + version + revision) + self.mainLayout = QtWidgets.QVBoxLayout() + self.addTopLayout(self) -- 2.41.0
aurtzy <at> gmail.com, maxim.cournoyer <at> gmail.com, guix-patches <at> gnu.org
:bug#65031
; Package guix-patches
.
(Sat, 29 Jun 2024 21:46:02 GMT) Full text and rfc822 format available.Message #16 received at 65031 <at> debbugs.gnu.org (full text, mbox):
From: aurtzy <aurtzy <at> gmail.com> To: 65031 <at> debbugs.gnu.org Cc: aurtzy <aurtzy <at> gmail.com> Subject: [PATCH v3] gnu: Add syncplay. Date: Sat, 29 Jun 2024 17:37:55 -0400
* gnu/packages/video.scm (syncplay): New variable. Change-Id: I72cdfa3db2b4b7ef6c3ef0b8585d138c272046a7 --- Hi Maxim, Just got around to looking at this package again with a new patch as things have changed since my last reply (and I didn't cc you before, oops..). In addition to addressing your initial review, notable changes from v2 include: - Update to current latest version of Syncplay (1.7.3). - Patch is no longer needed. - This patch puts syncplay in (gnu packages video) instead of the previous (gnu packages networking) since the latter seemed more concerned with system-level networking. Does this make sense? > > Is it compatible with Qt 6? To try it, replace qtwayland-5 with > > qtwayland, and specify a #:qtbase qtbase argument. > > Looks like it is! #:qtbase doesn't seem to be supported by gnu-build-system or > python-build-system (switched to the latter for this v2; elaborated further > below) and I had to make a few other adjustments to get it working, so I may > need advice if I've missed anything here. > > A patch has been added to fix a Qt 6 issue causing Syncplay to crash. > > > These should be inputs, not propagated-inputs, unless it can be used as > > a library? Also, please sort the inputs alphabetically. > > Fixed; not a library. It appears I was using an inappropriate build system for > this package - I originally made them propagated because gnu-build-system wasn't > doing whatever wrapping python-build-system does to make the inputs available, > so this v2 uses python-build-system instead now. > > Best, > > aurtzy Cheers, aurtzy gnu/packages/video.scm | 54 ++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 54 insertions(+) diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm index 411ad4c2bf..57a05f3eff 100644 --- a/gnu/packages/video.scm +++ b/gnu/packages/video.scm @@ -68,6 +68,7 @@ ;;; Copyright © 2023 Saku Laesvuori <saku <at> laesvuori.fi> ;;; Copyright © 2023 Jaeme Sifat <jaeme <at> runbox.com> ;;; Copyright © 2023 Zheng Junjie <873216071 <at> qq.com> +;;; Copyright © 2024 aurtzy <aurtzy <at> gmail.com> ;;; ;;; This file is part of GNU Guix. ;;; @@ -6349,3 +6350,56 @@ (define-public svtplay-dl broadcasters including SVT Play, Sveriges Radio, TV4 Play, along with many others.") (license license:expat))) + +(define-public syncplay + (package + (name "syncplay") + (version "1.7.3") + (source + (origin + (method git-fetch) + (uri (git-reference + (url "https://github.com/Syncplay/syncplay") + (commit (string-append "v" version)))) + (file-name (git-file-name name version)) + (sha256 + (base32 "08bgndszja4n2kql2qmzl6qrzawxvcwmywsc69lq0dzjnpdk96la")))) + (build-system python-build-system) + (arguments + (list + #:imported-modules `(,@%python-build-system-modules + (guix build qt-utils) + (guix build utils)) + #:modules '((guix build python-build-system) + (guix build qt-utils) + (guix build utils)) + #:phases #~(modify-phases %standard-phases + (delete 'check) + (replace 'install + (lambda _ + (invoke "make" "install" "DESTDIR=" + (string-append "PREFIX=" + #$output)))) + (add-after 'install 'wrap-qt + (lambda* (#:key inputs #:allow-other-keys) + (wrap-qt-program "syncplay" + #:output #$output + #:inputs inputs + #:qt-major-version "6")))))) + (native-inputs (list python-pyside-6)) + (inputs (list bash-minimal + python-certifi + python-idna + python-service-identity + python-twisted + qtwayland)) + (home-page "https://syncplay.pl") + (synopsis "Client/server to synchronize media playback on many computers") + (description + "Syncplay is a solution to synchronize video playback across multiple +instances of media players over the Internet. When one person pauses/unpauses +playback or skips to a position in the video, this is replicated across all +media players connected to the same server and in the same \"room\" (viewing +session). A built-in text chat for discussing the synced media is also +included for convenience.") + (license license:asl2.0))) base-commit: 78b881722f08325c76096557313a0faee847c66f -- 2.45.2
guix-patches <at> gnu.org
:bug#65031
; Package guix-patches
.
(Wed, 10 Jul 2024 02:49:01 GMT) Full text and rfc822 format available.Message #19 received at 65031 <at> debbugs.gnu.org (full text, mbox):
From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com> To: aurtzy <aurtzy <at> gmail.com> Cc: 65031 <at> debbugs.gnu.org Subject: Re: [bug#65031] [PATCH v3] gnu: Add syncplay. Date: Tue, 09 Jul 2024 22:46:50 -0400
Hello! aurtzy <aurtzy <at> gmail.com> writes: > * gnu/packages/video.scm (syncplay): New variable. > > Change-Id: I72cdfa3db2b4b7ef6c3ef0b8585d138c272046a7 > --- > > Hi Maxim, > > Just got around to looking at this package again with a new patch as things > have changed since my last reply (and I didn't cc you before, oops..). > > In addition to addressing your initial review, notable changes from v2 > include: > > - Update to current latest version of Syncplay (1.7.3). > - Patch is no longer needed. > - This patch puts syncplay in (gnu packages video) instead of the previous > (gnu packages networking) since the latter seemed more concerned with > system-level networking. Does this make sense? It does make sense :-). This sounds good. [...] > gnu/packages/video.scm | 54 ++++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 54 insertions(+) > > diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm > index 411ad4c2bf..57a05f3eff 100644 > --- a/gnu/packages/video.scm > +++ b/gnu/packages/video.scm > @@ -68,6 +68,7 @@ > ;;; Copyright © 2023 Saku Laesvuori <saku <at> laesvuori.fi> > ;;; Copyright © 2023 Jaeme Sifat <jaeme <at> runbox.com> > ;;; Copyright © 2023 Zheng Junjie <873216071 <at> qq.com> > +;;; Copyright © 2024 aurtzy <aurtzy <at> gmail.com> > ;;; > ;;; This file is part of GNU Guix. > ;;; > @@ -6349,3 +6350,56 @@ (define-public svtplay-dl > broadcasters including SVT Play, Sveriges Radio, TV4 Play, along with many > others.") > (license license:expat))) > + > +(define-public syncplay > + (package > + (name "syncplay") > + (version "1.7.3") > + (source > + (origin > + (method git-fetch) > + (uri (git-reference > + (url "https://github.com/Syncplay/syncplay") > + (commit (string-append "v" version)))) > + (file-name (git-file-name name version)) > + (sha256 > + (base32 "08bgndszja4n2kql2qmzl6qrzawxvcwmywsc69lq0dzjnpdk96la")))) > + (build-system python-build-system) > + (arguments > + (list > + #:imported-modules `(,@%python-build-system-modules > + (guix build qt-utils) > + (guix build utils)) > + #:modules '((guix build python-build-system) > + (guix build qt-utils) > + (guix build utils)) > + #:phases #~(modify-phases %standard-phases > + (delete 'check) > + (replace 'install > + (lambda _ > + (invoke "make" "install" "DESTDIR=" > + (string-append "PREFIX=" > + #$output)))) > + (add-after 'install 'wrap-qt > + (lambda* (#:key inputs #:allow-other-keys) > + (wrap-qt-program "syncplay" > + #:output #$output > + #:inputs inputs > + #:qt-major-version "6")))))) > + (native-inputs (list python-pyside-6)) > + (inputs (list bash-minimal > + python-certifi > + python-idna > + python-service-identity > + python-twisted > + qtwayland)) > + (home-page "https://syncplay.pl") > + (synopsis "Client/server to synchronize media playback on many computers") > + (description > + "Syncplay is a solution to synchronize video playback across multiple > +instances of media players over the Internet. When one person pauses/unpauses > +playback or skips to a position in the video, this is replicated across all > +media players connected to the same server and in the same \"room\" (viewing > +session). A built-in text chat for discussing the synced media is also > +included for convenience.") > + (license license:asl2.0))) > > base-commit: 78b881722f08325c76096557313a0faee847c66f This looks well crafted! Thank you! Reviewed-by: Maxim Cournoyer <maxim.cournoyer <at> gmail>
guix-patches <at> gnu.org
:bug#65031
; Package guix-patches
.
(Thu, 12 Sep 2024 02:27:02 GMT) Full text and rfc822 format available.Message #22 received at 65031 <at> debbugs.gnu.org (full text, mbox):
From: aurtzy <aurtzy <at> gmail.com> To: 65031 <at> debbugs.gnu.org Cc: aurtzy <aurtzy <at> gmail.com>, Maxim Cournoyer <maxim.cournoyer <at> gmail.com> Subject: Re: [PATCH] gnu: Add syncplay. Date: Wed, 11 Sep 2024 22:25:17 -0400
Hi Maxim, Small poke on this issue - is there anything else I can help with to get this merged? Cheers, aurtzy > Hello! > > aurtzy <aurtzy <at> gmail.com> writes: > > > * gnu/packages/video.scm (syncplay): New variable. > > > > Change-Id: I72cdfa3db2b4b7ef6c3ef0b8585d138c272046a7 > > --- > > > > Hi Maxim, > > > > Just got around to looking at this package again with a new patch as things > > have changed since my last reply (and I didn't cc you before, oops..). > > > > In addition to addressing your initial review, notable changes from v2 > > include: > > > > - Update to current latest version of Syncplay (1.7.3). > > - Patch is no longer needed. > > - This patch puts syncplay in (gnu packages video) instead of the previous > > (gnu packages networking) since the latter seemed more concerned with > > system-level networking. Does this make sense? > > It does make sense :-). This sounds good. > > [...] > > > gnu/packages/video.scm | 54 ++++++++++++++++++++++++++++++++++++++++++ > > 1 file changed, 54 insertions(+) > > > > diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm > > index 411ad4c2bf..57a05f3eff 100644 > > --- a/gnu/packages/video.scm > > +++ b/gnu/packages/video.scm > > @@ -68,6 +68,7 @@ > > ;;; Copyright © 2023 Saku Laesvuori <saku <at> laesvuori.fi> > > ;;; Copyright © 2023 Jaeme Sifat <jaeme <at> runbox.com> > > ;;; Copyright © 2023 Zheng Junjie <873216071 <at> qq.com> > > +;;; Copyright © 2024 aurtzy <aurtzy <at> gmail.com> > > ;;; > > ;;; This file is part of GNU Guix. > > ;;; > > @@ -6349,3 +6350,56 @@ (define-public svtplay-dl > > broadcasters including SVT Play, Sveriges Radio, TV4 Play, along with many > > others.") > > (license license:expat))) > > + > > +(define-public syncplay > > + (package > > + (name "syncplay") > > + (version "1.7.3") > > + (source > > + (origin > > + (method git-fetch) > > + (uri (git-reference > > + (url "https://github.com/Syncplay/syncplay") > > + (commit (string-append "v" version)))) > > + (file-name (git-file-name name version)) > > + (sha256 > > + (base32 "08bgndszja4n2kql2qmzl6qrzawxvcwmywsc69lq0dzjnpdk96la")))) > > + (build-system python-build-system) > > + (arguments > > + (list > > + #:imported-modules `(,@%python-build-system-modules > > + (guix build qt-utils) > > + (guix build utils)) > > + #:modules '((guix build python-build-system) > > + (guix build qt-utils) > > + (guix build utils)) > > + #:phases #~(modify-phases %standard-phases > > + (delete 'check) > > + (replace 'install > > + (lambda _ > > + (invoke "make" "install" "DESTDIR=" > > + (string-append "PREFIX=" > > + #$output)))) > > + (add-after 'install 'wrap-qt > > + (lambda* (#:key inputs #:allow-other-keys) > > + (wrap-qt-program "syncplay" > > + #:output #$output > > + #:inputs inputs > > + #:qt-major-version "6")))))) > > + (native-inputs (list python-pyside-6)) > > + (inputs (list bash-minimal > > + python-certifi > > + python-idna > > + python-service-identity > > + python-twisted > > + qtwayland)) > > + (home-page "https://syncplay.pl") > > + (synopsis "Client/server to synchronize media playback on many computers") > > + (description > > + "Syncplay is a solution to synchronize video playback across multiple > > +instances of media players over the Internet. When one person pauses/unpauses > > +playback or skips to a position in the video, this is replicated across all > > +media players connected to the same server and in the same \"room\" (viewing > > +session). A built-in text chat for discussing the synced media is also > > +included for convenience.") > > + (license license:asl2.0))) > > > > base-commit: 78b881722f08325c76096557313a0faee847c66f > > This looks well crafted! Thank you! > > Reviewed-by: Maxim Cournoyer <maxim.cournoyer <at> gmail>
Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
:aurtzy <aurtzy <at> gmail.com>
:Message #27 received at 65031-done <at> debbugs.gnu.org (full text, mbox):
From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com> To: aurtzy <aurtzy <at> gmail.com> Cc: 65031-done <at> debbugs.gnu.org Subject: Re: [PATCH] gnu: Add syncplay. Date: Sat, 14 Sep 2024 01:08:30 +0900
Hello! aurtzy <aurtzy <at> gmail.com> writes: > Hi Maxim, > > Small poke on this issue - is there anything else I can help with to > get this merged? Sorry for the delay! I've now merged it. -- Thanks, Maxim
Debbugs Internal Request <help-debbugs <at> gnu.org>
to internal_control <at> debbugs.gnu.org
.
(Sat, 12 Oct 2024 11:24:05 GMT) Full text and rfc822 format available.
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.