[bug#38811] gnu: Add gnome-menus.

[bug#38811] gnu: Add gnome-menus.

[Raghav Gururajan]

Hello Guix!

Please find the attached patch for adding gnome-patches.

Thank you!

Regards,
RG.

[bug#38811] gnu: Add gnome-menus.

[Raghav Gururajan]

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

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

[-- Attachment #2: 0001-gnu-Add-gnome-menus.patch --]
[-- Type: application/octet-stream, Size: 1656 bytes --]

From db24eb52caec6097b95d1604adcfeb8a29c72488 Mon Sep 17 00:00:00 2001
From: Raghav Gururajan <raghavgururajan@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.
---
 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)
+	      (uri (string-append "mirror://gnome/sources/gnome-menus/"
+				  (version-major+minor version) "/gnome-menus-" version ".tar.xz"))
+	      (sha256
+	       (base32 
+		"0x2blzqrapmbsbfzxjcdcpa3vkw9hq5k96h9kvjmy9kl415wcl68"))))
+    (build-system gnu-build-system)
+    (native-inputs
+     `(("gettext" ,gettext-minimal)
+       ("glib" ,glib)
+       ("pkg-config" ,pkg-config)))
+    (synopsis "GNOME Menus")
+    (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.")
+    (home-page "https://gitlab.gnome.org/GNOME/gnome-menus")
+    (license license:gpl2+)))
+
 (define-public devhelp
   (package
     (name "devhelp")
-- 
2.24.1

[bug#38811] gnu: Add gnome-menus.

[Jan Nieuwenhuizen]

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@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@gnu.org> | GNU LilyPond http://lilypond.org
Freelance IT http://JoyofSource.com | Avatar® http://AvatarAcademy.com

[bug#38811] gnu: Add gnome-menus.

[Raghav Gururajan]

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

Hello Jan!

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

Regards,
RG.

[-- Attachment #2: gnome-menus.patch --]
[-- Type: text/x-patch, Size: 1668 bytes --]

From f19b35c1bee1d48412b8481133e4f9d4995269e2 Mon Sep 17 00:00:00 2001
From: Raghav Gururajan <raghavgururajan@disroot.org>
Date: Mon, 30 Dec 2019 13:37:53 -0500
Subject: [PATCH] gnu: Add gnome-menus.

* gnu/packages/gnome.scm (gnome-menus). New variable.
---
 gnu/packages/gnome.scm | 23 +++++++++++++++++++++++
 1 file changed, 23 insertions(+)

diff --git a/gnu/packages/gnome.scm b/gnu/packages/gnome.scm
index e0712bf99a..69f4ce5cf0 100644
--- a/gnu/packages/gnome.scm
+++ b/gnu/packages/gnome.scm
@@ -246,6 +246,29 @@ Desktop.  It is designed to be as simple as possible and has some unique
 features to enable users to create their discs easily and quickly.")
     (license license:gpl2+)))
 
+(define-public gnome-menus
+  (package
+   (name "gnome-menus")
+   (version "3.32.0")
+   (source (origin
+            (method url-fetch)
+            (uri (string-append "mirror://gnome/sources/" name "/"
+                                (version-major+minor version) "/"
+                                name "-" version ".tar.xz"))
+            (sha256
+             (base32
+              "0x2blzqrapmbsbfzxjcdcpa3vkw9hq5k96h9kvjmy9kl415wcl68"))))
+   (build-system gnu-build-system)
+   (native-inputs
+    `(("gettext" ,gettext-minimal)
+      ("glib" ,glib)
+      ("pkg-config" ,pkg-config)))
+   (synopsis "Menu support for GNOME desktop")
+   (description "GNOME Menus contains the libgnome-menu library, the layout
+configuration files for the GNOME menu, as well as a simple menu editor.")
+   (home-page "https://gitlab.gnome.org/GNOME/gnome-menus")
+   (license license:gpl2)))
+
 (define-public deja-dup
   (package
     (name "deja-dup")
-- 
2.24.1

bug#38811: gnu: Add gnome-menus.

[Jan Nieuwenhuizen]

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@gnu.org> | GNU LilyPond http://lilypond.org
Freelance IT http://JoyofSource.com | Avatar® http://AvatarAcademy.com