Remove redundant progress notification from fsmCommit()
authorPanu Matilainen <pmatilai@redhat.com>
Tue, 17 Apr 2012 10:25:59 +0000 (13:25 +0300)
committerPanu Matilainen <pmatilai@redhat.com>
Tue, 17 Apr 2012 10:25:59 +0000 (13:25 +0300)
- fsmCommit() does not advance the archive or anything else, so
  this was simply issuing the same exact notification that we
  just did from rpmPackageFilesInstall().

lib/fsm.c

index c68283f..151d68e 100644 (file)
--- a/lib/fsm.c
+++ b/lib/fsm.c
@@ -1764,12 +1764,7 @@ static int fsmCommit(FSM_t fsm)
         }
     }
 
-    /* Notify on success. */
-    if (!rc) {
-        if (fsm->goal == FSM_PKGINSTALL) {
-            rpmpsmNotify(fsm->psm, RPMCALLBACK_INST_PROGRESS, rpmcpioTell(fsm->archive));
-        }
-    } else if (fsm->failedFile && *fsm->failedFile == NULL) {
+    if (rc && fsm->failedFile && *fsm->failedFile == NULL) {
         *fsm->failedFile = fsm->path;
         fsm->path = NULL;
     }