dist-git help wanted: write me a regex!

Jesse Keating jkeating at redhat.com
Mon Dec 21 17:24:08 UTC 2009


On Mon, 2009-12-21 at 11:07 -0500, James Cassell wrote:
> On Mon, 21 Dec 2009 01:53:42 -0500, Bruno Wolff III <bruno at wolff.to> wrote:
> 
> >
> >> /((Mon|Tues?|Wed|Thu(rs?)?|Fri|Sat|Sun)\s+(Jan|Feb|Mar|Apr|May|June?|July?|Aug|Sep|Oct|Nov|Dec)\s+[0-3]?[0-9]\s+(19|20)[0-9][0-9]\s+[A-Za-z0-9\s]+<[^\s@]+@[^\s@>]+>\s+2.[4-6].[0-9.-]+\s*)/
> > I don't think this will catch a period in the comment part of the email
> > address (as people often do after initials). Also if anyone is using  
> > hyphenated
> > names, I don't think those will get picked up. Since those entries are  
> > utf-8,
> > you need to worry about nonascii letters in the name. I am not sure how  
> > those
> > collate compared to ascii letters, but it might be safer to use [^<]+
> > (instead of [A-Za-z0-9\s]+)
> 
> You are correct.  Here's the improved version:
> /((Mon|Tues?|Wed|Thu(rs?)?|Fri|Sat|Sun)\s+(Jan|Feb|Mar|Apr|May|June?|July?|Aug|Sep|Oct|Nov|Dec)\s+[0-3]?[0-9]\s+(19|20)[0-9][0-9]\s+[^<]+<[^\s@]+@[^\s@>]+>\s+2.[4-6].[0-9.-]+\s*)/
> 
> -- 
> James Cassell
> 

I'm having some difficulty applying this.  It's going into a perl file
thusly:

$logmsg =~ s|/((Mon|Tues?|Wed|Thu(rs?)?|Fri|Sat|Sun)\s+(Jan|Feb|Mar|Apr|
May|June?|July?|Aug|Sep|Oct|Nov|Dec)\s+[0-3]?[0-9]\s+(19|20)[0-9][0-9]\s
+[^<]+<[^\s@]+@[^\s@>]+>\s+2.[4-6].[0-9.-]+\s*)/|mg

But that gives me Unmatched ( in regex; marked by <-- HERE in m/(( <--
HERE  or something along those lines.

The added "s|...|mg" is coming from other lines in this script which
look like:

$logmsg =~ s|^\s*\d\d*-\d\d*-\d\d*\s*[^<\n]*<[^>\n]*>\s*$|* \n|mg;

so I'm sure I'm screwing something up when putting it in the script.

-- 
Jesse Keating
Fedora -- Freedom² is a feature!
identi.ca: http://identi.ca/jkeating
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <http://listman.redhat.com/archives/fedora-devel-list/attachments/20091221/d6259611/attachment.sig>


More information about the fedora-devel-list mailing list