svn commit: samba r3737 - in branches/SAMBA_4_0/source: auth build/pidl build/smb_build lib lib/dcom lib/dcom/common lib/registry lib/registry/common lib/registry/reg_backend_dir lib/registry/reg_backend_gconf lib/registry/reg_backend_ldb lib/registry/reg_backend_nt4 lib/registry/reg_backend_rpc lib/registry/reg_backend_w95 lib/tdb libcli/auth librpc/rpc ntvfs ntvfs/cifs ntvfs/ipc ntvfs/nbench ntvfs/posix ntvfs/print ntvfs/simple ntvfs/unixuid rpc_server rpc_server/dcom rpc_server/remote smbd

Stefan (metze) Metzmacher metze at samba.org
Mon Nov 15 10:13:36 GMT 2004


Jelmer Vernooij schrieb:

> Stefan (metze) Metzmacher wrote:
> 
>> Jelmer Vernooij schrieb:
>>
>>> Stefan (metze) Metzmacher wrote:
>>>
>>>> jelmer at samba.org schrieb:
>>>>
>>>>> Author: jelmer
>>>>> Date: 2004-11-14 22:23:23 +0000 (Sun, 14 Nov 2004)
>>>>> New Revision: 3737
>>>>>
>>>>> WebSVN: 
>>>>> http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=3737 
>>>>>
>>>>>
>>>>> Log:
>>>>> - Get rid of the register_subsystem() and register_backend() 
>>>>> functions. 
>>>>
>>>>
>>>>
>>>>
>>>>
>>>> why? there was a very good reason for adding them.
>>>
>>>
>>>
>>>
>>> Originally, yes. However, they have a few issues:
>>>
>>> - Registration of a subsystem has to happen before registration of a 
>>> module
>>
>>
>> I don't see how using subsystem specific function solve that
>>
>>> - No type checking 
>>
>>
>> that's noe really a reason I think
>>
>>> - Need to recompile the files containing the _static_init_modules 
>>> macros after the configration changes.
>>
>>
>> I don't see how using subsystem specific function solve that
> 
> 
> This fixes the problem with module init functions being called earlier 
> then subsystem init functions, rather then the problem with shared 
> modules (which didn't work yet anyway).

the build system should just sort the functions and call the subsystems init functions first and 
then the module one's.



-- 
metze

Stefan Metzmacher <metze at samba.org> www.samba.org


More information about the samba-technical mailing list