[PATCH] Add makepasswd package

Jean-Baptiste Note wrote on 2 May 2020 22:37

Recipients:(name . guixpatch)(address . guix-patches@gnu.org)

Message-ID:87zhaq8453.fsf@m4x.org

Dear Guix maintainers,
Here is an attempt at packaging makepasswd, a run-time dependency ofx2go.
I've placed it in its own file which may be ill-advised for now; pleaselet me know where I could more sensibly move it.
Hopefully patches will follow that put more flesh in this x2go.scm file;however I find this program useful in its own right.
Kind regards,Jean-Baptiste

From f8c9c93e359f4f65a12709f244e6e8952db4bad9 Mon Sep 17 00:00:00 2001
* gnu/packages/x2go.scm: New file.
* gnu/packages/xorg.scm (makepasswd): New variable.--- gnu/packages/x2go.scm | 66 +++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 66 insertions(+) create mode 100644 gnu/packages/x2go.scm

Toggle diff (74 lines)diff --git a/gnu/packages/x2go.scm b/gnu/packages/x2go.scmnew file mode 100644index 0000000000..b77f81cee5--- /dev/null+++ b/gnu/packages/x2go.scm@@ -0,0 +1,66 @@+;;; GNU Guix --- Functional package management for GNU+;;;+;;; This file is part of GNU Guix.+;;;+;;; GNU Guix is free software; you can redistribute it and/or modify it+;;; under the terms of the GNU General Public License as published by+;;; the Free Software Foundation; either version 3 of the License, or (at+;;; your option) any later version.+;;;+;;; GNU Guix is distributed in the hope that it will be useful, but+;;; WITHOUT ANY WARRANTY; without even the implied warranty of+;;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the+;;; GNU General Public License for more details.+;;;+;;; You should have received a copy of the GNU General Public License+;;; along with GNU Guix.  If not, see <http://www.gnu.org/licenses/>.++(define-module (gnu packages x2go)+  #:use-module ((guix licenses) #:prefix license:)+  #:use-module (guix packages)+  #:use-module (guix git-download)+  #:use-module (guix build-system gnu)+  #:use-module (guix utils)+  #:use-module (gnu packages)+  #:use-module (gnu packages pkg-config)+  #:use-module (gnu packages tls)+  #:use-module (gnu packages xml)+  #:use-module (gnu packages docbook))++(define-public makepasswd+  (let ((commit "3545d57d3a589a392d7eb0df36a5286785345c9e")+        (revision "1"))+    (package+      (name "makepasswd")+      (version (git-version "0.5.4" revision commit))+      (source+       (origin+         (method git-fetch)+         (uri (git-reference+               (url "https://github.com/khorben/makepasswd")+               (commit commit)))+         (file-name (git-file-name name version))+         (sha256+          (base32 "0lspqyyxbk6h28yxnp7pd5aib161vrkzgasam5jpzn35n1jacx2j"))))+      (build-system gnu-build-system)+      (native-inputs+       `(("pkg-config" ,pkg-config)+         ("libxslt" ,libxslt)+         ("libxml2" ,libxml2)+         ("docbook-xsl" ,docbook-xsl)+         ("docbook-xml" ,docbook-xml)))+      (inputs+       `(("openssl" ,openssl)))+      (arguments+       `(#:phases+         (modify-phases %standard-phases+           (delete 'configure))+         #:make-flags (list "CC=gcc" (string-append "PREFIX=" (assoc-ref %outputs "out")))+         #:tests? #f))+      (synopsis "Generate (pseudo-)random passwords and hashes")+      (description+       "Makepasswd is a program that generates pseudo-random passwords of a+desired length.  It can also generate their corresponding hashes for a given+encryption algorithm if so desired.")+      (home-page "https://github.com/khorben/makepasswd")+      (license license:gpl3))))-- 2.26.2

Marius Bakke wrote on 5 May 2020 18:49

Recipients:

Message-ID:87bln2z5qg.fsf@devup.no

Jean-Baptiste Note <jean-baptiste.note@m4x.org> writes:

Toggle quote (8 lines)

> Dear Guix maintainers,>> Here is an attempt at packaging makepasswd, a run-time dependency of> x2go.>> I've placed it in its own file which may be ill-advised for now; please> let me know where I could more sensibly move it.

Perhaps password-utils.scm could work for this package?

Jean-Baptiste Note wrote on 5 May 2020 20:47

Recipients:(name . Marius Bakke)(address . mbakke@fastmail.com)(address . 41032@debbugs.gnu.org)

Message-ID:87zhamw75k.fsf@m4x.org

Hi Marius,
Thanks for your review and the heads up on the correct file.Please find an updated patch.
I've used guix-devel-mode and ran indent-according-to-mode on the bufferbefore producing the patch; let me know if this is not sufficient tocorrectly indent the code :)
Kind regards,Jean-Baptiste

From 3d8786e2b818e28e79fc62eb6872bea1400a1d24 Mon Sep 17 00:00:00 2001
* gnu/packages/password-utils.scm (makepasswd): New variable.--- gnu/packages/password-utils.scm | 39 +++++++++++++++++++++++++++++++++ 1 file changed, 39 insertions(+)

Toggle diff (56 lines)diff --git a/gnu/packages/password-utils.scm b/gnu/packages/password-utils.scmindex 2a570691a4..59c7befddc 100644--- a/gnu/packages/password-utils.scm+++ b/gnu/packages/password-utils.scm@@ -61,6 +61,7 @@   #:use-module (gnu packages crypto)   #:use-module (gnu packages cryptsetup)   #:use-module (gnu packages curl)+  #:use-module (gnu packages docbook)   #:use-module (gnu packages file)   #:use-module (gnu packages freedesktop)   #:use-module (gnu packages gettext)@@ -1168,3 +1169,41 @@ exhaustive mode to try every password given a charset or in dictionary mode to try every password contained in a file.")     (home-page "https://github.com/glv2/bruteforce-luks")     (license license:gpl3+)))++(define-public makepasswd+  (let ((commit "3545d57d3a589a392d7eb0df36a5286785345c9e")+        (revision "1"))+    (package+      (name "makepasswd")+      (version (git-version "0.5.4" revision commit))+      (source+       (origin+         (method git-fetch)+         (uri (git-reference+               (url "https://github.com/khorben/makepasswd")+               (commit commit)))+         (file-name (git-file-name name version))+         (sha256+          (base32 "0lspqyyxbk6h28yxnp7pd5aib161vrkzgasam5jpzn35n1jacx2j"))))+      (build-system gnu-build-system)+      (native-inputs+       `(("pkg-config" ,pkg-config)+         ("libxslt" ,libxslt)+         ("libxml2" ,libxml2)+         ("docbook-xsl" ,docbook-xsl)+         ("docbook-xml" ,docbook-xml)))+      (inputs+       `(("openssl" ,openssl)))+      (arguments+       `(#:phases+         (modify-phases %standard-phases+           (delete 'configure))+         #:make-flags (list "CC=gcc" (string-append "PREFIX=" (assoc-ref %outputs "out")))+         #:tests? #f))+      (synopsis "Generate (pseudo-)random passwords and hashes")+      (description+       "Makepasswd is a program that generates pseudo-random passwords of a+desired length.  It can also generate their corresponding hashes for a given+encryption algorithm if so desired.")+      (home-page "https://github.com/khorben/makepasswd")+      (license license:gpl3))))-- 2.26.2

Marius Bakke wrote on 6 May 2020 22:29

Recipients:(name . Jean-Baptiste Note)(address . jean-baptiste.note@m4x.org)(address . 41032-done@debbugs.gnu.org)

Message-ID:877dxo4xix.fsf@devup.no

Jean-Baptiste Note <jean-baptiste.note@m4x.org> writes:

Toggle quote (9 lines)

> Hi Marius,>> Thanks for your review and the heads up on the correct file.> Please find an updated patch.>> I've used guix-devel-mode and ran indent-according-to-mode on the buffer> before producing the patch; let me know if this is not sufficient to> correctly indent the code :)

Thanks! My Emacs agrees, so LGTM.
[...]

Toggle quote (6 lines)

> + (arguments> + `(#:phases> + (modify-phases %standard-phases> + (delete 'configure))> + #:make-flags (list "CC=gcc" (string-append "PREFIX=" (assoc-ref %outputs "out")))

I moved the second flag to its own line as 89 characters is a bit much.

Toggle quote (2 lines)

> + #:tests? #f))

And added a comment about why tests are disabled.

Toggle quote (8 lines)

> + (synopsis "Generate (pseudo-)random passwords and hashes")> + (description> + "Makepasswd is a program that generates pseudo-random passwords of a> +desired length. It can also generate their corresponding hashes for a given> +encryption algorithm if so desired.")> + (home-page "https://github.com/khorben/makepasswd")> + (license license:gpl3))))

Indeed GPL3 only :-/
Applied!

Closed

Your comment

Commenting via the web interface is currently disabled.

To comment on this conversation send email to 41032@debbugs.gnu.org

is:open	open issues
is:done	closed issues
submitter:<who>	search issue submitter
author:<who>	search by message author
date:yesterday..now	search by issue date
mdate:3m..2d	search by message date