[Oval-commits] r242 - trunk/oval-agent/debian

[Javier Fernandez-Sanguino Pen~a]

Author: jfs
Date: 2007-09-04 19:52:44 +0000 (Tue, 04 Sep 2007)
New Revision: 242

Modified:
   trunk/oval-agent/debian/postrm
Log:
Add possible bugs

Modified: trunk/oval-agent/debian/postrm
===================================================================
--- trunk/oval-agent/debian/postrm	2007-09-04 19:50:52 UTC (rev 241)
+++ trunk/oval-agent/debian/postrm	2007-09-04 19:52:44 UTC (rev 242)
@@ -19,6 +19,7 @@
 # the debian-policy package
 
 
+# (jfs) BUG: Dpkg should not be invoked from within a postrm script
 OVALSERVER=`dpkg -l | grep oval-server | awk '{print $1}'`
 
 case "$1" in
@@ -27,12 +28,18 @@
     ;;
 
     remove)
+        # (jfs) BUG: No need to remove the directory, dpkg will 
+        # remove it if empty, just remove the log file...
 	if [ "OVALSERVER" != "ii" ]; then
 		rm -rf /var/log/oval
 	else
 		rm -rf /var/log/oval/oval-agent.log
 	fi
 	
+        # (jfs) BUG: This check should be more strict, this user
+        # might have been created by the user but not by the package
+        # also, it makes more sense to have an oval-agent and oval-server
+        # users so that they do not overlap
 	if [ "OVALSERVER" != "ii" ]; then
 		if id oval >/dev/null; then
 			deluser oval