[PATCH] Fix the last of the defaults and include a test to check them

Garming Sam garming at catalyst.net.nz
Mon Feb 3 16:42:45 MST 2014


On 03/02/14 19:48, Michael Adam wrote:
> Ok, I need to look more thoroughly, but it seems
> that you kept a lot of the patches introducing the
> "function" metadata - those which I flagged "TODO" in my
> master-param branch. Ideally we would avoid pushing those
> to master, since they would be reverted after the corresponding
> renames anyways ... I will follow up later.
>
> Cheers - Michael
>

On top of the branch I just gave you, there's some other conflicting 
stuff, but certainly work in progress. Notably an initial attempt to 
remove the ctx table and s3_param.h. There's an attempt to perform 
substitutions with the lp string functions in the source4 code, so these 
changes are making a hit: 
http://git.catalyst.net.nz/gitweb?p=samba.git;a=commitdiff;h=5a461ece3303bf001797bf349af21e0a3897397c



git://git.catalyst.net.nz/samba.git  param-wip

http://git.catalyst.net.nz/gitweb?p=samba.git;a=shortlog;h=refs/heads/param-wip

Just looking through the remaining meta-data patches, it looks like 
they're all simply the ones which add a function name and were 
originally marked TODO.

I know it's a bit ugly to revert, but I think the only real alternative 
would be to really get those renames right here and now and so I can fix 
it all at once.


Cheers,

Garming Sam


More information about the samba-technical mailing list