Junio C Hamano <[email protected]> wrote:
> Eric Wong <[email protected]> writes:
> > Allowing PAGER_ENV to be set at build-time allows us to move
> > pager-specific knowledge out of our build. Currently, this
> > allows us to set a better default for FreeBSD where more(1)
> > is the same binary as less(1).
>
> Thanks for resurrecting, but I am not sure what "a better default"
> is from the above description and with the patch. Even though a
> naive reading of the above (i.e. "less" and "more" are the same)
> makes me expect that the patch will give the same set of default
> environment settings to those on FreeBSD, you give LESS=FRX and
> MORE=-R, i.e. they are configured differently.
Perhaps s/better/platform-appropriate/ ?
I just copied your original patch in setting MORE=-R
(but removed 'S' from LESS).
So v3 will be MORE=FRX, as less was added:
commit 98170c0c3ba86eb1cc975e7848d075bf2abc1ed0
Author: ps <[email protected]>
Date: Mon May 22 10:00:00 2000 +0000
bmake glue for less.
and more was nuked:
commit cde9059fa3e4dc7e259c3864d7536252a5c580a0
Author: ps <[email protected]>
Date: Mon May 29 13:31:51 2000 +0000
Nuke more from the repository.
And "git branch -r --contains" on both of those commits says
they showed up in the 5.0 release. However, further
investigation says more was even gone by the 4.1.0 release
git show origin/release/4.1.0:usr.bin/more # non-existent tree
git show origin/release/4.0.0:usr.bin/more # tree still exists
But, "git show origin/release/4.0.0:usr.bin/more/option.c"
reveals more from those days wouldn't handle -R anyways,
and hopefully nobody is still running 4.0.0...
ref: git://github.com/freebsd/freebsd.git
> > This also prepares us for introducing a run-time config knob to
> > override the build-time environment in the next commit.
>
> This is now gone, judging from 1/1 on the subject line being not
> 1/2, right?
Oops, yes :x
> > Originally-from:
> > https://public-inbox.org/git/[email protected]/
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
