This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [patch] break-interp.exp: handle /usr/sbin/prelink.bin


On Thu, Mar 18, 2010 at 12:15 AM, Mike Frysinger <vapier@gentoo.org> wrote:
> On Thursday 18 March 2010 02:15:07 Doug Evans wrote:
>> - ? ?if {$result == 1 && [regexp {^(/usr/sbin/prelink: [^ ]* does not have
>> .gnu.prelink_undo section\n?)*$} $output]} {
>> + ? ?if {$result == 1 && [regexp {^(/usr/sbin/prelink[^ ]*: [^ ]* does not
>> have .gnu.prelink_undo section\n?)*$} $output]} {
>
> since the prelink filename shouldnt have a colon in it, should that be used in
> the regex instead ?
> /usr/sbin/prelink[^:]*:
> -mike

Either is arguably wrong IMO.  Why can't the program have a colon or
space in its name?
But I don't have a strong opinion on which one to actually use, or
something else.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]