[PATCH 2/2] errormsg: add descriptive macros to replace overloaded error codes
Richard Guy Briggs
rgb at redhat.com
Mon May 29 15:36:32 UTC 2017
On 2017-05-24 17:46, Richard Guy Briggs wrote:
> 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.
Ok, I hadn't understood that you hadn't applied it despite reviewing it
and finding it fine. I'll send it to the list now.
> > -Steve
>
> - RGB
- 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