Dead code removal
authorPanu Matilainen <pmatilai@redhat.com>
Fri, 19 Mar 2010 07:18:25 +0000 (09:18 +0200)
committerPanu Matilainen <pmatilai@redhat.com>
Fri, 19 Mar 2010 07:18:25 +0000 (09:18 +0200)
- this code has last been active sometime < 2004, nobody's been missing it
- based on comments in https://bugzilla.redhat.com/show_bug.cgi?id=134497
  it was some highly specific kludge for somebody to begin with and
  shouldn't have been in rpm upstream ever but history is lost here...
- move the removePackage() outside the debug goo so the actual action
  stands out

lib/depends.c

index f8965de..77c3622 100644 (file)
@@ -316,13 +316,11 @@ addheader:
            if (rpmdsEVR(obsoletes) == NULL
             || rpmdsAnyMatchesDep(oh, obsoletes, _rpmds_nopromote)) {
                char * ohNEVRA = headerGetAsString(oh, RPMTAG_NEVRA);
-#ifdef DYING   /* XXX see http://bugzilla.redhat.com #134497 */
-               if (rpmVersionCompare(h, oh))
-#endif
-                   xx = removePackage(tsmem, oh, p);
                rpmlog(RPMLOG_DEBUG, "  Obsoletes: %s\t\terases %s\n",
                        rpmdsDNEVR(obsoletes)+2, ohNEVRA);
                ohNEVRA = _free(ohNEVRA);
+
+               removePackage(tsmem, oh, p);
            }
        }
        mi = rpmdbFreeIterator(mi);