On Mon, 13 Jul 2009, Catalin Marinas wrote:
>
> os.environ.setdefault('PAGER', config.get('stgit.pager'))
> + if os.environ.get('PAGER').startswith('less'):
> + os.environ.setdefault('LESS', '-FRSX')
> # FIXME: handle EDITOR the same way ?
That's not very smart.
First off, PAGER could easily be something like "/usr/local/bin/less".
Secondly, if it isn't less, so what? If it's not less, it won't care about
the LESS environment variable.
So just do an unconditional
os.environ.setdefault('LESS', '-FRSX')
and it will (a) work better, (b) be simpler and more efficient.
Linus
--
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