[PR PATCH] pinentry-dmenu: fix for musl-based installations

[PR PATCH] pinentry-dmenu: fix for musl-based installations

[ftrvxmtrx]

[-- Attachment #1: Type: text/plain, Size: 709 bytes --]

There is a new pull request by ftrvxmtrx against master on the void-packages repository

https://github.com/ftrvxmtrx/void-packages pinentry-dmenu-fix-on-musl
https://github.com/void-linux/void-packages/pull/35137

pinentry-dmenu: fix for musl-based installations
pinentry-dmenu without any user config would print to stdout which
would render the pipe useless - musl tries to find out whether stdout
is a terminal.

The patch sent upstream.

#### Testing the changes
- I tested the changes in this PR: **YES**

#### Local build testing
- I built this PR locally for my native architecture, (aarch64-musl)


A patch file from https://github.com/void-linux/void-packages/pull/35137.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-pinentry-dmenu-fix-on-musl-35137.patch --]
[-- Type: text/x-diff, Size: 3540 bytes --]

From 7ca9f8121f823ccf7e93cccfe878e717c8b55aeb Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Sigrid=20Solveig=20Hafl=C3=ADnud=C3=B3ttir?=
 <ftrvxmtrx@gmail.com>
Date: Thu, 20 Jan 2022 17:08:46 +0100
Subject: [PATCH] pinentry-dmenu: fix for musl-based installations

pinentry-dmenu without any user config would print to stdout which
would render the pipe useless - musl tries to find out whether stdout
is a terminal.

The patch sent upstream.
---
 srcpkgs/pinentry-dmenu/patches/musl.patch | 49 +++++++++++++++++++++++
 srcpkgs/pinentry-dmenu/template           |  2 +-
 2 files changed, 50 insertions(+), 1 deletion(-)
 create mode 100644 srcpkgs/pinentry-dmenu/patches/musl.patch

diff --git a/srcpkgs/pinentry-dmenu/patches/musl.patch b/srcpkgs/pinentry-dmenu/patches/musl.patch
new file mode 100644
index 000000000000..2f17a893a530
--- /dev/null
+++ b/srcpkgs/pinentry-dmenu/patches/musl.patch
@@ -0,0 +1,49 @@
+From dad5154a71b65c28217df4b55cc7a17b0a6b59cb Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?Sigrid=20Solveig=20Hafl=C3=ADnud=C3=B3ttir?=
+ <ftrvxmtrx@gmail.com>
+Date: Thu, 20 Jan 2022 16:44:25 +0100
+Subject: [PATCH] don't print "No config file found..." message
+
+On systems where musl is used as the libc, printf on pipe causes
+libc to check whether the output is a terminal (by calling ioctl
+TIOCGWINSZ) which results in broken pipe when an "OK" message is
+written. Creating an empty config file was the workaround, but
+it probably makes more sense to just disable the message altogether,
+as it's not supposed to be seen by the user regardless.
+
+	openat(AT_FDCWD, "/home/ftrvx/.gnupg/pinentry-dmenu.conf", O_RDONLY|O_LARGEFILE) = -1 ENOENT (No such file or directory)
+	ioctl(1, TIOCGWINSZ, 0x7ff2ba8a98)      = -1 ENOTTY (Not a tty)
+	writev(1, [{iov_base="", iov_len=0}, {iov_base="No config file found. Use defaul"..., iov_len=36}], 2) = 36
+	brk(NULL)                               = 0x558549b000
+	brk(0x55854a0000)                       = 0x55854a0000
+	getuid()                                = 1000
+	mlock(0x558549b020, 16384)              = 0
+	geteuid()                               = 1000
+	getuid()                                = 1000
+	geteuid()                               = 1000
+	getuid()                                = 1000
+	geteuid()                               = 1000
+	dup(0)                                  = 3
+	dup(1)                                  = 4
+	write(4, "OK Pleased to meet you, process "..., 37) = -1 EPIPE (Broken pipe)
+	--- SIGPIPE {si_signo=SIGPIPE, si_code=SI_USER, si_pid=25113, si_uid=1000} ---
+---
+ pinentry-dmenu.c | 2 --
+ 1 file changed, 2 deletions(-)
+
+diff --git a/pinentry-dmenu.c b/pinentry-dmenu.c
+index d716e76..453a350 100644
+--- a/pinentry-dmenu.c
++++ b/pinentry-dmenu.c
+@@ -784,8 +784,6 @@ main(int argc, char *argv[]) {
+ 		fprintf(stderr, "%s:%d: %s\n", config_error_file(&cfg),
+ 		        config_error_line(&cfg), config_error_text(&cfg));
+ 		return(EXIT_FAILURE);
+-	} else {
+-		printf("No config file found. Use defaults.\n");
+ 	}
+ 
+ 	pinentry_init("pinentry-dmenu");
+-- 
+2.34.1
+
diff --git a/srcpkgs/pinentry-dmenu/template b/srcpkgs/pinentry-dmenu/template
index c1bed06079ff..9bd4a7bd8438 100644
--- a/srcpkgs/pinentry-dmenu/template
+++ b/srcpkgs/pinentry-dmenu/template
@@ -1,7 +1,7 @@
 # Template file for 'pinentry-dmenu'
 pkgname=pinentry-dmenu
 version=0.2.2
-revision=2
+revision=3
 build_style=gnu-makefile
 makedepends="libXft-devel freetype-devel libconfig-devel
  libXinerama-devel libgpg-error-devel libassuan-devel

Re: pinentry-dmenu: fix for musl-based installations

[ericonr]

[-- Attachment #1: Type: text/plain, Size: 277 bytes --]

New comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/35137#issuecomment-1018561235

Comment:
Wow, that's such a strange strace :O 

I think we can merge it, but I'm asking in IRC to see what they make of the strace at least...

Re: [PR PATCH] [Closed]: pinentry-dmenu: fix for musl-based installations

[ftrvxmtrx]

[-- Attachment #1: Type: text/plain, Size: 535 bytes --]

There's a closed pull request on the void-packages repository

pinentry-dmenu: fix for musl-based installations
https://github.com/void-linux/void-packages/pull/35137

Description:
pinentry-dmenu without any user config would print to stdout which
would render the pipe useless - musl tries to find out whether stdout
is a terminal.

The patch sent upstream.

#### Testing the changes
- I tested the changes in this PR: **YES**

#### Local build testing
- I built this PR locally for my native architecture, (aarch64-musl)