From 226fceeaa78adbeef9bbf67fbcf789b93bfb7bf4 Mon Sep 17 00:00:00 2001
From: Radek Novacek <rnovacek@redhat.com>
Date: Thu, 1 Aug 2013 15:06:00 +0200
Subject: Add logging to the openlmi-mof-register script

Each command in the registration script is now logged into
/var/log/openlmi-install.log file.
---
 openlmi-mof-register   |  46 +++++++---
 openlmi-providers.spec | 237 +++++++++++++++++++------------------------------
 2 files changed, 127 insertions(+), 156 deletions(-)

diff --git a/openlmi-mof-register b/openlmi-mof-register
index 4e60e31..6107503 100755
--- a/openlmi-mof-register
+++ b/openlmi-mof-register
@@ -23,10 +23,27 @@
 pegasus_repository="/var/lib/Pegasus/"
 default_namespace="root/cimv2"
 
+function log()
+{
+    printf "%(%Y-%m-%d %H:%M:%S)T " -1 >&2
+    printf "$*\n" >&2
+}
+
+function log_command()
+{
+    log "COMMAND:" "$*" >&2
+    "$@"
+    code=$?
+    log "EXIT CODE:" "$code" >&2
+    return $code
+}
+
+trap 'log "DONE WITH EXIT CODE:" "$?"' EXIT
+
 function start_pegasus()
 {
     # Start Pegasus CIMOM in restricted mode, listening only on its unix socket
-    /usr/sbin/cimserver daemon=true \
+    log_command /usr/sbin/cimserver daemon=true \
             enableHttpConnection=false \
             enableHttpsConnection=false \
             enableRemotePrivilegedUserAccess=false \
@@ -40,7 +57,7 @@ function start_pegasus()
 
 function stop_pegasus()
 {
-    /usr/sbin/cimserver -s
+    log_command /usr/sbin/cimserver -s
 }
 
 function usage()
@@ -76,9 +93,9 @@ function register()
     mofs="$@"
     if [ $HAS_SFCBD -eq 1 ] && echo $cimom | grep -q 'all\|sfcbd';
     then
-        /usr/bin/sfcbstage -n $namespace ${reg:+-r} $reg $mofs
-        /usr/bin/sfcbrepos -f
-        /usr/bin/systemctl reload-or-try-restart sblim-sfcb.service
+        log_command /usr/bin/sfcbstage -n $namespace ${reg:+-r} $reg $mofs
+        log_command /usr/bin/sfcbrepos -f
+        log_command /usr/bin/systemctl reload-or-try-restart sblim-sfcb.service
     fi
 
     if [ $HAS_PEGASUS -eq  1 ] && echo $cimom | grep -q 'all\|tog-pegasus';
@@ -91,13 +108,17 @@ function register()
             CIMMOF="/usr/bin/cimmofl -aEV -R $pegasus_repository -n $namespace"
         fi
 
-        $CIMMOF -uc $mofs
+        log_command $CIMMOF -uc $mofs
         if [ $JUST_MOFS -eq 0 ]; then
             if [ -x $(dirname $0)/openlmi-register-pegasus ];
             then
+                log "COMMAND: cat \"$reg\" | $(dirname $0)/openlmi-register-pegasus -v \"$version\" | $CIMMOF -uc -n root/interop"
                 cat "$reg" | $(dirname $0)/openlmi-register-pegasus -v "$version" | $CIMMOF -uc -n root/interop
+                log "EXIT CODE:" "$?"
             else
+                log "COMMAND: cat \"$reg\" | /usr/libexec/openlmi-register-pegasus -v \"$version\" | $CIMMOF -uc -n root/interop"
                 cat "$reg" | /usr/libexec/openlmi-register-pegasus -v "$version" | $CIMMOF -uc -n root/interop
+                log "EXIT CODE:" "$?"
             fi
         fi
     fi
@@ -118,9 +139,9 @@ function unregister()
         for ((i=0; i<${#mofs[@]}; i++)); do
             bmofs[$i]=$(basename "${mofs[$i]}")
         done
-        /usr/bin/sfcbunstage -n $namespace ${reg:+-r} $(basename "$reg") ${bmofs[@]}
-        /usr/bin/sfcbrepos -f
-        /usr/bin/systemctl reload-or-try-restart sblim-sfcb.service
+        log_command /usr/bin/sfcbunstage -n $namespace ${reg:+-r} $(basename "$reg") ${bmofs[@]}
+        log_command /usr/bin/sfcbrepos -f
+        log_command /usr/bin/systemctl reload-or-try-restart sblim-sfcb.service
     fi
 
     if [ $HAS_PEGASUS -eq 1 ] && echo $cimom | grep -q 'all\|tog-pegasus';
@@ -139,7 +160,8 @@ function unregister()
             fi
             for provider in $providers;
             do
-                /usr/bin/cimprovider -d -m ${provider} && /usr/bin/cimprovider -r -m ${provider}
+                log_command /usr/bin/cimprovider -d -m ${provider}
+                log_command /usr/bin/cimprovider -r -m ${provider}
             done
         fi
         mofcomp -n $namespace -r ${mofs[@]}
@@ -149,6 +171,8 @@ function unregister()
     fi
 }
 
+log "STARTING" "$0" "$@"
+
 JUST_MOFS=0
 optspec=":hn:c:v:-:"
 
@@ -262,3 +286,5 @@ case $CMD in
         usage
         exit 1
 esac
+
+exit 0
diff --git a/openlmi-providers.spec b/openlmi-providers.spec
index 0552a8c..84a3c56 100644
--- a/openlmi-providers.spec
+++ b/openlmi-providers.spec
@@ -1,3 +1,5 @@
+%global logfile %{_localstatedir}/log/openlmi-install.log
+
 Name:           openlmi-providers
 Version:        0.1.0
 Release:        1%{?dist}
@@ -271,6 +273,7 @@ mkdir -p $RPM_BUILD_ROOT/%{_localstatedir}/lib/%{name}
 %{_libdir}/libopenlmicommon.so.*
 %attr(755, root, root) %{_bindir}/openlmi-mof-register
 %attr(755, root, root) %{_libexecdir}/openlmi-register-pegasus
+%ghost %logfile
 
 %files devel
 %{_bindir}/openlmi-doc-class2rst
@@ -396,27 +399,24 @@ mkdir -p $RPM_BUILD_ROOT/%{_localstatedir}/lib/%{name}
 if [ "$1" -gt 1 ]; then
     %{_bindir}/openlmi-mof-register --just-mofs unregister \
         %{_datadir}/%{name}/05_LMI_Qualifiers.mof \
-        %{_datadir}/%{name}/30_LMI_Jobs.mof \
-        > /dev/null 2>&1 || :;
-fi
+        %{_datadir}/%{name}/30_LMI_Jobs.mof || :;
+fi >> %logfile 2>&1
 
 %post
 /sbin/ldconfig
 if [ "$1" -ge 1 ]; then
     %{_bindir}/openlmi-mof-register --just-mofs register \
         %{_datadir}/%{name}/05_LMI_Qualifiers.mof \
-        %{_datadir}/%{name}/30_LMI_Jobs.mof \
-        > /dev/null 2>&1 || :;
-fi
+        %{_datadir}/%{name}/30_LMI_Jobs.mof || :;
+fi >> %logfile 2>&1
 
 %preun
 # Deregister only if not upgrading
 if [ "$1" -eq 0 ]; then
     %{_bindir}/openlmi-mof-register --just-mofs unregister \
         %{_datadir}/%{name}/05_LMI_Qualifiers.mof \
-        %{_datadir}/%{name}/30_LMI_Jobs.mof \
-        > /dev/null 2>&1 || :;
-fi
+        %{_datadir}/%{name}/30_LMI_Jobs.mof || :;
+fi >> %logfile 2>&1
 
 %postun -p /sbin/ldconfig
 
@@ -428,310 +428,255 @@ fi
 if [ "$1" -gt 1 ]; then
     %{_bindir}/openlmi-mof-register -v %{version} unregister \
         %{_datadir}/%{name}/60_LMI_Fan.mof \
-        %{_datadir}/%{name}/60_LMI_Fan.reg \
-        > /dev/null 2>&1 || :;
+        %{_datadir}/%{name}/60_LMI_Fan.reg || :;
     %{_bindir}/openlmi-mof-register --just-mofs -n root/interop -c tog-pegasus unregister \
-        %{_datadir}/%{name}/90_LMI_Fan_Profile.mof \
-        > /dev/null 2>&1 || :;
-fi
+        %{_datadir}/%{name}/90_LMI_Fan_Profile.mof || :;
+fi >> %logfile 2>&1
 
 %pre -n openlmi-powermanagement
 if [ "$1" -gt 1 ]; then
     %{_bindir}/openlmi-mof-register -v %{version} unregister \
         %{_datadir}/%{name}/60_LMI_PowerManagement.mof \
-        %{_datadir}/%{name}/60_LMI_PowerManagement.reg \
-        > /dev/null 2>&1 || :;
+        %{_datadir}/%{name}/60_LMI_PowerManagement.reg || :;
     %{_bindir}/openlmi-mof-register --just-mofs -n root/interop -c tog-pegasus unregister \
-        %{_datadir}/%{name}/90_LMI_PowerManagement_Profile.mof \
-        > /dev/null 2>&1 || :;
-fi
+        %{_datadir}/%{name}/90_LMI_PowerManagement_Profile.mof || :;
+fi >> %logfile 2>&1
 
 %pre -n openlmi-service
 if [ "$1" -gt 1 ]; then
     %{_bindir}/openlmi-mof-register -v %{version} unregister \
         %{_datadir}/%{name}/60_LMI_Service.mof \
-        %{_datadir}/%{name}/60_LMI_Service.reg \
-        > /dev/null 2>&1 || :;
+        %{_datadir}/%{name}/60_LMI_Service.reg || :;
     %{_bindir}/openlmi-mof-register --just-mofs -n root/interop -c tog-pegasus unregister \
-        %{_datadir}/%{name}/90_LMI_Service_Profile.mof \
-        > /dev/null 2>&1 || :;
-fi
+        %{_datadir}/%{name}/90_LMI_Service_Profile.mof || :;
+fi >> %logfile 2>&1
 
 %pre -n openlmi-account
 if [ "$1" -gt 1 ]; then
     %{_bindir}/openlmi-mof-register -v %{version} unregister \
         %{_datadir}/%{name}/60_LMI_Account.mof \
-        %{_datadir}/%{name}/60_LMI_Account.reg \
-        > /dev/null 2>&1 || :;
+        %{_datadir}/%{name}/60_LMI_Account.reg || :;
     %{_bindir}/openlmi-mof-register --just-mofs -n root/interop -c tog-pegasus unregister \
-        %{_datadir}/%{name}/90_LMI_Account_Profile.mof \
-        > /dev/null 2>&1 || :;
-fi
+        %{_datadir}/%{name}/90_LMI_Account_Profile.mof || :;
+fi >> %logfile 2>&1
 
 %pre -n openlmi-software
 if [ "$1" -gt 1 ]; then
     # delete indication filters
     %{_bindir}/openlmi-mof-register --just-mofs -n root/interop unregister \
-        %{_datadir}/%{name}/70_LMI_SoftwareIndicationFilters.mof \
-        > /dev/null 2>&1 || :;
+        %{_datadir}/%{name}/70_LMI_SoftwareIndicationFilters.mof || :;
     %{_bindir}/openlmi-mof-register --just-mofs -n root/interop -c tog-pegasus unregister \
-        %{_datadir}/%{name}/90_LMI_Software_Profile.mof \
-        > /dev/null 2>&1 || :;
-    %{_bindir}/openlmi-mof-register -v %{version} unregister \
+        %{_datadir}/%{name}/90_LMI_Software_Profile.mof || :;
+    %{_bindir}/openlmi-mof-register -v %{version} unregister
         %{_datadir}/%{name}/60_LMI_Software.mof \
-        %{_datadir}/%{name}/LMI_Software.reg \
-        > /dev/null 2>&1 || :;
-fi
+        %{_datadir}/%{name}/LMI_Software.reg || :;
+fi >> %logfile 2>&1
 
 %pre -n openlmi-logicalfile
 if [ "$1" -gt 1 ]; then
     %{_bindir}/openlmi-mof-register -v %{version} unregister \
         %{_datadir}/%{name}/60_LMI_LogicalFile.mof \
-        %{_datadir}/%{name}/60_LMI_LogicalFile.reg \
-        > /dev/null 2>&1 || :;
+        %{_datadir}/%{name}/60_LMI_LogicalFile.reg || :;
     %{_bindir}/openlmi-mof-register --just-mofs -n root/interop -c tog-pegasus unregister \
-        %{_datadir}/%{name}/90_LMI_LogicalFile_Profile.mof \
-        > /dev/null 2>&1 || :;
-fi
+        %{_datadir}/%{name}/90_LMI_LogicalFile_Profile.mof || :;
+fi >> %logfile 2>&1
 
 %pre -n openlmi-realmd
 if [ "$1" -gt 1 ]; then
     %{_bindir}/openlmi-mof-register -v %{version} unregister \
         %{_datadir}/%{name}/60_LMI_Realmd.mof \
-        %{_datadir}/%{name}/60_LMI_Realmd.reg \
-        > /dev/null 2>&1 || :;
+        %{_datadir}/%{name}/60_LMI_Realmd.reg || :;
     %{_bindir}/openlmi-mof-register --just-mofs -n root/interop -c tog-pegasus unregister \
-        %{_datadir}/%{name}/90_LMI_Realmd_Profile.mof \
-        > /dev/null 2>&1 || :;
-fi
+        %{_datadir}/%{name}/90_LMI_Realmd_Profile.mof || :;
+fi >> %logfile 2>&1
 
 %pre -n openlmi-hardware
 if [ "$1" -gt 1 ]; then
     %{_bindir}/openlmi-mof-register -v %{version} unregister \
         %{_datadir}/%{name}/60_LMI_Hardware.mof \
-        %{_datadir}/%{name}/60_LMI_Hardware.reg \
-        > /dev/null 2>&1 || :;
+        %{_datadir}/%{name}/60_LMI_Hardware.reg || :;
     %{_bindir}/openlmi-mof-register --just-mofs -n root/interop -c tog-pegasus unregister \
         %{_datadir}/%{name}/90_LMI_Hardware_Profile.mof \
-        %{_datadir}/%{name}/90_LMI_Hardware_Profile_DMTF.mof \
-        > /dev/null 2>&1 || :;
-fi
+        %{_datadir}/%{name}/90_LMI_Hardware_Profile_DMTF.mof || :;
+fi >> %logfile 2>&1
 
 %pre -n openlmi-pcp
 if [ "$1" -gt 1 ]; then
     %{_bindir}/openlmi-mof-register -v %{version} unregister \
         %{_datadir}/%{name}/60_LMI_PCP.mof \
         %{_localstatedir}/lib/%{name}/60_LMI_PCP_PMNS.mof \
-        %{_localstatedir}/lib/%{name}/60_LMI_PCP_PMNS.reg \
-        > /dev/null 2>&1 || :;
-fi
+        %{_localstatedir}/lib/%{name}/60_LMI_PCP_PMNS.reg || :;
+fi >> %logfile 2>&1
 
 %post -n openlmi-fan
 # Register Schema and Provider
 if [ "$1" -ge 1 ]; then
     %{_bindir}/openlmi-mof-register -v %{version} register \
         %{_datadir}/%{name}/60_LMI_Fan.mof \
-        %{_datadir}/%{name}/60_LMI_Fan.reg \
-        > /dev/null 2>&1 || :;
+        %{_datadir}/%{name}/60_LMI_Fan.reg || :;
     %{_bindir}/openlmi-mof-register --just-mofs -n root/interop -c tog-pegasus register \
-        %{_datadir}/%{name}/90_LMI_Fan_Profile.mof \
-        > /dev/null 2>&1 || :;
-fi
+        %{_datadir}/%{name}/90_LMI_Fan_Profile.mof || :;
+fi >> %logfile 2>&1
 
 %post -n openlmi-powermanagement
 if [ "$1" -ge 1 ]; then
     %{_bindir}/openlmi-mof-register -v %{version} register \
         %{_datadir}/%{name}/60_LMI_PowerManagement.mof \
-        %{_datadir}/%{name}/60_LMI_PowerManagement.reg \
-        > /dev/null 2>&1 || :;
+        %{_datadir}/%{name}/60_LMI_PowerManagement.reg || :;
     %{_bindir}/openlmi-mof-register --just-mofs -n root/interop -c tog-pegasus register \
-        %{_datadir}/%{name}/90_LMI_PowerManagement_Profile.mof \
-        > /dev/null 2>&1 || :;
-
-fi
+        %{_datadir}/%{name}/90_LMI_PowerManagement_Profile.mof || :;
+fi >> %logfile 2>&1
 
 %post -n openlmi-service
 if [ "$1" -ge 1 ]; then
     %{_bindir}/openlmi-mof-register -v %{version} register \
         %{_datadir}/%{name}/60_LMI_Service.mof \
-        %{_datadir}/%{name}/60_LMI_Service.reg \
-        > /dev/null 2>&1 || :;
+        %{_datadir}/%{name}/60_LMI_Service.reg || :;
     %{_bindir}/openlmi-mof-register --just-mofs -n root/interop -c tog-pegasus register \
-        %{_datadir}/%{name}/90_LMI_Service_Profile.mof \
-        > /dev/null 2>&1 || :;
-fi
+        %{_datadir}/%{name}/90_LMI_Service_Profile.mof || :;
+fi >> %logfile 2>&1
 
 %post -n openlmi-account
 if [ "$1" -ge 1 ]; then
     %{_bindir}/openlmi-mof-register -v %{version} register \
         %{_datadir}/%{name}/60_LMI_Account.mof \
-        %{_datadir}/%{name}/60_LMI_Account.reg \
-        > /dev/null 2>&1 || :;
+        %{_datadir}/%{name}/60_LMI_Account.reg || :;
     %{_bindir}/openlmi-mof-register --just-mofs -n root/interop -c tog-pegasus register \
-        %{_datadir}/%{name}/90_LMI_Account_Profile.mof \
-        > /dev/null 2>&1 || :;
-fi
+        %{_datadir}/%{name}/90_LMI_Account_Profile.mof || :;
+fi >> %logfile 2>&1
 
 %post -n openlmi-software
 if [ "$1" -ge 1 ]; then
     %{_bindir}/openlmi-mof-register -v %{version} register \
         %{_datadir}/%{name}/60_LMI_Software.mof \
-        %{_datadir}/%{name}/LMI_Software.reg \
-        > /dev/null 2>&1 || :;
+        %{_datadir}/%{name}/LMI_Software.reg || :;
     # install indication filters for sfcbd
     %{_bindir}/openlmi-mof-register --just-mofs -n root/interop register \
-        %{_datadir}/%{name}/70_LMI_SoftwareIndicationFilters.mof \
-        > /dev/null 2>&1 || :;
+        %{_datadir}/%{name}/70_LMI_SoftwareIndicationFilters.mof || :;
     %{_bindir}/openlmi-mof-register --just-mofs -n root/interop -c tog-pegasus register \
-        %{_datadir}/%{name}/90_LMI_Software_Profile.mof \
-        > /dev/null 2>&1 || :;
-fi
+        %{_datadir}/%{name}/90_LMI_Software_Profile.mof || :;
+fi >> %logfile 2>&1
 
 %post -n openlmi-logicalfile
 if [ "$1" -ge 1 ]; then
     %{_bindir}/openlmi-mof-register -v %{version} register \
         %{_datadir}/%{name}/60_LMI_LogicalFile.mof \
-        %{_datadir}/%{name}/60_LMI_LogicalFile.reg \
-        > /dev/null 2>&1 || :;
+        %{_datadir}/%{name}/60_LMI_LogicalFile.reg || :;
     %{_bindir}/openlmi-mof-register --just-mofs -n root/interop -c tog-pegasus register \
-        %{_datadir}/%{name}/90_LMI_LogicalFile_Profile.mof \
-        > /dev/null 2>&1 || :;
-fi
+        %{_datadir}/%{name}/90_LMI_LogicalFile_Profile.mof || :;
+fi >> %logfile 2>&1
 
 %post -n openlmi-realmd
 if [ "$1" -ge 1 ]; then
     %{_bindir}/openlmi-mof-register -v %{version} register \
         %{_datadir}/%{name}/60_LMI_Realmd.mof \
-        %{_datadir}/%{name}/60_LMI_Realmd.reg \
-        > /dev/null 2>&1 || :;
+        %{_datadir}/%{name}/60_LMI_Realmd.reg || :;
     %{_bindir}/openlmi-mof-register --just-mofs -n root/interop -c tog-pegasus register \
-        %{_datadir}/%{name}/90_LMI_Realmd_Profile.mof \
-        > /dev/null 2>&1 || :;
-fi
+        %{_datadir}/%{name}/90_LMI_Realmd_Profile.mof || :;
+fi >> %logfile 2>&1
 
 %post -n openlmi-hardware
 if [ "$1" -ge 1 ]; then
     %{_bindir}/openlmi-mof-register -v %{version} register \
         %{_datadir}/%{name}/60_LMI_Hardware.mof \
-        %{_datadir}/%{name}/60_LMI_Hardware.reg \
-        > /dev/null 2>&1 || :;
+        %{_datadir}/%{name}/60_LMI_Hardware.reg || :;
     %{_bindir}/openlmi-mof-register --just-mofs -n root/interop -c tog-pegasus register \
         %{_datadir}/%{name}/90_LMI_Hardware_Profile.mof \
-        %{_datadir}/%{name}/90_LMI_Hardware_Profile_DMTF.mof \
-        > /dev/null 2>&1 || :;
-fi
+        %{_datadir}/%{name}/90_LMI_Hardware_Profile_DMTF.mof || :;
+fi >> %logfile 2>&1
 
 %post -n openlmi-pcp
 if [ "$1" -ge 1 ]; then
     %{_bindir}/openlmi-mof-register -v %{version} register \
         %{_datadir}/%{name}/60_LMI_PCP.mof \
         %{_localstatedir}/lib/%{name}/60_LMI_PCP_PMNS.mof \
-        %{_localstatedir}/lib/%{name}/60_LMI_PCP_PMNS.reg \
-        > /dev/null 2>&1 || :;
-fi
+        %{_localstatedir}/lib/%{name}/60_LMI_PCP_PMNS.reg || :;
+fi >> %logfile 2>&1
 
 %preun -n openlmi-fan
 # Deregister only if not upgrading
 if [ "$1" -eq 0 ]; then
     %{_bindir}/openlmi-mof-register -v %{version} unregister \
         %{_datadir}/%{name}/60_LMI_Fan.mof \
-        %{_datadir}/%{name}/60_LMI_Fan.reg \
-        > /dev/null 2>&1 || :;
+        %{_datadir}/%{name}/60_LMI_Fan.reg || :;
     %{_bindir}/openlmi-mof-register --just-mofs -n root/interop -c tog-pegasus unregister \
-        %{_datadir}/%{name}/90_LMI_Fan_Profile.mof \
-        > /dev/null 2>&1 || :;
-fi
+        %{_datadir}/%{name}/90_LMI_Fan_Profile.mof || :;
+fi >> %logfile 2>&1
 
 %preun -n openlmi-powermanagement
 if [ "$1" -eq 0 ]; then
     %{_bindir}/openlmi-mof-register -v %{version} unregister \
         %{_datadir}/%{name}/60_LMI_PowerManagement.mof \
-        %{_datadir}/%{name}/60_LMI_PowerManagement.reg \
-        > /dev/null 2>&1 || :;
+        %{_datadir}/%{name}/60_LMI_PowerManagement.reg || :;
     %{_bindir}/openlmi-mof-register --just-mofs -n root/interop -c tog-pegasus unregister \
-        %{_datadir}/%{name}/90_LMI_PowerManagement_Profile.mof \
-        > /dev/null 2>&1 || :;
-fi
+        %{_datadir}/%{name}/90_LMI_PowerManagement_Profile.mof || :;
+fi >> %logfile 2>&1
 
 %preun -n openlmi-service
 if [ "$1" -eq 0 ]; then
     %{_bindir}/openlmi-mof-register -v %{version} unregister \
         %{_datadir}/%{name}/60_LMI_Service.mof \
-        %{_datadir}/%{name}/60_LMI_Service.reg \
-        > /dev/null 2>&1 || :;
+        %{_datadir}/%{name}/60_LMI_Service.reg || :;
     %{_bindir}/openlmi-mof-register --just-mofs -n root/interop -c tog-pegasus unregister \
-        %{_datadir}/%{name}/90_LMI_Service_Profile.mof \
-        > /dev/null 2>&1 || :;
-fi
+        %{_datadir}/%{name}/90_LMI_Service_Profile.mof || :;
+fi >> %logfile 2>&1
 
 %preun -n openlmi-account
 if [ "$1" -eq 0 ]; then
     %{_bindir}/openlmi-mof-register -v %{version} unregister \
         %{_datadir}/%{name}/60_LMI_Account.mof \
-        %{_datadir}/%{name}/60_LMI_Account.reg \
-        > /dev/null 2>&1 || :;
+        %{_datadir}/%{name}/60_LMI_Account.reg || :;
     %{_bindir}/openlmi-mof-register --just-mofs -n root/interop -c tog-pegasus unregister \
-        %{_datadir}/%{name}/90_LMI_Account_Profile.mof \
-        > /dev/null 2>&1 || :;
-fi
+        %{_datadir}/%{name}/90_LMI_Account_Profile.mof || :;
+fi >> %logfile 2>&1
 
 %preun -n openlmi-software
 if [ "$1" -eq 0 ]; then
     # delete indication filters
     %{_bindir}/openlmi-mof-register --just-mofs -n root/interop unregister \
-        %{_datadir}/%{name}/70_LMI_SoftwareIndicationFilters.mof \
-        > /dev/null 2>&1 || :;
+        %{_datadir}/%{name}/70_LMI_SoftwareIndicationFilters.mof || :;
     %{_bindir}/openlmi-mof-register --just-mofs -n root/interop -c tog-pegasus unregister \
-        %{_datadir}/%{name}/90_LMI_Software_Profile.mof \
-        > /dev/null 2>&1 || :;
+        %{_datadir}/%{name}/90_LMI_Software_Profile.mof || :;
     %{_bindir}/openlmi-mof-register -v %{version} unregister \
         %{_datadir}/%{name}/60_LMI_Software.mof \
-        %{_datadir}/%{name}/LMI_Software.reg \
-        > /dev/null 2>&1 || :;
-fi
+        %{_datadir}/%{name}/LMI_Software.reg || :;
+fi >> %logfile 2>&1
 
 %preun -n openlmi-logicalfile
 if [ "$1" -eq 0 ]; then
     %{_bindir}/openlmi-mof-register -v %{version} unregister \
         %{_datadir}/%{name}/60_LMI_LogicalFile.mof \
-        %{_datadir}/%{name}/60_LMI_LogicalFile.reg \
-        > /dev/null 2>&1 || :;
+        %{_datadir}/%{name}/60_LMI_LogicalFile.reg || :;
     %{_bindir}/openlmi-mof-register --just-mofs -n root/interop -c tog-pegasus unregister \
-        %{_datadir}/%{name}/90_LMI_LogicalFile_Profile.mof \
-        > /dev/null 2>&1 || :;
-fi
+        %{_datadir}/%{name}/90_LMI_LogicalFile_Profile.mof || :;
+fi >> %logfile 2>&1
 
 %preun -n openlmi-realmd
 if [ "$1" -eq 0 ]; then
     %{_bindir}/openlmi-mof-register -v %{version} unregister \
         %{_datadir}/%{name}/60_LMI_Realmd.mof \
-        %{_datadir}/%{name}/60_LMI_Realmd.reg \
-        > /dev/null 2>&1 || :;
+        %{_datadir}/%{name}/60_LMI_Realmd.reg || :;
     %{_bindir}/openlmi-mof-register --just-mofs -n root/interop -c tog-pegasus unregister \
-        %{_datadir}/%{name}/90_LMI_Realmd_Profile.mof \
-        > /dev/null 2>&1 || :;
-fi
+        %{_datadir}/%{name}/90_LMI_Realmd_Profile.mof || :;
+fi >> %logfile 2>&1
 
 %preun -n openlmi-hardware
 if [ "$1" -eq 0 ]; then
     %{_bindir}/openlmi-mof-register -v %{version} unregister \
         %{_datadir}/%{name}/60_LMI_Hardware.mof \
-        %{_datadir}/%{name}/60_LMI_Hardware.reg \
-        > /dev/null 2>&1 || :;
+        %{_datadir}/%{name}/60_LMI_Hardware.reg || :;
     %{_bindir}/openlmi-mof-register --just-mofs -n root/interop -c tog-pegasus unregister \
         %{_datadir}/%{name}/90_LMI_Hardware_Profile.mof \
-        %{_datadir}/%{name}/90_LMI_Hardware_Profile_DMTF.mof \
-        > /dev/null 2>&1 || :;
-fi
+        %{_datadir}/%{name}/90_LMI_Hardware_Profile_DMTF.mof || :;
+fi >> %logfile 2>&1
 
 %preun -n openlmi-pcp
 if [ "$1" -eq 0 ]; then
     %{_bindir}/openlmi-mof-register -v %{version} unregister \
         %{_datadir}/%{name}/60_LMI_PCP.mof \
         %{_localstatedir}/lib/%{name}/60_LMI_PCP_PMNS.mof \
-        %{_localstatedir}/lib/%{name}/60_LMI_PCP_PMNS.reg \
-        > /dev/null 2>&1 || :;
-fi
+        %{_localstatedir}/lib/%{name}/60_LMI_PCP_PMNS.reg || :;
+fi >> %logfile 2>&1
 
 %changelog
 * Wed Jul 31 2013 Radek Novacek <rnovacek@redhat.com> 0.1.0-1
-- 
cgit