Re: [PATCH 1/2] find: add SELinux support

2009-08-06 Thread Jim Meyering
Kamil Dudka wrote: > On Thu August 6 2009 12:36:06 Kamil Dudka wrote: >> On Thu August 6 2009 12:16:58 Jim Meyering wrote: >> > Kamil Dudka wrote: >> > > On Thu July 30 2009 13:57:00 Jim Meyering wrote: >> > >> > The attached incremental patch fixes it. Now it works on both Fedora >> > >> > and Deb

Re: [PATCH 1/2] find: add SELinux support

2009-08-06 Thread Jim Meyering
Kamil Dudka wrote: > On Thursday 06 of August 2009 14:01:37 Jim Meyering wrote: >> ... >> >> >> I'll move them to gnulib. >> > >> > Thanks! >> > >> > Just let me note there is no explicit dependency on the openat module >> > (namely lib/at-func.c). Is the dependency implied somehow? >> >> Good poin

Re: [PATCH 1/2] find: add SELinux support

2009-08-06 Thread Kamil Dudka
On Thursday 06 of August 2009 14:01:37 Jim Meyering wrote: > ... > > >> I'll move them to gnulib. > > > > Thanks! > > > > Just let me note there is no explicit dependency on the openat module > > (namely lib/at-func.c). Is the dependency implied somehow? > > Good point. > I'll add that module depen

Re: [PATCH 1/2] find: add SELinux support

2009-08-06 Thread Jim Meyering
... >> I'll move them to gnulib. > > Thanks! > > Just let me note there is no explicit dependency on the openat module (namely > lib/at-func.c). Is the dependency implied somehow? Good point. I'll add that module dependency. Thanks. I'll also do a stand-alone test, which should expose any other

Re: [PATCH 1/2] find: add SELinux support

2009-08-06 Thread Kamil Dudka
On Thu August 6 2009 13:23:38 Jim Meyering wrote: > Kamil Dudka wrote: > > On Thu August 6 2009 12:16:58 Jim Meyering wrote: > >> Kamil Dudka wrote: > >> > On Thu July 30 2009 13:57:00 Jim Meyering wrote: > >> >> > The attached incremental patch fixes it. Now it works on both > >> >> > Fedora and D

Re: [PATCH 1/2] find: add SELinux support

2009-08-06 Thread Jim Meyering
Kamil Dudka wrote: > On Thu August 6 2009 12:16:58 Jim Meyering wrote: >> Kamil Dudka wrote: >> > On Thu July 30 2009 13:57:00 Jim Meyering wrote: >> >> > The attached incremental patch fixes it. Now it works on both Fedora >> >> > and Debian. But unfortunately it does not work with the stable vers

Re: [PATCH 1/2] find: add SELinux support

2009-08-06 Thread Kamil Dudka
On Thu August 6 2009 12:36:06 Kamil Dudka wrote: > On Thu August 6 2009 12:16:58 Jim Meyering wrote: > > Kamil Dudka wrote: > > > On Thu July 30 2009 13:57:00 Jim Meyering wrote: > > >> > The attached incremental patch fixes it. Now it works on both Fedora > > >> > and Debian. But unfortunately it

Re: [PATCH 1/2] find: add SELinux support

2009-08-06 Thread Kamil Dudka
On Thu August 6 2009 12:16:58 Jim Meyering wrote: > Kamil Dudka wrote: > > On Thu July 30 2009 13:57:00 Jim Meyering wrote: > >> > The attached incremental patch fixes it. Now it works on both Fedora > >> > and Debian. But unfortunately it does not work with the stable version > >> > (4.4.2) of fin

Re: [PATCH 1/2] find: add SELinux support

2009-08-06 Thread Jim Meyering
Kamil Dudka wrote: > On Thu July 30 2009 13:57:00 Jim Meyering wrote: >> > The attached incremental patch fixes it. Now it works on both Fedora and >> > Debian. But unfortunately it does not work with the stable version >> > (4.4.2) of findutils. Could anybody point me to the relevant change in >>

Re: [PATCH 1/2] find: add SELinux support

2009-08-06 Thread Kamil Dudka
On Thu July 30 2009 13:57:00 Jim Meyering wrote: > > The attached incremental patch fixes it. Now it works on both Fedora and > > Debian. But unfortunately it does not work with the stable version > > (4.4.2) of findutils. Could anybody point me to the relevant change in > > findutils code? Thanks