From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:470:142:3::10]:57118) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1jFNR6-0002xS-O9 for guix-patches@gnu.org; Fri, 20 Mar 2020 15:36:06 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1jFNR5-0000Uq-8N for guix-patches@gnu.org; Fri, 20 Mar 2020 15:36:04 -0400 Received: from debbugs.gnu.org ([209.51.188.43]:39712) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1jFNR4-0000UJ-Pf for guix-patches@gnu.org; Fri, 20 Mar 2020 15:36:03 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1jFNR3-00086P-MR for guix-patches@gnu.org; Fri, 20 Mar 2020 15:36:01 -0400 Subject: [bug#40102] [PATCH] gnu packages gnome-xyz: Add topicons-redux Resent-Message-ID: Date: Fri, 20 Mar 2020 19:34:51 +0000 From: Ekaitz Zarraga Message-ID: In-Reply-To: <87fte29al1.fsf@devup.no> References: <87fte29al1.fsf@devup.no> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: Ekaitz Zarraga Errors-To: guix-patches-bounces+kyle=kyleam.com@gnu.org Sender: "Guix-patches" To: Marius Bakke Cc: "40102\\@debbugs.gnu.org" <40102@debbugs.gnu.org>, Leo Prikler Hi all, Thanks Marius for the feedback. =E2=80=90=E2=80=90=E2=80=90=E2=80=90=E2=80=90=E2=80=90=E2=80=90 Original Me= ssage =E2=80=90=E2=80=90=E2=80=90=E2=80=90=E2=80=90=E2=80=90=E2=80=90 On Friday, March 20, 2020 5:53 PM, Marius Bakke wrote= : > Such "autogenerated" tarballs from GitLab and GitHub are prone to > getting re-created with different time stamps, breaking the checksum. > Can you adjust this package to use git-fetch instead of url-fetch? I wasn't aware of that! Thank you very much. > Why are tests disabled? Please leave a comment. Project doesn't have any test. > Indentation is off here. If you are not using Emacs there is a script > that can help with indentation, see "Formatting Code" in the Guix manual. I'm using Vim with Vim paredit and it's own Sheme filetype, but it doesn't work very well sometimes. > > The line break here is not necessary. Solved > > - (synopsis "Displays legacy tray icons in the GNOME Shell top panel. > > +Rewritten fork of TopIcons Plus.") > > > > Please keep synopsis to single sentences. The second sentence can be > removed IMO. Done. > 'guix lint gnome-shell-extension-topicons-redux' will warn that > sentences should be separated by two spaces. Can you adjust accordingly? Done. > > - (license license:gpl2))) > > Is the license really GPL2 only? Unless the source code says explicitly > "GPL2 only", this should probably be "gpl2+", i.e. "GPL2 or any later > version". Done. > > Can you send an updated patch? > Here it comes: >From e62251601732587409c18dfe3af0cd56ecf39d5f Mon Sep 17 00:00:00 2001 From: Ekaitz Zarraga Date: Tue, 17 Mar 2020 13:54:24 +0100 Subject: [PATCH] gnu packages gnome-xyz: Add topicons-redux --- gnu/packages/gnome-xyz.scm | 43 ++++++++++++++++++++++++++++++++++++++ 1 file changed, 43 insertions(+) diff --git a/gnu/packages/gnome-xyz.scm b/gnu/packages/gnome-xyz.scm index b8cd32db08..39579bbea4 100644 --- a/gnu/packages/gnome-xyz.scm +++ b/gnu/packages/gnome-xyz.scm @@ -4,6 +4,7 @@ ;;; Copyright =C2=A9 2019 Giacomo Leidi ;;; Copyright =C2=A9 2020 Alex Griffin ;;; Copyright =C2=A9 2020 Jack Hill +;;; Copyright =C2=A9 2020 Ekaitz Zarraga ;;; ;;; This file is part of GNU Guix. ;;; @@ -146,6 +147,48 @@ GNOME Shell.") (home-page "https://github.com/ubuntu/gnome-shell-extension-appindicat= or/") (license license:gpl2+))) +(define-public gnome-shell-extension-topicons-redux + (package + (name "gnome-shell-extension-topicons-redux") + (version "6") + (source + (origin + (method git-fetch) + (uri (git-reference + (url "https://gitlab.com/pop-planet/TopIcons-Redux.git") + (commit version))) + (sha256 + (base32 "1dli9xb545n3xlj6q4wl0y5gzkm903zs47p8fiq71pdvbr6v38rj"))= )) + (build-system gnu-build-system) + (native-inputs + `(("glib" ,glib "bin"))) + (arguments + `(#:tests? #f ; No test defined in the project + #:phases + (modify-phases + %standard-phases + (delete 'configure) + (delete 'build) + (replace 'install + (lambda* (#:key outputs #:allow-other-keys) + (let ((out (assoc-ref outputs "out"))) + (invoke "make" + "install" + (string-append + "INSTALL_PATH=3D" + out + "/share/gnome-shell/extensions")))))))) + + (home-page "https://gitlab.com/pop-planet/TopIcons-Redux/") + (synopsis "Displays legacy tray icons in the GNOME Shell top panel.") + (description "Many applications, such as chat clients, downloaders, an= d +some media players, are meant to run long-term in the background even afte= r you +close their window. These applications remain accessible by adding an ico= n to +the GNOME Shell Legacy Tray. However, the Legacy Tray was removed in GNOM= E +3.26. TopIcons Redux brings those icons back into the top panel so that i= t's +easier to keep track of apps running in the backround.") + (license license:gpl2+))) + (define-public gnome-shell-extension-dash-to-dock (package (name "gnome-shell-extension-dash-to-dock") -- 2.25.1