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(a)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