summaryrefslogtreecommitdiffstats
path: root/debian
diff options
context:
space:
mode:
authorDonald Sharp <sharpd@cumulusnetworks.com>2016-04-13 02:44:28 +0200
committerDonald Sharp <sharpd@cumulusnetworks.com>2016-04-13 13:21:44 +0200
commitfd8155c0467ca83e9c8916ed6d8adcfb3d4b968a (patch)
tree758c863a6ff7444da2b264f353b91f9d7978663c /debian
parentlib, zebra: Refactor vrf creation a bit more (diff)
downloadfrr-fd8155c0467ca83e9c8916ed6d8adcfb3d4b968a.tar.xz
frr-fd8155c0467ca83e9c8916ed6d8adcfb3d4b968a.zip
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 <sharpd@cumulusnetworks.com> Reviewed-by: Dave Olson <olson@cumulusnetworks.com>
Diffstat (limited to 'debian')
-rw-r--r--debian/quagga.preinst9
1 files changed, 8 insertions, 1 deletions
diff --git a/debian/quagga.preinst b/debian/quagga.preinst
index 39448b2f6..1af0b5ba6 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.