Thank you, I'll fix it when I get home, but I wish I didn't have to type all those +ssh for bzr, git, hg, svn... On Sep 24, 2012 10:47 AM, "Pim Vullers" <[email protected]> wrote:
> Normally a + in regexp means 'or', so you need to escape it. I you > should use guess \+, so git\+ssh. But you should check the escape > character in the regexp/function documentation. > > On 09/24/2012 11:44 AM, David Gomes wrote: > > Indeed, I fixed the ? on the branch, but the + sure looks weird. Any > ideas? > > > > David "Munchor" Gomes > > > > On Sep 24, 2012 10:04 AM, "Pim Vullers" <[email protected] > > <mailto:[email protected]>> wrote: > > > > The question mark can be used to include both http and https by just > > writing https? (which makes the s optional). I agree that Sergey's > list > > (in combination with the one already listed, and I guess the + needs > > some escape in the regexp) should suffice. You definitely do not > want to > > use * since that would allow invalid schemes to become clickable > which > > does not help anybody. > > > > On 09/24/2012 10:46 AM, David Gomes wrote: > > > Thank you Shnatsel. What do you guys think of Mefrio's option? If > > I had > > > "*" for all protocols then "omgwtf://" would be a protocol and > that's > > > not right, is it? > > > > > > Regarding the RegEx syntax, I'll take another look at it because > you > > > seem to be right. > > > > > > On Sun, Sep 23, 2012 at 12:16 PM, Sergey "Shnatsel" Davidoff > > > <[email protected] <mailto:[email protected]> > > <mailto:[email protected] <mailto:[email protected]>>> > > wrote: > > > > > > 2012/9/23 David Gomes <[email protected] > > <mailto:[email protected]> > > > <mailto:[email protected] <mailto:[email protected] > >>> > > > > > > Hey everyone, > > > > > > > > > https://code.launchpad.net/~elementary-apps/pantheon-terminal/new-protocols > > > > > > If you have some free time, please check that branch and > add > > > some new protocols (or tell us about other ones with a > > reply to > > > this email): > > > > > > > > > > > > this.clickable("""(http?|ftp?|https?|ftps?|irc?|sftp?|mailto?)://\S+"""); > > > > > > The other day we discussed this on IRC and shnatsel and > > voldyman > > > told me about lots of protocols. The channel wasn't being > > logged > > > at the time and I forgot some of the protocols they > suggested. > > > > > > > > > The following URI schemes are relevant in a terminal: > > > > > > ldap, ldaps, nfs, smb, rsync, ssh, rlogin, telnet, git, > git+ssh, > > > bzr, bzr+ssh, svn, svn+ssh > > > > > > Depending on the regexp syntax (and you seem to have a really > > weird > > > one here, with "?" after every URI option - that typically > > makes the > > > last character optional) and implementation details, you also > > might > > > want to add matching a delimiter before the URI scheme, so that > > > "omgwtfhttp://"; won't be treated as a URl. > > > > > > -- > > > Sergey "Shnatsel" Davidoff > > > OS architect @ elementary > > > > > > > > > > > > > > > > > > -- > > Mailing list: https://launchpad.net/~elementary-dev-community > > Post to : [email protected] > > <mailto:[email protected]> > > Unsubscribe : https://launchpad.net/~elementary-dev-community > > More help : https://help.launchpad.net/ListHelp > > > > > -- > Mailing list: https://launchpad.net/~elementary-dev-community > Post to : [email protected] > Unsubscribe : https://launchpad.net/~elementary-dev-community > More help : https://help.launchpad.net/ListHelp >
-- Mailing list: https://launchpad.net/~elementary-dev-community Post to : [email protected] Unsubscribe : https://launchpad.net/~elementary-dev-community More help : https://help.launchpad.net/ListHelp
