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 676482 - man: URLs can't include dots right now
man: URLs can't include dots right now
Status: RESOLVED FIXED
Product: yelp
Classification: Applications
Component: Man Pages
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Yelp maintainers
Yelp maintainers
: 677716 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2012-05-21 10:28 UTC by Lennart Poettering
Modified: 2012-09-04 14:20 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
improve man: uri support (1.37 KB, patch)
2012-05-27 16:46 UTC, Matthias Clasen 		none Details | Review
another attempt (1.46 KB, patch)
2012-08-04 20:09 UTC, Matthias Clasen 		committed Details | Review
small test (3.35 KB, text/x-csrc)
2012-08-04 20:10 UTC, Matthias Clasen 		  Details

Description Lennart Poettering 2012-05-21 10:28:09 UTC 
There are a number of man pages which have a "." in their name, but yelp fails to parse these as man urls. URL escaping the dot (i.e. writing %2E instead) doesn't work either.

It would be great if yelp could parse these URLs, since many man pages rely on this, in particular a large number which document file formats. Example: man:NetworkManager.conf
Comment 1 Matthias Clasen 2012-05-27 16:46:10 UTC 
Created attachment 215098 [details] [review]
improve man: uri support
Comment 2 Shaun McCance 2012-06-08 19:22:50 UTC 
*** Bug 677716 has been marked as a duplicate of this bug. ***
Comment 3 Matthias Clasen 2012-06-27 16:03:47 UTC 
Shaun, any comment on this patch ?
Comment 4 Matthias Clasen 2012-07-24 13:50:55 UTC 
Ping ?
Comment 5 Shaun McCance 2012-07-29 23:29:16 UTC 
I don't really want to break a URI syntax that's been supported for the last decade.
Comment 6 Matthias Clasen 2012-08-04 20:09:42 UTC 
Created attachment 220325 [details] [review]
another attempt

Here is another iteration of the patch. This one tries harder to keep things like man:printf.1 working.
Comment 7 Matthias Clasen 2012-08-04 20:10:35 UTC 
Created attachment 220326 [details]
small test

Here is a small test that I used to convince myself that the new regex is better
Comment 8 Matthias Clasen 2012-08-16 21:49:19 UTC 
Shaun ?
Comment 9 Shaun McCance 2012-08-20 23:08:25 UTC 
Fails for all of /usr/share/man/mann, or whatever other makeshift non-initial-numeric categories $RANDOM_DISTRO decides to add. I'm debating whether to care.