Re: [Wireshark-commits] rev 37408: /trunk/epan/ /trunk/epan/: radius_dict.l

[Jeff Morriss <jeff.morriss.ws () gmail com>]

Guy Harris wrote:
> On May 26, 2011, at 7:15 AM, morriss () wireshark org wrote:
>
> > http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=37408
> >
> > User: morriss
> > Date: 2011/05/26 07:15 AM
> >
> > Log:
> > Escape the '.' in radius dictionary ATTRIBUTE names.  This may fix the OSX-10.5 builds too.
> >
> > Directory: /trunk/epan/
> >  Changes    Path             Action
> >  +4 -4      radius_dict.l    Modified
>
> Nope - 10.5 has, as I remember, an older version of Flex than 10.6, and it might not support that backslash escaping.  
> I'll see if I can fire up a 10.5 partition at work and see what version of Flex it has.

Well, the 10.5 buildbots didn't like r37405 (without the backslash). 
The backslash seemed to make sense (otherwise that period will match any 
character, right?) but the part about it fixing the builds was a bit of 
a punt.  Which obviously has missed.

The only other idea I have is to move the hyphen just before the close 
bracket to the start of the bracketed sequence.  Not sure why that would 
help, though...
___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev () wireshark org>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
            mailto:wireshark-dev-request () wireshark org?subject=unsubscribe