From patchwork Fri Apr 24 19:23:02 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Raghav Gururajan X-Patchwork-Id: 21455 Return-Path: X-Original-To: patchwork@mira.cbaines.net Delivered-To: patchwork@mira.cbaines.net Received: by mira.cbaines.net (Postfix, from userid 113) id C232127BBE4; Fri, 24 Apr 2020 20:29:58 +0100 (BST) X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on mira.cbaines.net X-Spam-Level: X-Spam-Status: No, score=-2.8 required=5.0 tests=BAYES_00,DKIM_SIGNED, MAILING_LIST_MULTI,T_DKIM_INVALID,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.2 Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mira.cbaines.net (Postfix) with ESMTP id 430EE27BBE1 for ; Fri, 24 Apr 2020 20:29:58 +0100 (BST) Received: from localhost ([::1]:50472 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1jS41N-00075S-N1 for patchwork@mira.cbaines.net; Fri, 24 Apr 2020 15:29:57 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:53498) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1jS3vf-0003fM-4f for guix-patches@gnu.org; Fri, 24 Apr 2020 15:24:03 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.90_1) (envelope-from ) id 1jS3ve-0007Na-Ac for guix-patches@gnu.org; Fri, 24 Apr 2020 15:24:02 -0400 Received: from debbugs.gnu.org ([209.51.188.43]:46423) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1jS3vd-0007NH-TD for guix-patches@gnu.org; Fri, 24 Apr 2020 15:24:01 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1jS3vd-0005Bn-MD for guix-patches@gnu.org; Fri, 24 Apr 2020 15:24:01 -0400 X-Loop: help-debbugs@gnu.org Subject: [bug#40756] gnu: Add ktsuss. (v3) Resent-From: Raghav Gururajan Original-Sender: "Debbugs-submit" Resent-CC: guix-patches@gnu.org Resent-Date: Fri, 24 Apr 2020 19:24:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 40756 X-GNU-PR-Package: guix-patches X-GNU-PR-Keywords: To: Nicolas Goaziou Cc: 40756@debbugs.gnu.org Received: via spool by 40756-submit@debbugs.gnu.org id=B40756.158775620019896 (code B ref 40756); Fri, 24 Apr 2020 19:24:01 +0000 Received: (at 40756) by debbugs.gnu.org; 24 Apr 2020 19:23:20 +0000 Received: from localhost ([127.0.0.1]:57969 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1jS3ux-0005Aq-Jq for submit@debbugs.gnu.org; Fri, 24 Apr 2020 15:23:19 -0400 Received: from knopi.disroot.org ([178.21.23.139]:55674) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1jS3uv-0005Ai-PB for 40756@debbugs.gnu.org; Fri, 24 Apr 2020 15:23:18 -0400 Received: from localhost (localhost [127.0.0.1]) by disroot.org (Postfix) with ESMTP id 1A08E26B95; Fri, 24 Apr 2020 21:23:17 +0200 (CEST) X-Virus-Scanned: Debian amavisd-new at disroot.org Received: from knopi.disroot.org ([127.0.0.1]) by localhost (disroot.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id OxQb3nED8QAJ; Fri, 24 Apr 2020 21:23:15 +0200 (CEST) Date: Fri, 24 Apr 2020 15:23:02 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=disroot.org; s=mail; t=1587756195; bh=BCM9O2O59wmYVx9QWal+TxiXFWHEMp497dSkYHrbtSo=; h=Date:From:To:Cc:Subject:In-Reply-To:References; b=irSmbJvfVu6/QBWShAzKkKLs3bqXYAydrORKzngMtOoJLzmSQxZ0Vu1DbZofo6zh2 VaYaam27aIdB8kwDyFTzScJ+suLnDiorY44C+Tfuyxm16oxchTAR9hbbI1fx3creGO +mwXVjEZTnUq+4W9M7vpZgR4Ro0G9F4EWWtpT+DXwvr+/MmdfAKqFsMhVTX3dovfwT J9pp5FEq94GgfmNkYMcafYCJf7yaXQRzt8zgigLKteBUJV1DgfdMGHb8zWOCR8bN8J gs0VdsvaM5k8h56/FQ/Oj7iAEy4zllRqWr8cCYS8Q9jY1mOK7Qfzl/lpujfkH5fQ1q REq1+Dkd37qSg== From: Raghav Gururajan Message-ID: <20200424152302.608f23c2.raghavgururajan@disroot.org> In-Reply-To: <20200424152131.62ac18f1.raghavgururajan@disroot.org> References: <20200421192622.7d6412e0.raghavgururajan@disroot.org> <871roeliph.fsf@nicolasgoaziou.fr> <20200424152131.62ac18f1.raghavgururajan@disroot.org> Mime-Version: 1.0 X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-Received-From: 209.51.188.43 X-BeenThere: guix-patches@gnu.org List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+patchwork=mira.cbaines.net@gnu.org Sender: "Guix-patches" X-getmail-retrieved-from-mailbox: Patches On Fri, 24 Apr 2020 15:21:31 -0400 Raghav Gururajan wrote: > Hi Nicolas! > > > > Subject: [PATCH 4/4] gnu: Add ktsuss. > > > > Thank you! > > > > It looks mostly good. I have some comments, tho. > > > > > + (modify-phases %standard-phases > > > + (add-after 'unpack 'patch > > > > Could you provide a more insightful name for the new phase? :) > > > > > + (lambda _ > > > + (substitute* "configure.ac" > > > + (("sudopath=`which sudo 2>/dev/null`") > > > + (string-append "sudopath=" > > > + (string-append (assoc-ref %build-inputs > > > "sudo") > > > + "/bin/sudo")))) > > > > Please use (lambda* (#:key inputs #:allow-other-keys) ...) and `inputs' > > instead of the magical %build-inputs. > > > > > + (synopsis "Graphical front-end for Su") > > > > I would write: > > > > Graphical front end for @command{su} > > > > > + (description "Ktsuss stands for 'keep the su simple, stupid', and as > > > the +name says, is a graphical version (frontend) of su written in C and > > > GTK+ 2. The +idea of the project is to remain simple and bug free.") > > > > What about : > > > > (description > > "Ktsuss stands for ``Keep the @command{su} simple, stupid''. > > It is a graphical version of @command{su} written in C and GTK+ 2, > > with simplicity in mind.") > > > > > + (license > > > + (list > > > + (license:non-copyleft "file://COPYING") > > > + (license:non-copyleft "file://LICENSE"))))) > > > > It looks like the license is `bsd-3'. Do you have any reason to think > > otherwise ? > > > > Could you send an updated patch? > > Thank you for the feedback. Please find the updated patch attached with this > email. > > Regards, > RG. From 4a54ba65fbb201fc346af8c4580ce2991ea8e68d Mon Sep 17 00:00:00 2001 From: Raghav Gururajan Date: Fri, 24 Apr 2020 15:18:59 -0400 Subject: [PATCH] gnu: Add ktsuss. * gnu/packages/admin.scm (ktsuss): New variable. --- gnu/packages/admin.scm | 49 ++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 49 insertions(+) diff --git a/gnu/packages/admin.scm b/gnu/packages/admin.scm index 619a419170..80c2e05336 100644 --- a/gnu/packages/admin.scm +++ b/gnu/packages/admin.scm @@ -52,6 +52,7 @@ (define-module (gnu packages admin) #:use-module (guix build-system cmake) #:use-module (guix build-system emacs) + #:use-module (guix build-system glib-or-gtk) #:use-module (guix build-system gnu) #:use-module (guix build-system meson) #:use-module (guix build-system perl) @@ -64,6 +65,7 @@ #:use-module (guix utils) #:use-module (gnu packages) #:use-module (gnu packages algebra) + #:use-module (gnu packages autogen) #:use-module (gnu packages autotools) #:use-module (gnu packages base) #:use-module (gnu packages bash) @@ -128,6 +130,53 @@ #:use-module (gnu packages xml) #:use-module (gnu packages xorg)) +(define-public ktsuss + (package + (name "ktsuss") + (version "2.1") + (source + (origin + (method git-fetch) + (uri + (git-reference + (url "https://github.com/nomius/ktsuss.git") + (commit version))) + (sha256 + (base32 "0q9931f9hp47v1n8scli4bdg2rkjpf5jf8v7jj2gdn83aia1r2hz")) + (file-name (git-file-name name version)))) + (build-system glib-or-gtk-build-system) + (arguments + `(#:configure-flags + (list + "--enable-sudo=yes") + #:phases + (modify-phases %standard-phases + (add-after 'unpack 'patch-sudo-path + (lambda* (#:key inputs #:allow-other-keys) + (substitute* "configure.ac" + (("sudopath=`which sudo 2>/dev/null`") + (string-append "sudopath=" + (string-append (assoc-ref inputs "sudo") + "/bin/sudo")))) + #t))))) + (native-inputs + `(("autoconf" ,autoconf) + ("autogen" ,autogen) + ("automake" ,automake) + ("libtool" ,libtool) + ("pkg-config" ,pkg-config))) + (inputs + `(("glib" ,glib) + ("gtk+" ,gtk+-2) + ("sudo" ,sudo))) + (synopsis "Graphical front end for @command{su}") + (description "Ktsuss stands for 'keep the @command{su} simple, stupid', +and as the name says, is a graphical version (frontend) of @command{su} +written in C and GTK+ 2. The idea of the project is to remain simple and +bug free.") + (home-page "https://github.com/nomius/ktsuss") + (license license:bsd-3))) + (define-public aide (package (name "aide") -- 2.26.1