Re: svn commit: r1700799 - in /subversion/trunk/subversion: include/svn_io.h libsvn_subr/stream.c svnadmin/svnadmin.c svnfsfs/load-index-cmd.c tests/libsvn_subr/stream-test.c

2015-09-11 Thread Evgeny Kotkov
Julian Foad writes: > Also for: >svnrdump load >svnmucc put - > > ? (These just came to mind when I thought about it.) I think, yes. Thank you for pointing this out, Julian. Regards, Evgeny Kotkov

Re: [PATCH] Fix typo in Italian translation

2015-09-11 Thread Daniel Shahaf
Andrea Pescetti wrote on Fri, Sep 11, 2015 at 08:19:05 +0200: > Hi, the attached patch fixes a trivial typo in the Italian > translation when using "svn propedit". +1 to commit, thanks!

[PATCH] parse-diff.c condition for unrecognized lines

2015-09-11 Thread Daniel Shahaf
Is the following patch correct? Index: subversion/libsvn_diff/parse-diff.c === --- subversion/libsvn_diff/parse-diff.c (revision 1702341) +++ subversion/libsvn_diff/parse-diff.c (working copy) @@ -1438,7 +1438,9 @@ svn_diff_parse_next

Re: svn commit: r1700799 - in /subversion/trunk/subversion: include/svn_io.h libsvn_subr/stream.c svnadmin/svnadmin.c svnfsfs/load-index-cmd.c tests/libsvn_subr/stream-test.c

2015-09-11 Thread Julian Foad
Evgeny Kotkov wrote: > - Enable buffering in three subcommands: svnadmin load, svnadmin load-revprops > and in svnfsfs load-index. Also for: svnrdump load svnmucc put - ? (These just came to mind when I thought about it.) - Julian

Re: svn commit: r1700799 - in /subversion/trunk/subversion: include/svn_io.h libsvn_subr/stream.c svnadmin/svnadmin.c svnfsfs/load-index-cmd.c tests/libsvn_subr/stream-test.c

2015-09-11 Thread Evgeny Kotkov
Stefan Fuhrmann writes: > So, you are absolutely right about APR buffering being enough for our > non-interactive "read stdin as a single large file" usage in load & friends. > > My last concern is as follows. We need an alternative implementation > for svn_stream_for_stdin. I see 3 options: > >

Re: svn commit: r1702305 - /subversion/trunk/subversion/libsvn_subr/stream.c

2015-09-11 Thread Stefan Fuhrmann
On Fri, Sep 11, 2015 at 11:05 AM, Stefan Fuhrmann < stefan.fuhrm...@wandisco.com> wrote: > On Fri, Sep 11, 2015 at 11:01 AM, Branko Čibej wrote: > >> On 10.09.2015 23:50, Evgeny Kotkov wrote: >> > Stefan Fuhrmann writes: >> > >> > + * Special files like STDIN will have no flags set at all. In

Re: svn commit: r1702305 - /subversion/trunk/subversion/libsvn_subr/stream.c

2015-09-11 Thread Stefan Fuhrmann
On Fri, Sep 11, 2015 at 11:01 AM, Branko Čibej wrote: > On 10.09.2015 23:50, Evgeny Kotkov wrote: > > Stefan Fuhrmann writes: > > > > + * Special files like STDIN will have no flags set at all. In that > case, > > + * we can't filter and must allow any operation - which may then fail > at >

Re: svn commit: r1702305 - /subversion/trunk/subversion/libsvn_subr/stream.c

2015-09-11 Thread Branko Čibej
On 10.09.2015 23:50, Evgeny Kotkov wrote: > Stefan Fuhrmann writes: > > + * Special files like STDIN will have no flags set at all. In that case, > + * we can't filter and must allow any operation - which may then fail at > + * the APR level further down the stack. > + */ > + apr_uint32_t