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 518673 - Extra slash in devhelp file makes external link broken
Extra slash in devhelp file makes external link broken
Status: RESOLVED FIXED
Product: glibmm
Classification: Bindings
Component: documentation
2.15.x
Other Linux
: Normal trivial
: ---
Assigned To: gtkmm-forge
gtkmm-forge
Depends on:
Blocks:
 
 
Reported: 2008-02-25 18:40 UTC by Deng Xiyue
Modified: 2008-02-26 01:49 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
10_devhelp_external_link_fix.patch (874 bytes, patch)
2008-02-25 18:45 UTC, Deng Xiyue
none Details | Review
alternative patch for glibmm devhelp fix (3.03 KB, patch)
2008-02-25 19:07 UTC, Jonathon Jongsma
committed Details | Review

Description Deng Xiyue 2008-02-25 18:40:57 UTC
The extra slash breaks the structure of references to other level of path hierarchy.  Fix is trivial.
Comment 1 Deng Xiyue 2008-02-25 18:45:03 UTC
Created attachment 105928 [details] [review]
10_devhelp_external_link_fix.patch

Forget to mention an example: The header link that goes back to gtkmm main page will be broken.
Comment 2 Jonathon Jongsma 2008-02-25 18:46:38 UTC
In that example, it will be broken before the patch but fixed after the patch?
Comment 3 Deng Xiyue 2008-02-25 18:55:34 UTC
Yep, at least reproducible on my Debian sid.
Comment 4 Jonathon Jongsma 2008-02-25 19:07:49 UTC
Created attachment 105930 [details] [review]
alternative patch for glibmm devhelp fix

Deng, could you test this patch instead?  The file that you patched was a generated file.
Comment 5 Deng Xiyue 2008-02-25 20:51:34 UTC
Thanks.  It'll work after proper regeneration.
Comment 6 Jonathon Jongsma 2008-02-26 01:49:27 UTC
Thanks for testing. committed as r623