add in-line compression to ldap handler

ChangeLog

@@ -15,13 +15,13 @@ version 0.9.4 -- unreleased
 	 . Fixed erroneous removal of tmpfile when it didn't exit
 	 . Fixed inversed vsname emptiness check
 	 . Fixed su quote usage to be more posixy
-	 . compress for sqldumps now happens in-line to save some disk space (Closes: #370778)
+	 . Compress for sqldumps now happens in-line to save some disk space (Closes: #370778)
 	pgsql:
 	 . Fixed inversed vsname emptiness check
 	 . Fixed su quote usage to be more posixy
 	 . Fixed shell expansion, thanks Thomas Kotzian (Closes: #363297)
 	 . postgres user UID is now the one from inside the vserver if necessary
-	 . compress now happens in-line to save some disk space (Closes: #370778)
+	 . Compress now happens in-line to save some disk space (Closes: #370778)
 	svn:
 	 . Fixed inversed vsname emptiness check
 	rdiff:
@@ -43,6 +43,8 @@ version 0.9.4 -- unreleased
 	 . Now forbid to (try to) include /.
 	sys:
 	 . Many more system checks were added, thanks to Petr Klíma
+	ldap:
+	 . Compress now happens in-line to save some disk space (Closes: #370778)
     lib changes
 	vserver:
 	 . init_vservers: fixed Debian bug #351083 (improper readlink syntax)

handlers/ldap

@@ -46,10 +46,18 @@ if [ "$ldif" == "yes" ]; then
       fi
       
       if [ "$method" == "slapcat" ]; then
+         if [ "$compress" == "yes" ]; then
+            execstr="$SLAPCAT -f $conf -b $dbsuffix | $GZIP"
+         else
             execstr="$SLAPCAT -f $conf -b $dbsuffix"
+         fi
          debug "$execstr"
+      else
+         if [ "$compress" == "yes" ]; then
+            execstr="$LDAPSEARCH -x -L -b ""$dbsuffix"" -D ""$binddn"" -y $passwordfile | $GZIP"
          else
             execstr="$LDAPSEARCH -x -L -b ""$dbsuffix"" -D ""$binddn"" -y $passwordfile"
+         fi
          [ -f "$passwordfile" ] || fatal "Password file $passwordfile not found. When method is set to ldapsearch, you must also specify a password file."
          debug "$execstr"
       fi
@@ -64,7 +72,11 @@ if [ "$ldif" == "yes" ]; then
             fatal "Couldn't create ldif dump file: $dumpdir/$dbsuffix.ldif"
          fi
          
+         if [ "$compress" == "yes" ]; then
+            output=`$execstr > $dumpdir/$dbsuffix.ldif.gz`
+         else
             output=`$execstr > $dumpdir/$dbsuffix.ldif`
+         fi
          code=$?
          if [ "$code" == "0" ]; then
             debug $output
@@ -73,10 +85,6 @@ if [ "$ldif" == "yes" ]; then
             warning $output
             warning "Failed ldif export of $dbsuffix"
          fi
-         if [ "$compress" == "yes" ]; then
-            output=`$GZIP -f "$dumpdir/$dbsuffix.ldif" 2>&1`
-            debug $output
-         fi
          
          if [ "$restart" == "yes" ]; then
             debug "Starting ldap server..."