Re: [Openvpn-devel] [PATCH 1/1] replace set with env

2020-01-06 Thread Christian Hesse
Tom Yan  on Mon, 2020/01/06 08:48:
> How about printenv (without grep)?

The variables are not known in advance. This needs to match all variables
starting with "enable_" and "with_".
main(a){char*c=/*Schoene Gruesse */"B?IJj;MEH"
"CX:;",b;for(a/*Best regards my address:*/=0;b=c[a++];)
putchar(b-1/(/*Chriscc -ox -xc - && ./x*/b/42*2-3)*42);}

Description: OpenPGP digital signature
Openvpn-devel mailing list

Re: [Openvpn-devel] [PATCH 1/1] replace set with env

2020-01-06 Thread Gert Doering

On Mon, Jan 06, 2020 at 08:48:00AM +, Tom Yan wrote:
> How about printenv (without grep)?


"env |grep" is POSIX standardized so should work everywhere.

printenv is a BSD invention, which not all unix variants are required
to have.

(Besides, printenv on its own cannot do "give me all environment variables
that start with 'with_'", which is exactly what we want here... so you'd
end up with "printenv | grep", which is less portable and more characters)

"If was one thing all people took for granted, was conviction that if you 
 feed honest figures into a computer, honest figures come out. Never doubted 
 it myself till I met a computer with a sense of humor."
 Robert A. Heinlein, The Moon is a Harsh Mistress

Gert Doering - Munich, Germany

Description: PGP signature
Openvpn-devel mailing list

Re: [Openvpn-devel] [PATCH 1/1] replace set with env

2020-01-06 Thread Tom Yan
How about printenv (without grep)?

From: Christian Hesse 
Sent: Monday, January 6, 2020 4:04:26 PM
To: OpenVPN Development 
Cc: Christian Hesse 
Subject: [Openvpn-devel] [PATCH 1/1] replace set with env

From: Christian Hesse 

The shell builtin `set` produces different output for different shells:

bash$ set | grep '^TERM='
dash$ set | grep '^TERM='

This may break reproducible builds depending on what shell is used.

Let's replace `set` with `env`, which is a real command and always
produces identical output.

Signed-off-by: Christian Hesse 
--- | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ b/
index a47e0a06..f13ff7b6 100644
--- a/
+++ b/
@@ -1326,7 +1326,7 @@ if test "${enable_async_push}" = "yes"; then

-CONFIGURE_DEFINES="`set | grep '^enable_.*=' ; set | grep '^with_.*='`"
+CONFIGURE_DEFINES="`env | grep '^enable_.*=' ; env | grep '^with_.*='`"
[Configuration settings])


Openvpn-devel mailing list
Openvpn-devel mailing list