Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 501886 - sys-apps/portage: Better default for less-opts= in dispatch-conf.conf
Summary: sys-apps/portage: Better default for less-opts= in dispatch-conf.conf
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Core system (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Portage team
URL:
Whiteboard:
Keywords: InVCS
Depends on:
Blocks:
 
Reported: 2014-02-20 12:21 UTC by Marc Schiffbauer
Modified: 2015-09-18 22:34 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Marc Schiffbauer gentoo-dev 2014-02-20 12:21:06 UTC
To increase usability of dispatch-conf the default values for less-opts= should be slightly changed:

from:
less-opts="--no-init --QUIT-AT-EOF"

to:
less-opts="--no-init --quit-if-one-screen"

This still lets less quit if the diff fits onto the screen, but will PREVENT less from quitting only because you reach EOF.

I saw new dispatch-conf users finding the current default behavior strange because you cannot predict when you will reach EOF and it will quit when you do not want it to do. When using less you expcect being able to scroll up all the time while inspecting the current diff.

Thoughts?
Comment 1 Marc Schiffbauer gentoo-dev 2015-03-10 01:21:19 UTC
Ping? PLease as wontfix if you think this is a bad idea... ;)
Comment 2 Zac Medico gentoo-dev 2015-03-10 16:27:49 UTC
There's a patch in the following branch:

    https://github.com/zmedico/portage/tree/bug_501886

I've posted it for review here:

    http://thread.gmane.org/gmane.linux.gentoo.portage.devel/5328
Comment 3 Zac Medico gentoo-dev 2015-03-10 17:53:49 UTC
This is in the master branch now:

https://github.com/gentoo/portage/commit/ff3c131aeee6f053a6f9a706c1f1be279696c957
Comment 4 Marc Schiffbauer gentoo-dev 2015-09-18 22:34:58 UTC
In released portage now. Thanks!