[Crash-utility] [PATCH] add option -E for foreach

Dave Anderson anderson at redhat.com
Wed Feb 29 16:15:56 UTC 2012



----- Original Message -----
> 
> 
> ----- Original Message -----
> > 
> > 
> > ----- Original Message -----
> > > Hello Dave,
> > > 
> > > I have modified my patch. In my opinion, regular expression should not
> > > mix with other task-identifying arguments, so I made some more modifications
> > 
> > Why not?  It would be far more useful if you could maintain
> > the current capability of providing a mixture of pid/task/name
> > task-identifiers along with this new regular-expression-matching
> > scheme.
> > 
> > Dave
> 
> In fact, I've just reworked (and simplified) your patch so that a
> 'regex' task identifier may be used in conjunction with the other
> task identifier types, make the capability much more useful.
> 
> In fact, it could be modified even further to allow multiple
> regular expression arguments to be allowed, by maintaining
> an regex_t array the same way as it's done for the other
> supported task identifiers.  I'll try that...
> 
> Dave

Done!  With that capability, this a pretty powerful task-identifier.
Thanks for getting the ball rolling -- queued for crash-6.0.5.

Thanks,
  Dave





More information about the Crash-utility mailing list