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 607498 - cannot open info sections that have "/" in title
cannot open info sections that have "/" in title
Status: RESOLVED FIXED
Product: yelp
Classification: Applications
Component: Info Pages
2.28.x
Other Linux
: Normal normal
: Fix Now
Assigned To: Yelp maintainers
Yelp maintainers
Depends on:
Blocks:
 
 
Reported: 2010-01-19 21:48 UTC by Edward Sheldrake
Modified: 2010-04-26 15:23 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Quick hack fix (1.17 KB, patch)
2010-01-19 21:48 UTC, Edward Sheldrake 		none Details | Review
refreshed patch for 2.30.0 (2.16 KB, patch)
2010-04-09 18:31 UTC, Edward Sheldrake 		none Details | Review

Description Edward Sheldrake 2010-01-19 21:48:56 UTC 
Created attachment 151788 [details] [review]
Quick hack fix

Yelp cannot display sections of .info files that have titles containing a "/" (forward slash) character.

Steps to reproduce:
1. Open grub's info page in yelp (run "yelp info:grub")
2. Click "* Booting::"
3. Click "* OS-specific notes::"
4. Click "* GNU/Linux::"

Actual results:
An error dialog "Page not found" "The page GNU was not found in the document /usr/share/info/grub.info.gz."

The "GNU/Hurd" and "DOS/Windows" sections give a similar error. Other info documents are affected, e.g. gawk's info has a section "Reading Files" -> "Getline", clicking "Getline/Variable" opens the "Getline" section.

Additional info:
Running yelp 2.28.1 from Fedora 12.
Comment 1 Jonathon Jongsma 2010-01-22 20:54:41 UTC 
This bug affects the gdb info page as well (e.g. section 'Running' > 'Input/Output', etc)
Comment 2 Edward Sheldrake 2010-04-09 18:31:43 UTC 
Created attachment 158319 [details] [review]
refreshed patch for 2.30.0

The same patch, but for 2.30.0, and with fixed coding style and re-indenting a block of code.

I will also explain the patch: on investigating this problem, I found the comment "FIXME: Super hacky.  We want the part before the slash for mallard IDs". The patch is supposed to make the special slash handling only be done for mallard documents, not all document types.
Comment 3 Shaun McCance 2010-04-09 21:42:04 UTC 
Looks good. Do you have write access to git.gnome.org? If so, please push this to the the gnome-2-30 branch only. master is a very different animal.
Comment 4 Edward Sheldrake 2010-04-10 10:50:30 UTC 
(In reply to comment #3)
> Looks good. Do you have write access to git.gnome.org? If so, please push this
> to the the gnome-2-30 branch only. master is a very different animal.

No, I don't have write access to git.gnome.org, I'm not a GNOME developer.
Comment 5 Shaun McCance 2010-04-26 15:23:09 UTC 
Fixed in 2.30.1. Thanks.