[PATCH] Fix 'net ads changetrustpw'

Andreas Schneider asn at samba.org
Thu Aug 10 07:30:26 UTC 2017


On Wednesday, 9 August 2017 19:03:34 CEST Rowland Penny via samba-technical 
wrote:
> On Wed, 09 Aug 2017 18:45:10 +0200
> Andreas Schneider via samba-technical <samba-technical at lists.samba.org>
> 
> wrote:
> > Hi,
> > 
> > as always, untested code is broken code. So I broke 'net ads
> > changetrustpw' in Samba 4.6.
> > 
> > The attached patch fixes the issue and adds a tests which verifies
> > that it is working.
> > 
> > 
> > Please review and push if OK.
> > 
> > 
> > Thanks,
> > 
> > 	Andreas
> 
> OK, I asked Andrew this and got a very non committal answer, so this
> seems to a good place to ask everyone, why does Samba use the ancient
> backticks ?
> 
> i.e `expr $failed + 1` instead of $((failed + 1))

I normally use $() but as this is copy and paste it is using the old way. On 
newer scripts you can see $() in my scripts at least.

If you want to can start changing it in all scripts :-)


	Andreas

-- 
Andreas Schneider                   GPG-ID: CC014E3D
Samba Team                             asn at samba.org
www.samba.org



More information about the samba-technical mailing list