Red Hat Bugzilla – Bug 243387
Package should require yelp for its help
Last modified: 2007-11-30 17:12:06 EST
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.]
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
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
(In reply to comment #1)
> You should ask yourself: will the package fail to work if yelp is not installed?
> If not, do not require yelp.
Depends on what working means. Of course it is only needed for the
help. When I click on the help button in the interface nothing happens
if yelp isn't installed. And on the console I get, quite logically:
Traceback (most recent call last):
File "/usr/bin/gnochm", line 1565, in on_contents_activate
gobject.GError: There was an error launching the default action command
associated with this location.
So it isn't plain obvious that it isn't needed. From the user
perspective having help system working seems to be better to
Seems that yelp requires only the usual gnome stuff, and
firefox. So it doesn't seems to me to be really problematic
to bring in yelp. Maybe it would be better for yelp not to require
firefox itself, but a renderer engine, but that's not an issue
> If you have already committed this fix blindly, I ask that you consider undoing
I try to avoid doing things blindly.
> it so derivative projects can continue using your package without help if they
> so choose.
In that case they can still remove the Requires.
> 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.
comps is irrelevant, the Requires and comps don't embed the same
In my opinion, if you want project that use yelp not to require
it you should come with guideline, otherwise it should certainly
be left to the packagers.