[PATCH] Add xtrlock

  • Done
  • quality assurance status badge
Details
3 participants
  • Ludovic Courtès
  • Maxim Cournoyer
  • Mehmet Tekman
Owner
unassigned
Submitted by
Mehmet Tekman
Severity
normal
M
M
Mehmet Tekman wrote on 7 Oct 2022 16:01
(address . guix-patches@gnu.org)
CAHHeYz+6XjU1YVCZJBsQ1KindfOHhLSDPH6LHPFd4qw4RWdbYA@mail.gmail.com
---
gnu/packages/xtrlock.scm | 71 ++++++++++++++++++++++++++++++++++++++++
1 file changed, 71 insertions(+)
create mode 100644 gnu/packages/xtrlock.scm

Toggle diff (81 lines)
diff --git a/gnu/packages/xtrlock.scm b/gnu/packages/xtrlock.scm
new file mode 100644
index 0000000000..2acf45e8b3
--- /dev/null
+++ b/gnu/packages/xtrlock.scm
@@ -0,0 +1,71 @@
+;;; GNU Guix --- Functional package management for GNU
+;;; Copyright © 2022 Mehmet Tekman <mtekman89@gmail.com>
+;;;
+;;; 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 xtrlock)
+ #:use-module (guix packages)
+ #:use-module (guix download)
+ #:use-module (guix build-system gnu)
+ #:use-module (gnu packages xorg)
+ #:use-module ((guix licenses) #:prefix license:)
+ )
+
+(define-public xtrlock
+ (let ((pkg-version "2.15")
+ (pkg-hash "0mgpysbvipd5h6x6zz4hng6b13gp3qjnpgny3azyj8k8dv85bppw")
+ (recipe-revision "0"))
+ (package
+ (name "xtrlock")
+ (version (string-append pkg-version ".r" recipe-revision))
+ (source (origin
+ (method url-fetch)
+ (uri (string-append
+ "mirror://debian/pool/main/x/xtrlock/xtrlock_"
+ pkg-version ".tar.xz"))
+ (sha256 (base32 pkg-hash))))
+ (build-system gnu-build-system)
+ (arguments
+ '(#:make-flags
+ (list "LDLIBS=-lX11 -lcrypt -lXi"
+ "CFLAGS=-Wall -DSHADOW_PWD -DMULTITOUCH")
+ #:phases
+ (modify-phases
+ %standard-phases
+ (delete 'configure)
+ (delete 'check)
+ (add-after 'unpack 'rename-makefile
+ (lambda _
+ (rename-file "Makefile.noimake" "Makefile")
+ (rename-file "xtrlock.man" "xtrlock.1")))
+ (replace 'install
+ (lambda* (#:key outputs #:allow-other-keys)
+ (let* ((out (assoc-ref outputs "out")))
+ (install-file "xtrlock" (string-append out "/bin/"))
+ (install-file "xtrlock.1" (string-append out
"/share/man/man1/"))))))))
+ (native-inputs (list libx11 libxi libxfixes))
+ (home-page "https://packages.debian.org/sid/xtrlock")
+ (synopsis "Minimal X display lock program")
+ (description "xtrlock locks the X server till the user enters
their password at the
+keyboard. While xtrlock is running, the mouse and keyboard are grabbed and
+the mouse cursor becomes a padlock. Output displayed by X programs, and
+windows put up by new X clients, continue to be visible, and any new output is
+displayed normally. The mouse and keyboard are returned when the user types
+their password, followed by Enter or Newline. If an incorrect password is
+entered the bell is sounded. Pressing Backspace or Delete erases one
+character of a password partially typed; pressing Escape or Clear clears
+anything that has been entered.")
+ (license license:gpl3))))
--
2.38.0
M
M
Maxim Cournoyer wrote on 8 Oct 2022 07:08
(name . Mehmet Tekman)(address . mtekman89@gmail.com)(address . 58356@debbugs.gnu.org)
87czb2297f.fsf@gmail.com
Hi Mehmet,

Mehmet Tekman <mtekman89@gmail.com> writes:

Toggle quote (5 lines)
> ---
> gnu/packages/xtrlock.scm | 71 ++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 71 insertions(+)
> create mode 100644 gnu/packages/xtrlock.scm

I'd put this into (gnu packages xdisorg), next to xlockmore :-).

Toggle quote (32 lines)
> diff --git a/gnu/packages/xtrlock.scm b/gnu/packages/xtrlock.scm
> new file mode 100644
> index 0000000000..2acf45e8b3
> --- /dev/null
> +++ b/gnu/packages/xtrlock.scm
> @@ -0,0 +1,71 @@
> +;;; GNU Guix --- Functional package management for GNU
> +;;; Copyright © 2022 Mehmet Tekman <mtekman89@gmail.com>
> +;;;
> +;;; 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 xtrlock)
> + #:use-module (guix packages)
> + #:use-module (guix download)
> + #:use-module (guix build-system gnu)
> + #:use-module (gnu packages xorg)
> + #:use-module ((guix licenses) #:prefix license:)
> + )

^ lonely parens, and please sort lexicographically.

Toggle quote (19 lines)
> +(define-public xtrlock
> + (let ((pkg-version "2.15")
> + (pkg-hash "0mgpysbvipd5h6x6zz4hng6b13gp3qjnpgny3azyj8k8dv85bppw")
> + (recipe-revision "0"))
> + (package
> + (name "xtrlock")
> + (version (string-append pkg-version ".r" recipe-revision))
> + (source (origin
> + (method url-fetch)
> + (uri (string-append
> + "mirror://debian/pool/main/x/xtrlock/xtrlock_"
> + pkg-version ".tar.xz"))
> + (sha256 (base32 pkg-hash))))
> + (build-system gnu-build-system)
> + (arguments
> + '(#:make-flags
> + (list "LDLIBS=-lX11 -lcrypt -lXi"
> + "CFLAGS=-Wall -DSHADOW_PWD -DMULTITOUCH")

Does the build system not take care of the libraries link options
itself?

Toggle quote (4 lines)
> + #:phases
> + (modify-phases
> + %standard-phases

Nitpick: no need to put %standard-phases on a new line.

Toggle quote (13 lines)
> + (delete 'configure)
> + (delete 'check)
> + (add-after 'unpack 'rename-makefile
> + (lambda _
> + (rename-file "Makefile.noimake" "Makefile")
> + (rename-file "xtrlock.man" "xtrlock.1")))
> + (replace 'install
> + (lambda* (#:key outputs #:allow-other-keys)
> + (let* ((out (assoc-ref outputs "out")))
> + (install-file "xtrlock" (string-append out "/bin/"))
> + (install-file "xtrlock.1" (string-append out
> "/share/man/man1/"))))))))

Use 'guix style' to have it auto-formatted.

Toggle quote (3 lines)
> + (native-inputs (list libx11 libxi libxfixes))
> + (home-page "https://packages.debian.org/sid/xtrlock")

There must be a better home page for the upstream of xtrlock than
Debian? Edit: apparently Debian is the upstream. Odd!

Toggle quote (5 lines)
> + (synopsis "Minimal X display lock program")
> + (description "xtrlock locks the X server till the user enters
> their password at the
> +keyboard. While xtrlock is running, the mouse and keyboard are grabbed and

Please reflow the paragraph; M-j in Emacs for example.

Toggle quote (9 lines)
> +the mouse cursor becomes a padlock. Output displayed by X programs, and
> +windows put up by new X clients, continue to be visible, and any new output is
> +displayed normally. The mouse and keyboard are returned when the user types
> +their password, followed by Enter or Newline. If an incorrect password is
> +entered the bell is sounded. Pressing Backspace or Delete erases one
> +character of a password partially typed; pressing Escape or Clear clears
> +anything that has been entered.")
> + (license license:gpl3))))

This is gpl3+ (GPL 3 or later). See the license at
contains the "or any later version" text.

Could you please send a v2? You could use something like: "git
send-email -v2 --to='58356@debbugs.gnu.org' -1" to send the tip of your
branch.

--
Thanks,
Maxim
M
M
Maxim Cournoyer wrote on 8 Oct 2022 07:09
control message for bug #58356
(address . control@debbugs.gnu.org)
87bkqm2976.fsf@gmail.com
tags 58356 + moreinfo
quit
M
M
Mehmet Tekman wrote on 11 Oct 2022 12:45
Re: bug#58356: [PATCH] Add xtrlock
(address . 58356@debbugs.gnu.org)
CAHHeYzKeXMUOYa_rPyKuV6h9JM6Gt=i_+VpdQKTpiMsPa8wVEw@mail.gmail.com
Hello, thanks again for the review!

The following has been changed:
* Placed into xdisorg.scm just after the xlockmore recipe
* Removed the CFLAGS, but kept the LDLIBS since it won't build without them
* General formatting (lint, style)
* License fix

I found it easier to spawn a new branch off master and create a new
patch than to append to the existing branch. I hope that's okay. The
patch is attached.

Best,
Mehmet
M
M
Mehmet Tekman wrote on 11 Oct 2022 13:27
(address . 58356@debbugs.gnu.org)
CAHHeYzLRAA6ohu2KPNwfkjJ=KzKLd6uEakZzU3ARBzOegpdnCg@mail.gmail.com
I've just noticed that though the package builds without the CFLAGS,
it doesn't work well in context to the shadow file.
I've reincluded it in this next patch (diff from master).

Best,
Mehmet
From aced8b924a194495233b0698f2185b9b80d10963 Mon Sep 17 00:00:00 2001
From: Mehmet Tekman <mtekman89@gmail.com>
Date: Tue, 11 Oct 2022 12:38:13 +0200
Subject: [PATCH] gnu: Add xtrlock.

* gnu/packages/xdisorg.scm (xtrlock): New variable.
---
gnu/packages/xdisorg.scm | 49 ++++++++++++++++++++++++++++++++++++++++
1 file changed, 49 insertions(+)

Toggle diff (69 lines)
diff --git a/gnu/packages/xdisorg.scm b/gnu/packages/xdisorg.scm
index 9908f29191..fc25c14ea7 100644
--- a/gnu/packages/xdisorg.scm
+++ b/gnu/packages/xdisorg.scm
@@ -56,6 +56,7 @@
;;; Copyright © 2022 Jai Vetrivelan <jaivetrivelan@gmail.com>
;;; Copyright © 2022 Derek Chuank <derekchuank@outlook.com>
;;; Copyright © 2022 Wamm K. D. <jaft.r@outlook.com>
+;;; Copyright © 2022 Mehmet Tekman <mtekman89@gmail.com>
;;;
;;; This file is part of GNU Guix.
;;;
@@ -1099,6 +1100,54 @@ (define-public xlockmore
(license (license:non-copyleft #f "See xlock.c.")
))) ; + GPLv2 in modes/glx/biof.c.
+(define-public xtrlock
+ (package
+ (name "xtrlock")
+ (version "2.15")
+ (source (origin
+ (method url-fetch)
+ (uri (string-append
+ "mirror://debian/pool/main/x/xtrlock/xtrlock_" version
+ ".tar.xz"))
+ (sha256
+ (base32
+ "0mgpysbvipd5h6x6zz4hng6b13gp3qjnpgny3azyj8k8dv85bppw"))))
+ (build-system gnu-build-system)
+ (arguments
+ ;; LDLIBS are required for the package to build.
+ ;; CFLAGS are required for it to use the shadow file, and to
+ ;; be compatible with multi-touch devices.
+ '(#:make-flags (list "CFLAGS=-Wall -DSHADOW_PWD -DMULTITOUCH"
+ "LDLIBS=-lX11 -lcrypt -lXi")
+ #:phases (modify-phases %standard-phases
+ (delete 'configure)
+ (delete 'check)
+ (add-after 'unpack 'rename-makefile
+ (lambda _
+ (rename-file "Makefile.noimake" "Makefile")
+ (rename-file "xtrlock.man" "xtrlock.1")))
+ (replace 'install
+ (lambda* (#:key outputs #:allow-other-keys)
+ (let* ((out (assoc-ref outputs "out")))
+ (install-file "xtrlock"
+ (string-append out "/bin/"))
+ (install-file "xtrlock.1"
+ (string-append out "/share/man/man1/"))))))))
+ (inputs (list libx11 libxi libxfixes))
+ (home-page "https://packages.debian.org/sid/xtrlock")
+ (synopsis "Minimal X display lock program")
+ (description
+ "xtrlock locks the X server till the user enters their password at the
+keyboard. While xtrlock is running, the mouse and keyboard are grabbed and
+the mouse cursor becomes a padlock. Output displayed by X programs, and
+windows put up by new X clients, continue to be visible, and any new output is
+displayed normally. The mouse and keyboard are returned when the user types
+their password, followed by Enter or Newline. If an incorrect password is
+entered the bell is sounded. Pressing Backspace or Delete erases one
+character of a password partially typed; pressing Escape or Clear clears
+anything that has been entered.")
+ (license license:gpl3+)))
+
(define-public xosd
(package
(name "xosd")
--
2.38.0
L
L
Ludovic Courtès wrote on 17 Aug 2023 19:04
(name . Mehmet Tekman)(address . mtekman89@gmail.com)(address . 58356-done@debbugs.gnu.org)
87il9dvbvw.fsf_-_@gnu.org
Hi,

Mehmet Tekman <mtekman89@gmail.com> skribis:

Toggle quote (7 lines)
> From aced8b924a194495233b0698f2185b9b80d10963 Mon Sep 17 00:00:00 2001
> From: Mehmet Tekman <mtekman89@gmail.com>
> Date: Tue, 11 Oct 2022 12:38:13 +0200
> Subject: [PATCH] gnu: Add xtrlock.
>
> * gnu/packages/xdisorg.scm (xtrlock): New variable.

Finally applied, with the changes below.

Thanks!

Ludo’.
Toggle diff (55 lines)
diff --git a/gnu/packages/xdisorg.scm b/gnu/packages/xdisorg.scm
index e4049a5dc2..294a18cdd2 100644
--- a/gnu/packages/xdisorg.scm
+++ b/gnu/packages/xdisorg.scm
@@ -1164,22 +1164,25 @@ (define-public xtrlock
;; LDLIBS are required for the package to build.
;; CFLAGS are required for it to use the shadow file, and to
;; be compatible with multi-touch devices.
- '(#:make-flags (list "CFLAGS=-Wall -DSHADOW_PWD -DMULTITOUCH"
- "LDLIBS=-lX11 -lcrypt -lXi")
- #:phases (modify-phases %standard-phases
- (delete 'configure)
- (delete 'check)
- (add-after 'unpack 'rename-makefile
- (lambda _
- (rename-file "Makefile.noimake" "Makefile")
- (rename-file "xtrlock.man" "xtrlock.1")))
- (replace 'install
- (lambda* (#:key outputs #:allow-other-keys)
- (let* ((out (assoc-ref outputs "out")))
- (install-file "xtrlock"
- (string-append out "/bin/"))
- (install-file "xtrlock.1"
- (string-append out "/share/man/man1/"))))))))
+ (list #:make-flags
+ #~'("CFLAGS=-O2 -g -Wall -DSHADOW_PWD -DMULTITOUCH"
+ "LDLIBS=-lX11 -lcrypt -lXi")
+ #:phases
+ #~(modify-phases %standard-phases
+ (delete 'configure)
+ (delete 'check)
+ (add-after 'unpack 'rename-makefile
+ (lambda _
+ (rename-file "Makefile.noimake" "Makefile")
+ (rename-file "xtrlock.man" "xtrlock.1")))
+ (replace 'install
+ (lambda* (#:key outputs #:allow-other-keys)
+ (let* ((out (assoc-ref outputs "out")))
+ (install-file "xtrlock"
+ (string-append out "/bin/"))
+ (install-file "xtrlock.1"
+ (string-append out
+ "/share/man/man1/"))))))))
(inputs (list libx11 libxi libxfixes))
(home-page "https://packages.debian.org/sid/xtrlock")
(synopsis "Minimal X display lock program")
@@ -1193,7 +1196,7 @@ (define-public xtrlock
entered the bell is sounded. Pressing Backspace or Delete erases one
character of a password partially typed; pressing Escape or Clear clears
anything that has been entered.")
- (license license:gpl3+)))
+ (license license:gpl2+)))
(define-public xosd
(package
Closed
?
Your comment

This issue is archived.

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

To respond to this issue using the mumi CLI, first switch to it
mumi current 58356
Then, you may apply the latest patchset in this issue (with sign off)
mumi am -- -s
Or, compose a reply to this issue
mumi compose
Or, send patches to this issue
mumi send-email *.patch