Ben Collins-Sussman wrote:
> On Mon, 2004-01-12 at 04:29, Stuart Robertson wrote:
>
>>The requirement for 'svnlook diff' to be run from directory to which it has
>>write permissions should perhaps be documented somewhere. Or, the svnlook
>>should perhaps be updated to write to create temporary files in /tmp and not
>>try to write directly to the working directory.
>
> Actually, we've known about this bug for a long time, and have been
> waiting for APR to grow a function that returns an OS-specific tmpdir.
> APR *has* that function now, so it would be trivial to make svnlook's
> main.c use that function instead of this current code:
>
> /* Temporary subdirectory created for use by svnlook. ### todo: we
> need to either find a way to query APR for a suitable (that is,
> writable) temporary directory, or add this #define to the
> svn_private_config.h stuffs (with a default of perhaps
> "/tmp/.svnlook" */
> #define SVNLOOK_TMPDIR ".svnlook"
>
> Anyone wanna submit a patch?
Sure. Fixed in trunk r8253.
/Tobias
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Jan 12 15:23:14 2004