New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Search: no longer output to $PAGER #2656
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I understand that several people are against my change and I'm perfectly fine with reverting this if there's a majority against this.
But please note that one of the reasons why I added this is that $PAGER
-support was added to several commands (like nix-env
or nix-store
) already, so if you'd like to remove this, please drop all usages for consistency :)
Users who don't want a pager can |
I wasn't aware of the |
looks like NIX_PAGER is only in the release notes and should be documented in the manual:
|
IMHO, if |
The pager behaviour is consistent with other tools such as |
i believe it is inconsistent with other package search tools such as |
Inconsistent in what way? |
Other package managers don't use a pager to display their search results. |
I marked this as stale due to inactivity. → More info |
Output search results to stdout, let the user see the results for install command.
As requested in #1261