svn commit: samba r23108 - in branches: SAMBA_3_0/source/lib/replace SAMBA_3_0_26/source/lib/replace SAMBA_4_0/source/lib/replace

derrell.lipman at derrell.lipman at
Thu May 24 13:22:14 GMT 2007

"Stefan (metze) Metzmacher" <metze at> writes:

> Stefan (metze) Metzmacher schrieb:
>> derrell.lipman at schrieb:
>>> "Stefan (metze) Metzmacher" <metze at> writes:
>>>> seems that revision 22772 wasn't merged into all branches...
>>> Sorry, I think I merged it to all of the Samba3 branches, but it didn't
>>> occur to me that the same code was used in Samba4.
>>>> maybe we should better move this to samba3 instead of libreplace...
>>>> as libreplace will not provide replacements for readahead().
>>> That change should be merged into libreplace in Samba4.  The problem it
>>> was intended to solve -- an existing readahead() function but no header
>>> that declares it -- would exist just as much in Samba4 as in Samba3
>>> (even if nothing in Samba4 currently uses readahead).  Please go ahead
>>> and merge that in with the current work.  (There's also a configure test
>>> that was added in the same patch.)
>> Hi Derell,
>> I know of the initial problem.
>> But as the test for readahead function is in in samba3
>> the test for the prototype should also be there, as it really only
>> belongs to samba3.
> and we do exactly this for a lot of other functions in samba3's
> and include/includes.h...

Ok, so there's no attempt to keep samba3's and samba4's libreplace the


More information about the samba-technical mailing list