Summary: | emerge: --ask could ask to read pending news items | ||
---|---|---|---|
Product: | Portage Development | Reporter: | Fabian Groffen <grobian> |
Component: | Core - Interface (emerge) | Assignee: | Portage team <dev-portage> |
Status: | RESOLVED FIXED | ||
Severity: | enhancement | CC: | pacho |
Priority: | Normal | Keywords: | InVCS |
Version: | unspecified | ||
Hardware: | All | ||
OS: | All | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | |||
Bug Blocks: | 484436 | ||
Attachments: |
Fixes the bug
emerge --read-news: prompt only if --ask |
Description
Fabian Groffen
2014-07-17 09:26:45 UTC
There's this notion that emerge should not be interactive apart from the merge question. autounmasking is an exception. Personally I don't see a problem with your suggestion. It makes sense. You would not want to waste time calculating dependencies twice. If I see no objections within a week or two, I'll go ahead and implement this. Thanks for the idea. Well, I'm not really sure about changing interactivity here. You can always do ^z or use another terminal to read the news. But it's just a matter of habit. Created attachment 382716 [details, diff]
Fixes the bug
This is released in portage-2.2.14_rc1 Reopening. I'd like to request having this feature optional. Either through FEATURES or --option. Otherwise it totally disturbs my workflow with portage which I have for years. Thanks. It's its own option in git now. (In reply to Alexander Berntsen from comment #7) > It's its own option in git now. Looking at the changes: https://github.com/gentoo/portage/commit/7d605312d48ae5f7755f640ef78c97b424399bd0 That may be the only UserQuery that is not conditional on --ask (it's conditional on --read-news instead). Maybe it should be conditional on both? Created attachment 387084 [details, diff]
emerge --read-news: prompt only if --ask
This patch makes --read-news have no effect unless --ask is enabled.
I don't care particularly much. Patch looks good to me. Go ahead and push it. (In reply to Alexander Berntsen from comment #10) > I don't care particularly much. Patch looks good to me. Go ahead and push it. Thanks, it's in git now: https://github.com/gentoo/portage/commit/3a22c197d00b81efeb78fe75359b49316844c2e2 This is in the portage-2.2.15 release. |