RFC: new testparam check

Noel Power nopower at suse.com
Tue Apr 11 13:42:00 UTC 2017


Ok done, please see attached

Noel

On 11/04/17 12:37, David Disseldorp wrote:
> On Tue, 11 Apr 2017 12:26:26 +0100, Noel Power via samba-technical wrote:
>
>> On 11/04/17 12:08, Andreas Schneider wrote:
>>> On Tuesday, 11 April 2017 12:42:59 CEST Noel Power via samba-technical wrote:  
>>>> Hi,
>>>>
>>>> Had a situation where a customer had a weird problem, after much head
>>>> scratching and debugging it turned out to be a typo in one of the values
>>>> in 'name resolve order' causing the problem. Really would have helped
>>>> save alot of time if testparam complained especially as the values for
>>>> this param should be from a known set of valid values. I'll happily open
>>>> a samba bug and update the the patch to reflect that and any other
>>>> comments/issues if this is useful. Thoughts ?  
>>> RB+ ;-)
>>>  
>> ok, since at least 1 person thinks it's useful here is a version with a
>> bug # :->>
> Please add a test for this to ./source3/script/tests/test_testparm_s3.sh
> Otherwise, this looks good to me too.
>
>
> Cheers, David


-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-s3-script-tests-Test-for-illegal-value-detection-for.patch
Type: text/x-diff
Size: 1694 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20170411/f1750b85/0002-s3-script-tests-Test-for-illegal-value-detection-for.diff>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Check-for-valid-values-for-global-conf-name-resolve-.patch
Type: text/x-diff
Size: 4619 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20170411/f1750b85/0001-Check-for-valid-values-for-global-conf-name-resolve-.diff>


More information about the samba-technical mailing list