On 2/3/14, 12:02 PM, Stefan Sperling wrote: > Below is a very simple proof of concept patch that enables a > pager for 'help', 'diff', and 'blame'. Currently it only looks > for a PAGER or SVN_PAGER environment variable. My plan is to > also add a 'pager-cmd' option for this purpose.
I suspect we should allow the pager to be used with all our commands, though most of them should be disabled by default. I'd say the following should be enabled by default: blame cat diff list log mergeinfo (at least with --show-revs) proplist status A lot of these will make a lot more sense if we can reasonably default to something like the FRSX option set that git passes to less by default. They do this by setting the LESS environment variable (if not already set and the pager has no arguments). In our case I'd suggest FX. I guess R doesn't do much for us at this point since we don't emit color codes and S seems like a bad idea for a default. Branko convinced me that we should default to less (or more on Windows) if PAGER and SVN_PAGER is missing. So I'd like to see the following configuration: [helpers] enable-pager=yes|no (defaults to yes) pager-cmd=command [args] (defaults to SVN_PAGER, or PAGER or configure time choice) [blame] enable-pager=default|yes|no (default uses whatever yes/no state is on the helpers section) pager-cmd=command [args] (defaults to whatever the configuration of the helpers section was) and so on for additional sub-commands. If there are objections to putting this in ~/.subversion/config we could put it in ~/.subversion/cmdline to make it clear these are really for our command line client and not everything else. Part of the reason for each sub command being configurable is so you can do things like pagers that colorize output and what not (e.g. things like grc).
