GNU bug report logs - #38811
gnu: Add gnome-menus.

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 38811 in the body.
You can then email your comments to 38811 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: "Raghav Gururajan" <raghavgururajan <at> disroot.org>
To: guix-patches <at> gnu.org
Subject: gnu: Add gnome-menus.
Date: Mon, 30 Dec 2019 12:52:29 +0000

Hello Guix!

Please find the attached patch for adding gnome-patches.

Thank you!

Regards,
RG.

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

From: "Raghav Gururajan" <raghavgururajan <at> disroot.org>
To: 38811 <at> debbugs.gnu.org
Subject: Re: gnu: Add gnome-menus.
Date: Mon, 30 Dec 2019 12:55:05 +0000

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

OOPS! Forgot to attach. Here, I have attached.

[0001-gnu-Add-gnome-menus.patch (application/octet-stream, attachment)]

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

From: Jan Nieuwenhuizen <janneke <at> gnu.org>
To: "Raghav Gururajan" <raghavgururajan <at> disroot.org>
Cc: 38811 <at> debbugs.gnu.org
Subject: Re: [bug#38811] gnu: Add gnome-menus.
Date: Mon, 30 Dec 2019 14:21:09 +0100

Raghav Gururajan writes:

Hello Raghav,

I was about to commit your patch with changes and then decided there are
a bit too many small things to fix.  In essence your package is fine,
just a a number of cleanups are needed.  Comments in-line below.

Have you tried to running the package; I do not use GNOME?

> From db24eb52caec6097b95d1604adcfeb8a29c72488 Mon Sep 17 00:00:00 2001
> From: Raghav Gururajan <raghavgururajan <at> disroot.org>
> Date: Mon, 30 Dec 2019 07:47:30 -0500
> Subject: [PATCH] gnu: Add gnome-menus.
>
> * gnu/packages/gnome.scm (gnome-menus). New Variable.

Use lower case on variable: New variable.

> ---
>  gnu/packages/gnome.scm | 21 +++++++++++++++++++++
>  1 file changed, 21 insertions(+)
>
> diff --git a/gnu/packages/gnome.scm b/gnu/packages/gnome.scm
> index e0712bf99a..607a546768 100644
> --- a/gnu/packages/gnome.scm
> +++ b/gnu/packages/gnome.scm
> @@ -3848,6 +3848,27 @@ It is a basic GtkUIManager replacement based on GAction.  It is suitable for
>  both a traditional UI or a modern UI with a GtkHeaderBar.")
>      (license license:lgpl2.1+)))
>  
> +(define-public gnome-menus
> +  (package
> +    (name "gnome-menus")
> +    (version "3.32.0")
> +    (source (origin
> +	      (method url-fetch)
   ^
Use spaces instead of TABs.

> +	      (uri (string-append "mirror://gnome/sources/gnome-menus/"
> +				  (version-major+minor version) "/gnome-menus-" version ".tar.xz"))

Use spaces instead of TABs, add line break to stay within 80 columns.

> +	      (sha256
> +	       (base32 
                      ^
Remove trailing whitespace.

> +		"0x2blzqrapmbsbfzxjcdcpa3vkw9hq5k96h9kvjmy9kl415wcl68"))))
> +    (build-system gnu-build-system)
> +    (native-inputs
> +     `(("gettext" ,gettext-minimal)
> +       ("glib" ,glib)
> +       ("pkg-config" ,pkg-config)))
> +    (synopsis "GNOME Menus")

This is too non-descriptive, use something like

"GNOME implementation of the freedesktop menu specification"

> +    (description "It contains the libgnome-menu library, the layout configuration files for the GNOME menu, as well as a simple menu editor. The libgnome-menu library implements the 'Desktop Menu Specification' from freedesktop.org.")

Instead of "It", start with

   GNOME Menus contains ...

add line breaks to stay within 80 columns, use two spaces after each sentence.

> +    (home-page "https://gitlab.gnome.org/GNOME/gnome-menus")
> +    (license license:gpl2+)))

It looks like the package is licensed partly under gpl2 and lgpl2 (not
gpl2+).  Can you please double check?

> +
>  (define-public devhelp
>    (package
>      (name "devhelp")

Can you please send and updated patch?  Most of the corrections are
reported by guix lint, before you send it please run

    ./pre-inst-env guix lint gnome-menus

and make sure there are no errors reported.

Thanks for your contribution!

Greetings,
janneke

-- 
Jan Nieuwenhuizen <janneke <at> gnu.org> | GNU LilyPond http://lilypond.org
Freelance IT http://JoyofSource.com | Avatar® http://AvatarAcademy.com

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

From: Raghav Gururajan <raghavgururajan <at> disroot.org>
To: Jan Nieuwenhuizen <janneke <at> gnu.org>
Cc: 38811 <at> debbugs.gnu.org
Subject: Re: [bug#38811] gnu: Add gnome-menus.
Date: Mon, 30 Dec 2019 13:41:25 -0500

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

Hello Jan!

Please find the revised patch attached with this email. :-)

Regards,
RG.

[gnome-menus.patch (text/x-patch, attachment)]

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

From: Jan Nieuwenhuizen <janneke <at> gnu.org>
To: Raghav Gururajan <raghavgururajan <at> disroot.org>
Cc: 38811-done <at> debbugs.gnu.org
Subject: Re: [bug#38811] gnu: Add gnome-menus.
Date: Mon, 30 Dec 2019 21:02:04 +0100

Raghav Gururajan writes:

Hello Raghav,

> Please find the revised patch attached with this email. :-)

Thanks, applied and pushed to master with two minor changes.

> +(define-public gnome-menus
> +  (package
> +   (name "gnome-menus")

Updated to two spaces of indentation.

> +            (uri (string-append "mirror://gnome/sources/" name "/"

I changed this to "mirror://gnome/sources/gnome-menus/"

Greetings,
janneke

-- 
Jan Nieuwenhuizen <janneke <at> gnu.org> | GNU LilyPond http://lilypond.org
Freelance IT http://JoyofSource.com | Avatar® http://AvatarAcademy.com

Previous Next

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