guix-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 1/3] gnu: Add imlib2.


From: Alex Kost
Subject: Re: [PATCH 1/3] gnu: Add imlib2.
Date: Thu, 25 Sep 2014 21:36:46 +0400
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux)

Many thanks to Andreas for detailed comments about licenses.

Ludovic Courtès (2014-09-25 16:56 +0400) wrote:

> Alex Kost <address@hidden> skribis:
>
>> +(define-public imlib2
>
> [...]
>
>> +    (home-page "http://sourceforge.net/projects/enlightenment/";)
>> +    (synopsis
>> +     "Library for loading, saving, rendering and manipulating image files")
>
> s/Library for//

Done.  Is there any convention about that?  I'm asking because there are
several "types" of synopses:

- Several synopses begin with "Library for" (libuv, fontconfig,
  lightning, ...)

- Few ones begin with "Library to" (cloog, petsc, ...)

- Some packages have the following synopsis structure:

    <package-name>, a library ...

  Some of them begin with a capital letter (libjpeg, libgpg-error,
  freetype, ...), and some with a small one (fpill, libogg, libpaper, ...)

- Many synopses end with "library" (libsamplerate, clucene, libgcrypt, ...)

Also (as I'm writing about synopses): is there a convention to begin a
synopsis with a capital letter?  There are synopses beginning with a
small letter (not counting synopses beginning with a package name):
gnome-vfs, cdparanoia, signing-party, ...

Also there are 2 packages with synopses beginning with an article:
“pkg-config” and “id3lib”, but it's a lower-case article (perhaps "guix
lint" should be adjusted for this case).

> Actually this should probably go to (gnu packages image), which is home
> for all its friends.

OK, initially I wanted to put it there, but then I found that “giflib”
is in a separate file, so I thought that maybe "image.scm" contains some
special image libraries.

If “giflib” also should be put in "image.scm", please don't move it
there before I push these imlib2/giblib commits  :-)
Thanks.

The modified patch is attached.  Good enough?

Attachment: 0001-gnu-Add-imlib2.patch
Description: Text Data


reply via email to

[Prev in Thread] Current Thread [Next in Thread]