On Fri, Mar 11, 2011 at 2:26 PM, David Woodhouse <dwmw2 at infradead.org> wrote: > On Fri, 2011-03-11 at 14:24 -0500, Ray Kohler wrote: >> >> I do see one very small Makefile bug right now - after the auth-dialog >> is gone, the install target probably shouldn't be creating >> /usr/libexec any longer, since nothing here needs it. (Also, how about >> installing the manpage during make install? Or was it a deliberate >> decision not to do that?) > > Very good points both; thanks. > > Criticism is preferred in 'diff -up' form though... :) Ok, here is your patch-formatted feedback. Hopefully gmail doesn't mangle it, since I can't send mail directly from my dev box.