Samba 3.0.32 - internal change notify "legacy" mechanism
Dina Fine
dina at exanet.com
Mon Jan 19 05:56:17 GMT 2009
Hi
I had a bug in the watch of vfs_notify_hash.c - The watch applied on the
directory relative path instead of the full path.
So when samba changed working directory, the watch failed.
The fix is attached.
Thanks
Dina
-----Original Message-----
From: Dina Fine
Sent: Monday, December 29, 2008 6:02 PM
To: 'Volker.Lendecke at SerNet.DE'
Cc: Shlomi Yaakobovich; Itay Dar; samba-technical at lists.samba.org
Subject: RE: Samba 3.0.32 - internal change notify "legacy" mechanism
Hi
Well, I applied all (almost :)) your comments.
We use Clear Case so I couldn't easily create a git patch, sorry for
that.
Thanks
Dina
-----Original Message-----
From: Volker Lendecke [mailto:Volker.Lendecke at SerNet.DE]
Sent: Monday, December 29, 2008 4:53 PM
To: Dina Fine
Cc: Shlomi Yaakobovich; Itay Dar; samba-technical at lists.samba.org
Subject: Re: Samba 3.0.32 - internal change notify "legacy" mechanism
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: vfs_notify_hash_patch
Type: application/octet-stream
Size: 1342 bytes
Desc: vfs_notify_hash_patch
Url : http://lists.samba.org/archive/samba-technical/attachments/20090119/84447b12/vfs_notify_hash_patch.obj
More information about the samba-technical
mailing list