Remove pointless rpmdbSync() function

- The DB backend is supposed to take care of syncing when necessary,
  syncing all indexes "just in case" whether they were written to
  doesn't make a whole lot of sense.

lib/rpmdb.c

@@ -466,16 +466,6 @@ exit:
     return rc;
 }
 
-int rpmdbSync(rpmdb db)
-{
-    int rc;
-    if (db == NULL) return 0;
-
-    rc = dbiSync(db->db_pkgs, 0);
-    rc += dbiForeach(db->db_indexes, db->db_ndbi, dbiSync, 0);
-    return rc;
-}
-
 static rpmdb newRpmdb(const char * root, const char * home,
 		      int mode, int perms, int flags)
 {

lib/rpmdb_internal.h

@@ -58,14 +58,6 @@ int rpmdbInit(const char * prefix, int perms);
 RPM_GNUC_INTERNAL
 int rpmdbClose (rpmdb db);
 
-/** \ingroup rpmdb
- * Sync all database indices.
- * @param db		rpm database
- * @return		0 on success
- */
-RPM_GNUC_INTERNAL
-int rpmdbSync (rpmdb db);
-
 /** \ingroup rpmdb
  * Rebuild database indices from package headers.
  * @param prefix	path to top of install tree

lib/transaction.c

@@ -1409,7 +1409,6 @@ static int rpmtsProcess(rpmts ts)
 		   rpmteTypeString(p), failed > 1 ? _("skipped") : _("failed"));
 	    rc++;
 	}
-	(void) rpmdbSync(rpmtsGetRdb(ts));
     }
     rpmtsiFree(pi);
     return rc;