[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