[Bug 12285] gnome-terminal doesn't match nntp or file URLs.

[bugzilla-daemon at bugzilla.ubuntu.com]

Please do not reply to this email.  You can add comments at
http://bugzilla.ubuntu.com/show_bug.cgi?id=12285
Ubuntu | gnome-terminal





------- Additional Comments From ralph at inputplus.co.uk  2005-06-30 14:13 UTC -------
(In reply to comment #1)
> Thanks for your bug, what is the issue with "file"?

Oops.  You're right, `file' isn't missing from SCHEME's definition.  The
problem is the regular expression that uses SCHEME.

    #define USERCHARS "-A-Za-z0-9"
    #define PASSCHARS "-A-Za-z0-9,?;.:/!%$^*&~\"#'"
    #define HOSTCHARS "-A-Za-z0-9"
    #define SCHEME    "(news|telnet|nttp|file|https?|ftps?|webcal)"
    #define USER      "[" USERCHARS "]+(:["PASSCHARS "]+)?"

      terminal_widget_match_add (screen->priv->term,
          "\\<(" SCHEME "://(" USER "@)?)[" HOSTCHARS ".]+(:[0-9]+)?"
          "(/[-A-Za-z0-9_$.+!*(),;:@&=?/~#%]*[^]'.}>) \t\r\n,\\\"])?\\>",
          FLAVOR_AS_IS);

It insists on there being one or more of [HOSTCHARS.] after the first
two slashes so it fails to match `file:///etc/passwd'.  I don't know if
any other of the listed schemes can also have `///'.

> I've forwarded it upstream:
> http://bugzilla.gnome.org/show_bug.cgi?id=309201

OK, what's the convention?  Do I continue to add to this bug knowing
that they'll read it, e.g. the above explanation?

-- 
Configure bugmail: http://bugzilla.ubuntu.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.