[PATCH] Introduce smbspool_krb5_wrapper bug #11690
Ralph Boehme
rb at sernet.de
Fri Jan 22 21:26:42 UTC 2016
On Fri, Jan 22, 2016 at 10:18:30PM +0100, Andreas Schneider wrote:
> On Friday, January 22, 2016 05:03:07 PM Ralph Boehme wrote:
> > Hi!
>
> Hi Ralph,
>
> > Nice work!
>
> thanks.
>
> > I may be completely off, but wouldn't it make sense to
> >
> > On Fri, Jan 22, 2016 at 04:38:30PM +0100, Andreas Schneider wrote:
> > > + return execv("/usr/bin/smbspool", argv);
> >
> > preserve PATH and use execvp("smbspool", argv) instead in order to
> > avoid having the patch hardcoded?
>
> Yes, good idea. Or we could pass the install prefix to it. Do you want to
> implement it or should I do it on Monday? :-)
it's your patchset, I'm just nitpicking. :)))
Have a nice weekend!
-Ralph
--
SerNet GmbH, Bahnhofsallee 1b, 37081 Göttingen
phone: +49-551-370000-0, fax: +49-551-370000-9
AG Göttingen, HRB 2816, GF: Dr. Johannes Loxen
http://www.sernet.de,mailto:kontakt@sernet.de
More information about the samba-technical
mailing list