# HG changeset patch # User Nicolas Dumazet # Date 1247496545 -7200 # Node ID f90bbf1ea09f4d704d31630143a5ada4078a7ff0 # Parent b55d44719b47eafc2ecbf313c169c5d3b98541e3 inotify: fix issue1375, add a test. The biggest problem was the data structure, which did not allow changing a file into a directory or vice versa. This problem is fixed by b55d44719b47. The walk() method also had an issue in this case: - we know 'path' as a directory. inotify server sleeps. - 'path' is deleted - 'path' is recreated as a file - the server catches up here, and see the deletion. it instantiates a scan(), which in its turn calls for walk(repo, path). - walk() then assumes that 'path' is a directory and calls os.listdir on it, which raises an OSError(errno.ENOTDIR) Catch the error, and yield the file instead of the directory contents. diff -r b55d44719b47 -r f90bbf1ea09f hgext/inotify/server.py --- a/hgext/inotify/server.py Tue May 26 23:00:35 2009 +0900 +++ b/hgext/inotify/server.py Mon Jul 13 16:49:05 2009 +0200 @@ -89,7 +89,11 @@ yield fullpath, dirs, files except OSError, err: - if err.errno not in walk_ignored_errors: + if err.errno == errno.ENOTDIR: + # fullpath was a directory, but has since been replaced + # by a file. + yield fullpath, dirs, files + elif err.errno not in walk_ignored_errors: raise return walkit(root, root == '') diff -r b55d44719b47 -r f90bbf1ea09f tests/test-inotify --- a/tests/test-inotify Tue May 26 23:00:35 2009 +0900 +++ b/tests/test-inotify Mon Jul 13 16:49:05 2009 +0200 @@ -35,4 +35,19 @@ hg status . cd .. +#issue 1375 +#Testing that we can remove a folder and then add a file with the same name +echo % issue 1375 + +mkdir h +echo h > h/h +hg ci -Am t +hg rm h + +echo h >h +hg add h + +hg status +hg ci -m0 + kill `cat hg.pid` diff -r b55d44719b47 -r f90bbf1ea09f tests/test-inotify.out --- a/tests/test-inotify.out Tue May 26 23:00:35 2009 +0900 +++ b/tests/test-inotify.out Mon Jul 13 16:49:05 2009 +0200 @@ -32,3 +32,9 @@ ? hg.pid M dir/x M x +% issue 1375 +adding h/h +adding hg.pid +removing h/h +A h +R h/h