Bug 449705 - Should Requires(pre): /bin/sed
Summary: Should Requires(pre): /bin/sed
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: texinfo
Version: rawhide
Hardware: All
OS: Linux
low
low
Target Milestone: ---
Assignee: Vitezslav Crhonek
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-06-03 01:46 UTC by Jeremy Katz
Modified: 2008-06-05 11:02 UTC (History)
1 user (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2008-06-05 11:02:44 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Jeremy Katz 2008-06-03 01:46:23 UTC
%post scriptlet calls sed but package doesn't require sed to be installed first
leading to noise in install.log

Comment 1 Vitezslav Crhonek 2008-06-03 08:53:08 UTC
Jeremy, please teach me, why it should be Requires(pre): /bin/sed, when sed is
actually used in %post? At the moment, there's Requires(post): /bin/sed, this is
wrong?

Comment 2 Jeremy Katz 2008-06-03 14:47:57 UTC
Whoops, yeah, I just typo'd.

The problem is actually that you've introduced a dependency loop.  Both sed and
info require each other to be installed first.  Given that the sed invocation
was added to fix something cosmetic, it should probably be removed or at least
just be
  [ -x /bin/sed ] && /bin/sed ...
without the requires

Comment 3 Vitezslav Crhonek 2008-06-04 13:51:07 UTC
OK, thanks for clarification.


Note You need to log in before you can comment on or make changes to this bug.