[PATCH 2/2] errormsg: add descriptive macros to replace overloaded error codes

Richard Guy Briggs rgb at redhat.com
Wed May 24 21:46:58 UTC 2017


On 2017-05-24 16:02, Steve Grubb wrote:
> On Monday, May 8, 2017 9:52:00 AM EDT Richard Guy Briggs wrote:
> > > > Ok, so coming back to patch acceptance, if I read correctly your
> > > > comments, reduce the four new error types to two?
> > > 
> > > Yes, two are needed. One for missing filter/action and one for we are
> > > attempting an incompatible comparison between fields.
> > 
> > Ok, here you go:
> >         https://github.com/linux-audit/audit-userspace/pull/21
> 
> Was just going to look over this patch and now I see its not on the mail list. 
> This version looks fine. I kind of prefer patches sent to the mail list so 
> that any discussion is archived. 

No problem.  I'll continue sending patches to the list.  Now that the
repo is on github, it is so much more convenient to just push the
changes on one of my branches and make a pull request.  I expect it is
equally easy to accept a pull request and merge straight on github.

> -Steve

- RGB

--
Richard Guy Briggs <rgb at redhat.com>
Sr. S/W Engineer, Kernel Security, Base Operating Systems
Remote, Ottawa, Red Hat Canada
IRC: rgb, SunRaycer
Voice: +1.647.777.2635, Internal: (81) 32635




More information about the Linux-audit mailing list