Hi, On Fri, Mar 26, 2010 at 17:18, Johannes Schindelin <Johannes.Schindelin@xxxxxx> wrote: >> I have also the feeling that -O potion does not like pager with >> arguments, be it from GIT_PAGER, PAGER, or from the command line. > > You are correct. That's why I said "-O[<pager>]" and not > "-O[<pager-with-arguments-that-cause-whitespace-problems-especially-with-spaces-in-directory-names-so-you-need-to-make-a-script-wrapper-anyway>]" I don't know how common it is to have arguments for the pager in GIT_PAGER or PAGER or whether git actually support this, but your first patch definitely does not work with this, regardless of your -O[<pager>] addition of the second patch. Bert -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html