Samba 3.0.32 - internal change notify "legacy" mechanism

Volker Lendecke Volker.Lendecke at SerNet.DE
Mon Dec 29 14:52:38 GMT 2008


On Mon, Dec 29, 2008 at 04:22:38PM +0200, Dina Fine wrote:
> Attached is a new vfs change notify module. The module handles only the

Thanks!

Some comments:

You don't need to patch the file "configure", this is
auto-generated from "configure.in" by ./autogen.sh

Same applies to config.h.in.

vfs_notify_hash.c does not carry a valid personal copyright.

Why is timed_event_handler not static?

The indentation in that file is a bit erratic, can you
reformat that so that it compiles with the stuff in
README.Coding? Not 100% required, but it would be great.

Can you make the timeout parameter a module-specific one?
For examples, look for example at the recycle module and its
use of lp_parm_int.

Can you split adding the module as such and the changes to
smbd/ into two patches? You might also want to use "git
format-patch" to send the patches. This way you would get
proper credit in "git blame". Bonus if you send patches that
apply cleanly to 3.2, 3.3 and master :-)

Thanks,

Volker
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : http://lists.samba.org/archive/samba-technical/attachments/20081229/e68e90df/attachment.bin


More information about the samba-technical mailing list