[PATCH 25/37] param: replace lp_set_cmdline in s3 helpers to store_cmdline
Andrew Bartlett
abartlet at samba.org
Sun Jul 6 21:17:36 MDT 2014
On Mon, 2014-07-07 at 15:01 +1200, Garming Sam wrote:
> Hi,
>
> I've found the cause of the issue, which was that the command-line
> parametric options weren't being overridden correctly. The new patch
> gets rid of special casing which caused the problem in the first
> place.
>
> The rest of the patches which were reviewed remain the same, it's just
> these two patches which have changed and actually need to be
> re-reviewed.
>
> Patches changed:
> param: replace lp_set_cmdline in s3 helpers to store_cmdline
> param: Amend docs.py test to check dumping of flagged parameters
>
> The extra test added for docs.py was also amended.
>
>
> http://git.catalyst.net.nz/gitweb?p=samba.git;a=shortlog;h=refs/heads/before-service-ptrs-review3
> git://git.catalyst.net.nz/samba.git before-service-ptrs-review3
Reviewed-by: Andrew Bartlett <abartlet at samba.org>
Can I have another team reviewer of these two:
http://git.catalyst.net.nz/gitweb?p=samba.git;a=commitdiff;h=485603b8294f0d7561ccda42dfd807646b89b9e4
http://git.catalyst.net.nz/gitweb?p=samba.git;a=commitdiff;h=9f2696129e8e7159e9346b6c940b7783f75de819
Thanks!
(I still have a record of the other reviews done, and will apply those
tags before pushing).
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
More information about the samba-technical
mailing list