After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 331831 - Search results may not display all available man pages
Search results may not display all available man pages
Status: RESOLVED FIXED
Product: yelp
Classification: Applications
Component: Man Pages
git master
Other All
: Normal minor
: ---
Assigned To: Yelp maintainers
Yelp maintainers
Depends on: 339595
Blocks:
 
 
Reported: 2006-02-20 00:09 UTC by Andreas Kohn
Modified: 2006-05-11 18:12 UTC
See Also:
GNOME target: ---
GNOME version: 2.13/2.14



Description Andreas Kohn 2006-02-20 00:09:07 UTC
Please describe the problem:
yelp-search-pager.c(process_man_result) splits the result of running apropos
with the search term at the '-', and skips the result if the splitting resulted
in more than 2 parts.
This could (and does) happen if the manpage description and/or the manpage name
contain literal '-'s, for example:

mktexlsr(1)              - create ls-R databases

This result would be skipped.

Steps to reproduce:
Search using yelp for man pages containing '-' in their name or description.

Actual results:
Search results are skipped, i.e. you don't see those in the resulting page.

Expected results:
Manpages should be listed.

Does this happen every time?
Yes.

Other information:
I have no idea how to parse the apropos output differently to avoid this issue
without introducing other parsing problems, unfortunately.
Comment 1 Don Scorgie 2006-02-27 19:09:29 UTC
Confirming.  It affects a whole slew of man pages (try searching for apt).

Unfortunately, any fix is going to be a fair chunk of code and (basically) a rewrite of the process_man_result function (and probably process_info_result) which isn't going to happen before gnome 2.14.  I'll look at this again 2.14 is out.
Comment 2 Andreas Kohn 2006-03-08 07:28:20 UTC
I was just thinking if it could make sens to provide some information to the user,
a la "Sorry for the incovenience, but a few results were skipped. Use <whatever command> to view those"? 
Comment 3 Don Scorgie 2006-04-24 16:53:25 UTC
OK, new man result parser in bug #339595 (with various other improvements) - marking this as a dependant.
Comment 4 Don Scorgie 2006-05-11 18:12:19 UTC
Patch from bug #339595 has been applied to head.  This fixes the issues raised in this bug.  Closing.