Hi,

On Mon, Sep 19, 2016 at 06:13:30AM +0900, Hideki Yamane wrote:
> control: tags -1 +patch
> 
> Hi,
> 
> On Sun, 18 Sep 2016 15:07:42 +0900
> Osamu Aoki <osamua...@e01.itscom.net> wrote:
> 
> > I think including lv as a part of Japanese task is OK but its
> > update-alternative preference should be demoted so it will not be the
> > default choice for the pager.
> 
>   Selection    Path            Priority   Status
> ------------------------------------------------------------
>   0            /usr/bin/lv      80        auto mode
> * 1            /bin/less        77        manual mode
>   2            /bin/more        50        manual mode
>   3            /usr/bin/lv      80        manual mode
>   4            /usr/bin/pg      10        manual mode
>   5            /usr/bin/w3m     25        manual mode

FYI (here):
$ update-alternatives --config pager
There are 5 choices for the alternative pager (providing /usr/bin/pager).

  Selection    Path            Priority   Status
------------------------------------------------------------
* 0            /bin/less        77        auto mode
  1            /bin/less        77        manual mode
  2            /bin/more        50        manual mode
  3            /usr/bin/most    60        manual mode
  4            /usr/bin/pg      10        manual mode
  5            /usr/bin/w3m     25        manual mode

>  Now we have lv as priority=80 and it'd be better to downgrade to between
>  50 and 77, such as 70. Using "less" for pager is best, and lv is better 
>  than "more", IMO. patch attached.

I agree "lv" is better than "more" of priority 50 and not better than
"less" of priority 77

But "most" of priority 60 is another good UTF-8 capable pager with
backward scroll capability etc.

So priority of "lv" should be between 51-59.  Let's pick 55 and:

... 
> --- a/debian/postinst
> +++ b/debian/postinst
> @@ -25,7 +25,7 @@ set -e
>  case "$1" in
>      configure)
>       update-alternatives --quiet --install /usr/bin/pager pager \
-               /usr/bin/lv 80 --slave /usr/share/man/man1/pager.1.gz \
+               /usr/bin/lv 55 --slave /usr/share/man/man1/pager.1.gz \
>               pager.1.gz /usr/share/man/man1/lv.1.gz

Reply via email to