From mboxrd@z Thu Jan  1 00:00:00 1970
From: iyzsong@member.fsf.org (=?utf-8?B?5a6L5paH5q2m?=)
Subject: Re: [PATCH] gnu: Add gnome-screenshot.
Date: Mon, 22 Feb 2016 20:02:46 +0800
Message-ID: <87mvqtq7fd.fsf@member.fsf.org>
References: <183cde6c51ace356e15a10ef3daa7f88@openmailbox.org>
Mime-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable
Return-path: <guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org>
Received: from eggs.gnu.org ([2001:4830:134:3::10]:37802)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <iyzsong@openmailbox.org>) id 1aXpCq-0003lB-Hh
	for guix-devel@gnu.org; Mon, 22 Feb 2016 07:03:13 -0500
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <iyzsong@openmailbox.org>) id 1aXpCm-0004Hh-3d
	for guix-devel@gnu.org; Mon, 22 Feb 2016 07:03:12 -0500
Received: from smtp9.openmailbox.org ([62.4.1.43]:60238)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <iyzsong@openmailbox.org>) id 1aXpCl-0004Gh-UW
	for guix-devel@gnu.org; Mon, 22 Feb 2016 07:03:08 -0500
In-Reply-To: <183cde6c51ace356e15a10ef3daa7f88@openmailbox.org>
	(rennes@openmailbox.org's message of "Sat, 20 Feb 2016 23:30:44
	-0600")
List-Id: "Development of GNU Guix and the GNU System distribution."
	<guix-devel.gnu.org>
List-Unsubscribe: <https://lists.gnu.org/mailman/options/guix-devel>,
	<mailto:guix-devel-request@gnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/archive/html/guix-devel>
List-Post: <mailto:guix-devel@gnu.org>
List-Help: <mailto:guix-devel-request@gnu.org?subject=help>
List-Subscribe: <https://lists.gnu.org/mailman/listinfo/guix-devel>,
	<mailto:guix-devel-request@gnu.org?subject=subscribe>
Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org
Sender: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org
To: rennes@openmailbox.org
Cc: guix-devel@gnu.org

rennes@openmailbox.org writes:

> Hi,
>
> i attached gnome-screenshot patch for review.
>
> GNOME Screenshot is a utility used for taking screenshots of the screen.
>
> Thanks
>
> From be7026e12d1ae7611f4b85188fb6400799c39c25 Mon Sep 17 00:00:00 2001
> From: Rene Saavedra <rennes@openmailbox.org>
> Date: Sat, 20 Feb 2016 23:25:12 -0600
> Subject: [PATCH] gnu: Add gnome-screenshot.
>
> * gnu/packages/gnome.scm (gnome-screenshot): New variable.
>
> ---
>  gnu/packages/gnome.scm | 31 +++++++++++++++++++++++++++++++
>  1 file changed, 31 insertions(+)
>
> diff --git a/gnu/packages/gnome.scm b/gnu/packages/gnome.scm
> index 363d920..8406056 100644
> --- a/gnu/packages/gnome.scm
> +++ b/gnu/packages/gnome.scm
> @@ -4711,6 +4711,37 @@ can add your own files to the collection.")
>                     license:cc-by-sa2.0
>                     license:cc-by-sa3.0))))
>=20=20
> +(define-public gnome-screenshot
> +  (package
> +    (name "gnome-screenshot")
> +    (version "3.18.0")
> +    (source
> +     (origin
> +       (method url-fetch)
> +       (uri (string-append "mirror://gnome/sources/" name "/"
> +                           (version-major+minor version) "/"
> +                           name "-" version ".tar.xz"))
> +       (sha256
> +        (base32
> +         "0hc8m435q7yzvrw7jpi53kaxpmrd9w59sm7c5wibh2ng9azlv9pb"))))
> +    (build-system glib-or-gtk-build-system)
> +    (native-inputs
> +     `(("glib:bin" ,glib "bin")  ;; XXX: for glib-compile-schemas, gio-2=
.0.
Use single =E2=80=98;=E2=80=99 for inline comment.
=E2=80=9CXXX=E2=80=9D is for TODO or FIXME, here it=E2=80=99s just note.
> +       ("intltool" ,intltool)
> +       ("pkg-config" ,pkg-config)))
> +    (inputs
> +     `(("gtk+" ,gtk+)
> +       ("libcanberra" ,libcanberra)
> +       ("libx11" ,libx11)
> +       ("libxext" ,libxext)))
> +    (home-page "https://git.gnome.org/browse/gnome-screenshot")
> +    (synopsis "Take pictures of your screen")
> +    (description
> +     "GNOME Screenshot is a utility used for taking screenshots of the e=
ntire
> +screen, a window or a user defined area of the screen, with optional
> +beautifying border effects.")
> +    (license license:gpl2+)))
> +
>  (define-public gnome
>    (package
>      (name "gnome")
Pushed with comment little adjusted, thanks!