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@xxxxxxxx> 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
Attachment:
signature.asc
Description: This is a digitally signed message part
-- fedora-devel-list mailing list fedora-devel-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/fedora-devel-list
