[PATCH] Make loadparm more common

[Andrew Bartlett]

On Fri, 2014-03-28 at 13:42 +1300, Andrew Bartlett wrote:

> 
> Even more patches are at
> http://git.catalyst.net.nz/gitweb?p=samba.git;a=shortlog;h=refs/heads/loadparm-talloc-to-review3
> 
> Please review!  (I do realise that reviewing 98 patches will take some
> time, but if we can make a start on the first 14 that would be most
> helpful).

Garming,

I've spent much of today running tests on the loadparm code, and I've
added in a talloc_tos() related fix at the start of the series. 

However, I also decided to run a test on every revision, to ensure we
don't end up in bisect failures in the future.  The problem test is the
test_defaults_s4 in the docs.py test, and starting at this commit it
breaks:

http://git.catalyst.net.nz/gitweb?p=samba.git;a=commitdiff;h=de2c4a7a4303f050042be2a7627ea6e6860a4201

This commit fixes it:
http://git.catalyst.net.nz/gitweb?p=samba.git;a=commitdiff;h=32a4428628ba7f75d4a7fccfeea8eaeaf671450e

The rest of the tree seems fine, and so we can still ask for the first
55 patches to be reviewed while we work on these. 

BTW, The test I'm running on every single commit is:

make -j && make test TESTS="docs libsmbclient netapi testparm"

Additionally, the branch as a whole passes a full autobuild on
sn-devel. 

Team,

It would be great to get a review on this, so we can get at least the
first half into master.

Thanks,

Andrew Bartlett

-- 
Andrew Bartlett
http://samba.org/~abartlet/
Authentication Developer, Samba Team  http://samba.org
Samba Developer, Catalyst IT          http://catalyst.net.nz/services/samba