all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: ng0 <ng0@n0.is>
To: Kei Kebreau <kkebreau@posteo.net>
Cc: 28917-done@debbugs.gnu.org
Subject: bug#28917: gst-transcoder + python-pycanberra + gavl (pitivi dependencies)
Date: Tue, 28 Nov 2017 19:04:55 +0000	[thread overview]
Message-ID: <20171128190455.wp33pxei2snh7ozq@abyayala> (raw)
In-Reply-To: <87efpopyge.fsf@posteo.net>

[-- Attachment #1: Type: text/plain, Size: 4603 bytes --]

Kei Kebreau transcribed 5.0K bytes:
> ng0 <ng0@infotropique.org> writes:
> 
> > Kei Kebreau transcribed 3.2K bytes:
> >> ng0 <ng0@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@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

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

      reply	other threads:[~2017-11-28 19:06 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-10-20 19:35 [bug#28917] gst-transcoder + python-pycanberra + gavl (pitivi dependencies) ng0
2017-10-21 21:29 ` ng0
2017-10-21 21:32   ` ng0
2017-10-25 14:24     ` Kei Kebreau
2017-10-26 20:07       ` ng0
2017-10-27 12:28         ` Kei Kebreau
2017-11-28 19:04           ` ng0 [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20171128190455.wp33pxei2snh7ozq@abyayala \
    --to=ng0@n0.is \
    --cc=28917-done@debbugs.gnu.org \
    --cc=kkebreau@posteo.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.