Re: [PATCH] Sleep for timestamps in the right places
From: Julian Foad <julianfoad_at_btopenworld.com>
Date: Tue, 2 Apr 2013 21:04:24 +0100 (BST)
Philip Martin wrote:
OK, thanks; I did that (inside update_internal()), and a few more things, and committed r1463721.
For the future, it seems to me that the tracking of whether we need to sleep should be done inside libsvn_wc, as only there do we really know whether we have made a change that relies on the timestamp. The current code is all in libsvn_client and can only assume that if it called something like svn_wc_revert or svn_wc_do_update then it will need to sleep, which is a crude assumption (not always true). We would also have the ability to store more detailed information there, such as the latest timestamp that we are relying on, so that we wouldn't have to sleep for 1 second starting from when we start to sleep but only starting from when the last WC update was, which might have been some time ago in some cases. But I'm not planning on doing any of that.
This is an archived mail posted to the Subversion Dev mailing list.