Bug 243378 - Package should require yelp for its help
Package should require yelp for its help
Product: Fedora
Classification: Fedora
Component: epiphany-extensions (Show other bugs)
All Linux
low Severity low
: ---
: ---
Assigned To: Peter Gordon
Fedora Extras Quality Assurance
: Reopened
Depends On: 243478
  Show dependency treegraph
Reported: 2007-06-08 12:25 EDT by Matěj Cepl
Modified: 2007-11-30 17:12 EST (History)
2 users (show)

See Also:
Fixed In Version: 2.19.2-4
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2007-06-10 00:12:59 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)

  None (edit)
Description Matěj Cepl 2007-06-08 12:25:12 EDT
This package uses yelp for its help. Therefore, it should require package yelp for its operation.

[Filed by a script; forgive him if he made a mistake for once.]
Comment 1 Peter Gordon 2007-06-09 00:42:31 EDT
Hmm. Will fix this momentarily. Thanks for the catch...
Comment 2 Peter Gordon 2007-06-09 01:16:56 EDT
Built into dist-f8; and it should be fixed with the next rawhide updates. Thanks.
Comment 3 Christopher Aillon 2007-06-09 09:17:40 EDT
Ugh.  No...
Comment 4 Christopher Aillon 2007-06-09 09:20:04 EDT
1. Does epiphany-extensions require help in order to function?  NO.
2. Does not having yelp installed lose MAJOR functionality?  NO.  (Just help)

Also, this requires would belong better in epiphany, not -extensions.  This
should be WONTFIX.
Comment 5 Christopher Aillon 2007-06-09 09:24:31 EDT
I deleted the build, so it will not appear in rawhide.  Please revert from cvs.
Comment 7 Matěj Cepl 2007-06-09 18:43:47 EDT
Whole story is in bug 243478 which this bug depends on.
Comment 8 Christopher Aillon 2007-06-09 18:51:15 EDT
All of these bug reports are (most likely) wrong.  The package should NOT
require yelp.  We went through effort to ensure that yelp is not a requirement
for the desktop.

You should ask yourself: will the package fail to work if yelp is not installed?
 If not, do not require yelp.

If you have already committed this fix blindly, I ask that you consider undoing
it so derivative projects can continue using your package without help if they
so choose.

We have added yelp to the default install via comps so that all installs get
yelp by default unless they specifically ask for it to be removed.  We should
honor that.  Please undo your changes if you have made them and/or mark this bug
WONTFIX.  Thanks.
Comment 9 Peter Gordon 2007-06-10 00:12:59 EDT
Yelp dependency reverted. Thanks for the catch, Chris. 

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