GNU bug report logs - #28917
gst-transcoder + python-pycanberra + gavl (pitivi dependencies)

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 28917 in the body.
You can then email your comments to 28917 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: ng0 <ng0 <at> infotropique.org>
To: guix-patches <at> gnu.org
Subject: gst-transcoder + python-pycanberra + gavl (pitivi dependencies)
Date: Fri, 20 Oct 2017 19:35:25 +0000

[Message part 1 (text/plain, inline)]

This adds gst-transcoder, python-pycanberra and gavl
required for pitivi.
-- 
ng0
GnuPG: A88C8ADD129828D7EAC02E52E22F9BBFEE348588
GnuPG: https://dist.ng0.infotropique.org/dist/keys/
https://www.infotropique.org https://ng0.infotropique.org

[0001-gnu-Add-gst-transcoder.patch (text/plain, attachment)]

[0002-gnu-Add-python-pycanberra.patch (text/plain, attachment)]

[0003-gnu-Add-gavl.patch (text/plain, attachment)]

[signature.asc (application/pgp-signature, inline)]

Message #8 received at 28917 <at> debbugs.gnu.org (full text, mbox):

From: ng0 <ng0 <at> infotropique.org>
To: 28917 <at> debbugs.gnu.org
Subject: Re: [bug#28917] gst-transcoder + python-pycanberra + gavl (pitivi
 dependencies)
Date: Sat, 21 Oct 2017 21:29:16 +0000

[Message part 1 (text/plain, inline)]

ng0 transcribed 10K bytes:
> This adds gst-transcoder, python-pycanberra and gavl
> required for pitivi.

Yarg. I've sent the wrong python-pycanberra. Correction incoming.
-- 
ng0
GnuPG: A88C8ADD129828D7EAC02E52E22F9BBFEE348588
GnuPG: https://dist.ng0.infotropique.org/dist/keys/
https://www.infotropique.org https://ng0.infotropique.org

[signature.asc (application/pgp-signature, inline)]

Message #11 received at 28917 <at> debbugs.gnu.org (full text, mbox):

From: ng0 <ng0 <at> infotropique.org>
To: 28917 <at> debbugs.gnu.org
Subject: Re: [bug#28917] gst-transcoder + python-pycanberra + gavl (pitivi
 dependencies)
Date: Sat, 21 Oct 2017 21:32:50 +0000

[Message part 1 (text/plain, inline)]

ng0 transcribed 1.4K bytes:
> ng0 transcribed 10K bytes:
> > This adds gst-transcoder, python-pycanberra and gavl
> > required for pitivi.
> 
> Yarg. I've sent the wrong python-pycanberra. Correction incoming.

Correction appended.

-- 
ng0
GnuPG: A88C8ADD129828D7EAC02E52E22F9BBFEE348588
GnuPG: https://dist.ng0.infotropique.org/dist/keys/
https://www.infotropique.org https://ng0.infotropique.org

[0001-gnu-Add-gst-transcoder.patch (text/plain, attachment)]

[0002-gnu-Add-python-pycanberra.patch (text/plain, attachment)]

[0003-gnu-Add-gavl.patch (text/plain, attachment)]

[signature.asc (application/pgp-signature, inline)]

Message #14 received at 28917 <at> debbugs.gnu.org (full text, mbox):

From: Kei Kebreau <kkebreau <at> posteo.net>
To: ng0 <ng0 <at> infotropique.org>
Cc: 28917 <at> debbugs.gnu.org
Subject: Re: [bug#28917] gst-transcoder + python-pycanberra + gavl (pitivi
 dependencies)
Date: Wed, 25 Oct 2017 10:24:06 -0400

ng0 <ng0 <at> infotropique.org> writes:

> ng0 transcribed 1.4K bytes:
>> ng0 transcribed 10K bytes:
>> > This adds gst-transcoder, python-pycanberra and gavl
>> > required for pitivi.
>> 
>> Yarg. I've sent the wrong python-pycanberra. Correction incoming.
>
> Correction appended.
>
> -- 
> ng0
> GnuPG: A88C8ADD129828D7EAC02E52E22F9BBFEE348588
> GnuPG: https://dist.ng0.infotropique.org/dist/keys/
> https://www.infotropique.org https://ng0.infotropique.org
>
>
>
> From b97a66068be69bc53d654d6204cdb1df5c889dd6 Mon Sep 17 00:00:00 2001
> From: ng0 <ng0 <at> infotropique.org>
> Date: Fri, 20 Oct 2017 18:32:12 +0000
> Subject: [PATCH 3/3] gnu: Add gavl.
>
> * gnu/packages/video.scm (gavl): New variable.
> ---
>  gnu/packages/video.scm | 34 ++++++++++++++++++++++++++++++++++
>  1 file changed, 34 insertions(+)
>
> diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
> index baebe6404..7294c5bf2 100644
> --- a/gnu/packages/video.scm
> +++ b/gnu/packages/video.scm
> @@ -69,6 +69,7 @@
>    #:use-module (gnu packages databases)
>    #:use-module (gnu packages dejagnu)
>    #:use-module (gnu packages docbook)
> +  #:use-module (gnu packages documentation)
>    #:use-module (gnu packages elf)
>    #:use-module (gnu packages file)
>    #:use-module (gnu packages flex)
> @@ -2499,3 +2500,36 @@ tools for styling them, including a built-in real-time video preview.")
>      (synopsis "GStreamer Transcoding API")
>      (description "GStreamer Transcoding API")
>      (license license:lgpl2.1)))
> +
> +(define-public gavl
> +  (package
> +    (name "gavl")
> +    (version "1.4.0")
> +    (source
> +     (origin
> +       (method url-fetch)
> +       (uri (string-append "mirror://sourceforge/gmerlin/"
> +                           name "/" version "/"
> +                           name "-" version ".tar.gz"))
> +       (file-name (string-append name "-" version ".tar.gz"))
> +       (sha256
> +        (base32
> +         "1kikkn971a14zzm7svi7190ldc14fjai0xyhpbcmp48s750sraji"))))
> +    (build-system gnu-build-system)
> +    (native-inputs
> +     `(("pkg-config" ,pkg-config)
> +       ("doxygen" ,doxygen)))
> +    (home-page "http://gmerlin.sourceforge.net")
> +    (synopsis "low level library for multimedia API building")
> +    (description
> +     "Gavl is short for Gmerlin Audio Video Library.  It is a low level
> +library, upon which multimedia APIs can be built.  Gavl handles all the
> +details of audio and video formats like colorspaces, samplerates,

"samplerates" should be two words.

> +multichannel configurations etc.  It provides standardized definitions for
                              ^
               a comma after "configurations" would be nice.
> +those formats as well as container structures for carrying audio samples or
> +video images inside an application.
> +
> +In addition, it handles the sometimes ugly task to convert between all

Instead of "ugly task to convert..." perhaps we can say "ugly task of
converting..."

> +these formats and provides some elementary operations (copying, scaling,
> +alpha blending etc).")
> +    (license license:gpl3)))

I suppose the changes to gavl's description could be forwarded upstream,
seeing as this is the same description as the one
gmerlin.sourceforge.net uses.

Everything else LGTM. Thanks for working on this!

Message #17 received at 28917 <at> debbugs.gnu.org (full text, mbox):

From: ng0 <ng0 <at> infotropique.org>
To: Kei Kebreau <kkebreau <at> posteo.net>
Cc: 28917 <at> debbugs.gnu.org, ng0 <ng0 <at> infotropique.org>
Subject: Re: [bug#28917] gst-transcoder + python-pycanberra + gavl (pitivi
 dependencies)
Date: Thu, 26 Oct 2017 20:07:19 +0000

[Message part 1 (text/plain, inline)]

Kei Kebreau transcribed 3.2K bytes:
> ng0 <ng0 <at> infotropique.org> writes:
> 
> > ng0 transcribed 1.4K bytes:
> >> ng0 transcribed 10K bytes:
> >> > This adds gst-transcoder, python-pycanberra and gavl
> >> > required for pitivi.
> >> 
> >> Yarg. I've sent the wrong python-pycanberra. Correction incoming.
> >
> > Correction appended.
> >
> > -- 
> > ng0
> > GnuPG: A88C8ADD129828D7EAC02E52E22F9BBFEE348588
> > GnuPG: https://dist.ng0.infotropique.org/dist/keys/
> > https://www.infotropique.org https://ng0.infotropique.org
> >
> >
> >
> > From b97a66068be69bc53d654d6204cdb1df5c889dd6 Mon Sep 17 00:00:00 2001
> > From: ng0 <ng0 <at> infotropique.org>
> > Date: Fri, 20 Oct 2017 18:32:12 +0000
> > Subject: [PATCH 3/3] gnu: Add gavl.
> >
> > * gnu/packages/video.scm (gavl): New variable.
> > ---
> >  gnu/packages/video.scm | 34 ++++++++++++++++++++++++++++++++++
> >  1 file changed, 34 insertions(+)
> >
> > diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
> > index baebe6404..7294c5bf2 100644
> > --- a/gnu/packages/video.scm
> > +++ b/gnu/packages/video.scm
> > @@ -69,6 +69,7 @@
> >    #:use-module (gnu packages databases)
> >    #:use-module (gnu packages dejagnu)
> >    #:use-module (gnu packages docbook)
> > +  #:use-module (gnu packages documentation)
> >    #:use-module (gnu packages elf)
> >    #:use-module (gnu packages file)
> >    #:use-module (gnu packages flex)
> > @@ -2499,3 +2500,36 @@ tools for styling them, including a built-in real-time video preview.")
> >      (synopsis "GStreamer Transcoding API")
> >      (description "GStreamer Transcoding API")
> >      (license license:lgpl2.1)))
> > +
> > +(define-public gavl
> > +  (package
> > +    (name "gavl")
> > +    (version "1.4.0")
> > +    (source
> > +     (origin
> > +       (method url-fetch)
> > +       (uri (string-append "mirror://sourceforge/gmerlin/"
> > +                           name "/" version "/"
> > +                           name "-" version ".tar.gz"))
> > +       (file-name (string-append name "-" version ".tar.gz"))
> > +       (sha256
> > +        (base32
> > +         "1kikkn971a14zzm7svi7190ldc14fjai0xyhpbcmp48s750sraji"))))
> > +    (build-system gnu-build-system)
> > +    (native-inputs
> > +     `(("pkg-config" ,pkg-config)
> > +       ("doxygen" ,doxygen)))
> > +    (home-page "http://gmerlin.sourceforge.net")
> > +    (synopsis "low level library for multimedia API building")
> > +    (description
> > +     "Gavl is short for Gmerlin Audio Video Library.  It is a low level
> > +library, upon which multimedia APIs can be built.  Gavl handles all the
> > +details of audio and video formats like colorspaces, samplerates,
> 
> "samplerates" should be two words.
> 
> > +multichannel configurations etc.  It provides standardized definitions for
>                               ^
>                a comma after "configurations" would be nice.
> > +those formats as well as container structures for carrying audio samples or
> > +video images inside an application.
> > +
> > +In addition, it handles the sometimes ugly task to convert between all
> 
> Instead of "ugly task to convert..." perhaps we can say "ugly task of
> converting..."
> 
> > +these formats and provides some elementary operations (copying, scaling,
> > +alpha blending etc).")
> > +    (license license:gpl3)))
> 
> I suppose the changes to gavl's description could be forwarded upstream,
> seeing as this is the same description as the one
> gmerlin.sourceforge.net uses.

Maybe… Besides having the correct text it requires writing
the email etc. If I don't forget it, I'll get to it eventually
at some point… this year.

> Everything else LGTM. Thanks for working on this!
> 
Thanks for the review.

Updated patches appended.
-- 
ng0
GnuPG: A88C8ADD129828D7EAC02E52E22F9BBFEE348588
GnuPG: https://dist.ng0.infotropique.org/dist/keys/
https://www.infotropique.org https://ng0.infotropique.org

[0001-gnu-Add-gst-transcoder.patch (text/plain, attachment)]

[0002-gnu-Add-python-pycanberra.patch (text/plain, attachment)]

[0003-gnu-Add-gavl.patch (text/plain, attachment)]

[signature.asc (application/pgp-signature, inline)]

Message #20 received at 28917 <at> debbugs.gnu.org (full text, mbox):

From: Kei Kebreau <kkebreau <at> posteo.net>
To: ng0 <ng0 <at> infotropique.org>
Cc: 28917 <at> debbugs.gnu.org
Subject: Re: [bug#28917] gst-transcoder + python-pycanberra + gavl (pitivi
 dependencies)
Date: Fri, 27 Oct 2017 08:28:01 -0400

[Message part 1 (text/plain, inline)]

ng0 <ng0 <at> infotropique.org> writes:

> Kei Kebreau transcribed 3.2K bytes:
>> ng0 <ng0 <at> infotropique.org> writes:
>> 
>> > ng0 transcribed 1.4K bytes:
>> >> ng0 transcribed 10K bytes:
>> >> > This adds gst-transcoder, python-pycanberra and gavl
>> >> > required for pitivi.
>> >> 
>> >> Yarg. I've sent the wrong python-pycanberra. Correction incoming.
>> >
>> > Correction appended.
>> >
>> > -- 
>> > ng0
>> > GnuPG: A88C8ADD129828D7EAC02E52E22F9BBFEE348588
>> > GnuPG: https://dist.ng0.infotropique.org/dist/keys/
>> > https://www.infotropique.org https://ng0.infotropique.org
>> >
>> >
>> >
>> > From b97a66068be69bc53d654d6204cdb1df5c889dd6 Mon Sep 17 00:00:00 2001
>> > From: ng0 <ng0 <at> infotropique.org>
>> > Date: Fri, 20 Oct 2017 18:32:12 +0000
>> > Subject: [PATCH 3/3] gnu: Add gavl.
>> >
>> > * gnu/packages/video.scm (gavl): New variable.
>> > ---
>> >  gnu/packages/video.scm | 34 ++++++++++++++++++++++++++++++++++
>> >  1 file changed, 34 insertions(+)
>> >
>> > diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
>> > index baebe6404..7294c5bf2 100644
>> > --- a/gnu/packages/video.scm
>> > +++ b/gnu/packages/video.scm
>> > @@ -69,6 +69,7 @@
>> >    #:use-module (gnu packages databases)
>> >    #:use-module (gnu packages dejagnu)
>> >    #:use-module (gnu packages docbook)
>> > +  #:use-module (gnu packages documentation)
>> >    #:use-module (gnu packages elf)
>> >    #:use-module (gnu packages file)
>> >    #:use-module (gnu packages flex)
>> > @@ -2499,3 +2500,36 @@ tools for styling them, including a built-in real-time video preview.")
>> >      (synopsis "GStreamer Transcoding API")
>> >      (description "GStreamer Transcoding API")
>> >      (license license:lgpl2.1)))
>> > +
>> > +(define-public gavl
>> > +  (package
>> > +    (name "gavl")
>> > +    (version "1.4.0")
>> > +    (source
>> > +     (origin
>> > +       (method url-fetch)
>> > +       (uri (string-append "mirror://sourceforge/gmerlin/"
>> > +                           name "/" version "/"
>> > +                           name "-" version ".tar.gz"))
>> > +       (file-name (string-append name "-" version ".tar.gz"))
>> > +       (sha256
>> > +        (base32
>> > +         "1kikkn971a14zzm7svi7190ldc14fjai0xyhpbcmp48s750sraji"))))
>> > +    (build-system gnu-build-system)
>> > +    (native-inputs
>> > +     `(("pkg-config" ,pkg-config)
>> > +       ("doxygen" ,doxygen)))
>> > +    (home-page "http://gmerlin.sourceforge.net")
>> > +    (synopsis "low level library for multimedia API building")

I also missed this the first time: the synopsis must start with an
upper-case letter or digit. I fixed it before pushing the patch.

>> > +    (description
>> > +     "Gavl is short for Gmerlin Audio Video Library.  It is a low level
>> > +library, upon which multimedia APIs can be built.  Gavl handles all the
>> > +details of audio and video formats like colorspaces, samplerates,
>> 
>> "samplerates" should be two words.
>> 
>> > +multichannel configurations etc.  It provides standardized definitions for
>>                               ^
>>                a comma after "configurations" would be nice.
>> > +those formats as well as container structures for carrying audio samples or
>> > +video images inside an application.
>> > +
>> > +In addition, it handles the sometimes ugly task to convert between all
>> 
>> Instead of "ugly task to convert..." perhaps we can say "ugly task of
>> converting..."
>> 
>> > +these formats and provides some elementary operations (copying, scaling,
>> > +alpha blending etc).")
>> > +    (license license:gpl3)))
>> 
>> I suppose the changes to gavl's description could be forwarded upstream,
>> seeing as this is the same description as the one
>> gmerlin.sourceforge.net uses.
>
> Maybe… Besides having the correct text it requires writing
> the email etc. If I don't forget it, I'll get to it eventually
> at some point… this year.
>
>> Everything else LGTM. Thanks for working on this!
>> 
> Thanks for the review.
>
> Updated patches appended.

Excellent, and thank you. These have been pushed to master.

[signature.asc (application/pgp-signature, inline)]

Message #25 received at 28917-done <at> debbugs.gnu.org (full text, mbox):

From: ng0 <ng0 <at> n0.is>
To: Kei Kebreau <kkebreau <at> posteo.net>
Cc: 28917-done <at> debbugs.gnu.org, ng0 <ng0 <at> n0.is>
Subject: Re: [bug#28917] gst-transcoder + python-pycanberra + gavl (pitivi
 dependencies)
Date: Tue, 28 Nov 2017 19:04:55 +0000

[Message part 1 (text/plain, inline)]

Kei Kebreau transcribed 5.0K bytes:
> ng0 <ng0 <at> infotropique.org> writes:
> 
> > Kei Kebreau transcribed 3.2K bytes:
> >> ng0 <ng0 <at> infotropique.org> writes:
> >> 
> >> > ng0 transcribed 1.4K bytes:
> >> >> ng0 transcribed 10K bytes:
> >> >> > This adds gst-transcoder, python-pycanberra and gavl
> >> >> > required for pitivi.
> >> >> 
> >> >> Yarg. I've sent the wrong python-pycanberra. Correction incoming.
> >> >
> >> > Correction appended.
> >> >
> >> > -- 
> >> > ng0
> >> > GnuPG: A88C8ADD129828D7EAC02E52E22F9BBFEE348588
> >> > GnuPG: https://dist.ng0.infotropique.org/dist/keys/
> >> > https://www.infotropique.org https://ng0.infotropique.org
> >> >
> >> >
> >> >
> >> > From b97a66068be69bc53d654d6204cdb1df5c889dd6 Mon Sep 17 00:00:00 2001
> >> > From: ng0 <ng0 <at> infotropique.org>
> >> > Date: Fri, 20 Oct 2017 18:32:12 +0000
> >> > Subject: [PATCH 3/3] gnu: Add gavl.
> >> >
> >> > * gnu/packages/video.scm (gavl): New variable.
> >> > ---
> >> >  gnu/packages/video.scm | 34 ++++++++++++++++++++++++++++++++++
> >> >  1 file changed, 34 insertions(+)
> >> >
> >> > diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
> >> > index baebe6404..7294c5bf2 100644
> >> > --- a/gnu/packages/video.scm
> >> > +++ b/gnu/packages/video.scm
> >> > @@ -69,6 +69,7 @@
> >> >    #:use-module (gnu packages databases)
> >> >    #:use-module (gnu packages dejagnu)
> >> >    #:use-module (gnu packages docbook)
> >> > +  #:use-module (gnu packages documentation)
> >> >    #:use-module (gnu packages elf)
> >> >    #:use-module (gnu packages file)
> >> >    #:use-module (gnu packages flex)
> >> > @@ -2499,3 +2500,36 @@ tools for styling them, including a built-in real-time video preview.")
> >> >      (synopsis "GStreamer Transcoding API")
> >> >      (description "GStreamer Transcoding API")
> >> >      (license license:lgpl2.1)))
> >> > +
> >> > +(define-public gavl
> >> > +  (package
> >> > +    (name "gavl")
> >> > +    (version "1.4.0")
> >> > +    (source
> >> > +     (origin
> >> > +       (method url-fetch)
> >> > +       (uri (string-append "mirror://sourceforge/gmerlin/"
> >> > +                           name "/" version "/"
> >> > +                           name "-" version ".tar.gz"))
> >> > +       (file-name (string-append name "-" version ".tar.gz"))
> >> > +       (sha256
> >> > +        (base32
> >> > +         "1kikkn971a14zzm7svi7190ldc14fjai0xyhpbcmp48s750sraji"))))
> >> > +    (build-system gnu-build-system)
> >> > +    (native-inputs
> >> > +     `(("pkg-config" ,pkg-config)
> >> > +       ("doxygen" ,doxygen)))
> >> > +    (home-page "http://gmerlin.sourceforge.net")
> >> > +    (synopsis "low level library for multimedia API building")
> 
> I also missed this the first time: the synopsis must start with an
> upper-case letter or digit. I fixed it before pushing the patch.
> 
> >> > +    (description
> >> > +     "Gavl is short for Gmerlin Audio Video Library.  It is a low level
> >> > +library, upon which multimedia APIs can be built.  Gavl handles all the
> >> > +details of audio and video formats like colorspaces, samplerates,
> >> 
> >> "samplerates" should be two words.
> >> 
> >> > +multichannel configurations etc.  It provides standardized definitions for
> >>                               ^
> >>                a comma after "configurations" would be nice.
> >> > +those formats as well as container structures for carrying audio samples or
> >> > +video images inside an application.
> >> > +
> >> > +In addition, it handles the sometimes ugly task to convert between all
> >> 
> >> Instead of "ugly task to convert..." perhaps we can say "ugly task of
> >> converting..."
> >> 
> >> > +these formats and provides some elementary operations (copying, scaling,
> >> > +alpha blending etc).")
> >> > +    (license license:gpl3)))
> >> 
> >> I suppose the changes to gavl's description could be forwarded upstream,
> >> seeing as this is the same description as the one
> >> gmerlin.sourceforge.net uses.
> >
> > Maybe… Besides having the correct text it requires writing
> > the email etc. If I don't forget it, I'll get to it eventually
> > at some point… this year.
> >
> >> Everything else LGTM. Thanks for working on this!
> >> 
> > Thanks for the review.
> >
> > Updated patches appended.
> 
> Excellent, and thank you. These have been pushed to master.

Thanks!

Closing the bug.

N.
-- 
GnuPG: A88C8ADD129828D7EAC02E52E22F9BBFEE348588
GnuPG: https://c.n0.is/ng0_pubkeys/tree/keys
  WWW: https://n0.is

[signature.asc (application/pgp-signature, inline)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.