From fd8155c0467ca83e9c8916ed6d8adcfb3d4b968a Mon Sep 17 00:00:00 2001 From: Donald Sharp Date: Tue, 12 Apr 2016 20:44:28 -0400 Subject: [PATCH] debian: Add the creation of the quagga user to quaggavty group The creation of the quagga user was not the only place to add the quagga user to the quaggavty group. If we are reinstalling quagga over a old version of code then we need to check to see if the quagga user is in the quaggavty and do the right thing. Signed-off-by: Donald Sharp Reviewed-by: Dave Olson --- debian/quagga.preinst | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/debian/quagga.preinst b/debian/quagga.preinst index 39448b2f6b..1af0b5ba63 100644 --- a/debian/quagga.preinst +++ b/debian/quagga.preinst @@ -24,7 +24,14 @@ if ! getent passwd quagga >/dev/null; then --gecos "Quagga routing suite" \ --shell /bin/false \ quagga >/dev/null - usermod -a -G quaggavty quagga >/dev/null +fi + +# We may be installing over an older version of +# quagga and as such we need to intelligently +# check to see if the quagga user is in the quaggavty +# group. +if ! /usr/bin/id quagga | grep &>/dev/null 'quaggavty'; then + usermod -a -G quaggavty quagga >/dev/null fi # Do not change permissions when upgrading as it would violate policy. -- 2.39.5