Jason A. Donenfeld wrote: > On Mon, Jun 26, 2017 at 09:45:14PM +0100, [email protected] wrote: > > another fix for RHEL7/etc., the version of bash included doesn't support > > 'local -n' (neither does Amazon AMI Linux either) > > > > diff --git a/src/tools/wg-quick.bash b/src/tools/wg-quick.bash > > index 6edcd3a..afec996 100755 > > --- a/src/tools/wg-quick.bash > > +++ b/src/tools/wg-quick.bash > > @@ -54,7 +54,7 @@ parse_options() { > > } > > > > read_bool() { > > - local -n out="$1" > > + local out="$1" > > case "$2" in > > true) out=1 ;; > > false) out=0 ;; > > > > Before removing `-n`, did you even read the bash man page to learn what > it did? Things are most certainly broken with this patch, and the config > option that uses this will no longer do anything. Nack.
I don't have a system that has that option in it, so no. As it stands wg-quick does not work on a RHEL/Amazon AMI instance. Feel free to ignore the patch of course if it breaks it the logic. wg-quick up/down worked for me after this, but it wasn't an extensive test case. _______________________________________________ WireGuard mailing list [email protected] https://lists.zx2c4.com/mailman/listinfo/wireguard
