RPM

filter redundant provides, requires and conflicts

Bug #637153 reported by Jeff Johnson
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
RPM
Opinion
Low
Jeff Johnson
CentOS
New
Unknown

Bug Description

tracker

Jeff Johnson (n3npq)
tags: added: rpm.org rpmbuild
Changed in centos:
status: Unknown → New
Revision history for this message
Jeff Johnson (n3npq) wrote :

rpm already unifies all {N,EVR,F} tuples in dependencies.

What is confusing is that the flags variable has per-scriptlet context markers
that are part of the unification while only the {N,EVR} is actually displayed,
so there's a perception of duplication.

E.g. if a package has both a %post and a %preun scriptlet, then
there will be 2 "/bin/sh" dependencies displayed, each with a different
context marker.

Because the context markers are orthogonal bits, its feasible to unify
the dependencies. But there are some complexities in unduplicating
dependencies based soley on a union of context markers.

There's also a fair amount of "instant incompatibility" risk for a rather
small gain in the size of most headers. There are bigger gains to be
had in size reduction if headers were compressed in pkgs, where the
compressor would easily find the redundant strings in the Huffman dictionary

Changed in rpm:
status: New → Confirmed
importance: Undecided → Low
Revision history for this message
Jeff Johnson (n3npq) wrote :

There's also a boolean expression reduction that could be
attempted to eliminate one of these dependencies:

     Requires: foo > 1
     Requires: foo > 2

where the (implicit) && condition reduces to

     Requires: foo > 2

Again increased complexity for little gain imho.

Jeff Johnson (n3npq)
Changed in rpm:
status: Confirmed → Won't Fix
status: Won't Fix → Opinion
assignee: nobody → Jeff Johnson (n3npq)
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.