[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] pager support for command line client

From: Ben Reser <ben_at_reser.org>
Date: Mon, 03 Feb 2014 22:12:41 -0800

On 2/3/14, 4:31 PM, Daniel Shahaf wrote:
> Do you need to call __close if there is an error after __start? i.e.,
> is there a state in which stdout will be printed to in the error path?

Yes you do. Just try running svn diff against a non-working copy. You'll end
up with a broken terminal that you have to reset to see what you're doing.

> What about the case SVN_PAGER="/nonexistent"? Should it fall back to
> just printing to stdout, or bail out on account of failing to execute
> "/nonexistent"?

Print to stdout with a warning why the pager isn't working.
Received on 2014-02-04 07:13:25 CET

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.