[PATCH] Introduce smbspool_krb5_wrapper bug #11690

Andreas Schneider asn at samba.org
Fri Jan 22 22:11:35 UTC 2016


On Friday, January 22, 2016 10:26:42 PM Ralph Boehme wrote:
> 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. :)))

How about this?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: master-smbspool_krb5_wrapper_v2.patch
Type: text/x-patch
Size: 8477 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20160122/5469cc92/master-smbspool_krb5_wrapper_v2.bin>


More information about the samba-technical mailing list