* [bug#38811] gnu: Add gnome-menus. @ 2019-12-30 12:52 Raghav Gururajan 2019-12-30 12:55 ` Raghav Gururajan 0 siblings, 1 reply; 5+ messages in thread From: Raghav Gururajan @ 2019-12-30 12:52 UTC (permalink / raw) To: 38811 Hello Guix! Please find the attached patch for adding gnome-patches. Thank you! Regards, RG. ^ permalink raw reply [flat|nested] 5+ messages in thread
* [bug#38811] gnu: Add gnome-menus. 2019-12-30 12:52 [bug#38811] gnu: Add gnome-menus Raghav Gururajan @ 2019-12-30 12:55 ` Raghav Gururajan 2019-12-30 13:21 ` Jan Nieuwenhuizen 0 siblings, 1 reply; 5+ messages in thread From: Raghav Gururajan @ 2019-12-30 12:55 UTC (permalink / raw) To: 38811 [-- 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 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* [bug#38811] gnu: Add gnome-menus. 2019-12-30 12:55 ` Raghav Gururajan @ 2019-12-30 13:21 ` Jan Nieuwenhuizen 2019-12-30 18:41 ` Raghav Gururajan 0 siblings, 1 reply; 5+ messages in thread From: Jan Nieuwenhuizen @ 2019-12-30 13:21 UTC (permalink / raw) To: Raghav Gururajan; +Cc: 38811 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 ^ permalink raw reply [flat|nested] 5+ messages in thread
* [bug#38811] gnu: Add gnome-menus. 2019-12-30 13:21 ` Jan Nieuwenhuizen @ 2019-12-30 18:41 ` Raghav Gururajan 2019-12-30 20:02 ` bug#38811: " Jan Nieuwenhuizen 0 siblings, 1 reply; 5+ messages in thread From: Raghav Gururajan @ 2019-12-30 18:41 UTC (permalink / raw) To: Jan Nieuwenhuizen; +Cc: 38811 [-- 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 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* bug#38811: gnu: Add gnome-menus. 2019-12-30 18:41 ` Raghav Gururajan @ 2019-12-30 20:02 ` Jan Nieuwenhuizen 0 siblings, 0 replies; 5+ messages in thread From: Jan Nieuwenhuizen @ 2019-12-30 20:02 UTC (permalink / raw) To: Raghav Gururajan; +Cc: 38811-done 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 ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2019-12-30 20:03 UTC | newest] Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-12-30 12:52 [bug#38811] gnu: Add gnome-menus Raghav Gururajan 2019-12-30 12:55 ` Raghav Gururajan 2019-12-30 13:21 ` Jan Nieuwenhuizen 2019-12-30 18:41 ` Raghav Gururajan 2019-12-30 20:02 ` bug#38811: " Jan Nieuwenhuizen
Code repositories for project(s) associated with this public inbox https://git.savannah.gnu.org/cgit/guix.git This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).