Re: Revert r23859 - conditionalize skipped sleeping for timestamps on maintainer mode

On 11/3/07, Justin Erenkrantz <justin@erenkrantz.com> wrote:
> On Nov 3, 2007 10:49 PM, David Glasser <glasser@davidglasser.net> wrote:
> > svn_sleep_for_timestamps exists for a very real reason. Making it
> > easy to disable will make it easy for users to hurt themselves. If
> > all you need is to set an environment variable to make svn "magically
> > faster", then users will do so cargo-cultishly and hurt themselves.
> > (And blame svn.)
>
> I would be +1 to renaming the environment variable to:
>
> SVN_I_LOVE_CORRUPTED_WORKING_COPIES_SO_DISABLE_SLEEP_FOR_TIMESTAMPS
>
> =)
>
> Let's make it downright explicit in the variable name that they're
> shooting themselves in the foot if they don't know what they are
> doing.
>
> I also believe that there are file systems that have sub-second
> filestamps, so on those, it should in theory be safe (as long as we do
> the right things to preserve it - which I *think* we do). -- justin

Yup, that's exactly what I'm suggesting. (And doing more research on
the sub-second thing wouldn't kill either.)

--dave

-- 
David Glasser | glasser_at_davidglasser.net | http://www.davidglasser.net/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org