Re: [PATCH] Add svnrdump

From

Ramkumar Ramachandra <artagnon@gmail.com>

Date

Jul 9, 2010, 13:43 UTC

Message-ID

<20100709134318.GC12315@debian>

In-Reply-To

<alpine.561.2.00.1007081113210.3936@daniel2.local>

Hi Daniel,

Daniel Shahaf writes:

> > > +  /* Use a temporary file to measure the text-content-length */
> > > +  apr_err = apr_temp_dir_get(&tempdir, hb->pool);
> 
> svn_io_temp_dir()

Fixed.

> > > +  if (apr_err != APR_SUCCESS)
> > > +    SVN_ERR(svn_error_wrap_apr(apr_err, NULL));
> > > +
> > > +  hb->temp_filepath = apr_psprintf(eb->pool, "%s/svn-fe-XXXXXX",
> > > tempdir);
> > 
> 
> os.path.join()
> 
> Err, I mean, svn_dirent_join().

Fixed. Thanks :)

-- Ram