| Summary: | >=sys-apps/portage-2.2_rc2 does not support interactive ebuilds if in background mode (--jobs or --quiet) | ||
|---|---|---|---|
| Product: | Portage Development | Reporter: | Jeremy Olexa (darkside) (RETIRED) <darkside> |
| Component: | Core | Assignee: | Portage team <dev-portage> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | aagaande, angelos, ikelos, m.debruijne, mailingdotlist, pacho, tskuzzy |
| Priority: | High | Keywords: | InVCS, REGRESSION |
| Version: | 2.2 | ||
| Hardware: | All | ||
| OS: | Linux | ||
| URL: | http://archives.gentoo.org/gentoo-dev/msg_1266d26d46320ed4d373a8d540577cad.xml | ||
| Whiteboard: | |||
| Package list: | Runtime testing required: | --- | |
| Bug Depends on: | 151113 | ||
| Bug Blocks: | 210077, 234300, 246251 | ||
| Attachments: | force all output to stdio if the merge list contains at least one interactive package | ||
|
Description
Jeremy Olexa (darkside) (RETIRED)
2008-07-29 17:00:27 UTC
*** Bug 233977 has been marked as a duplicate of this bug. *** The license acceptance can be handled via ACCEPT_LICENCE and license groups as described in bug #152593. However, support for interactive ebuilds may still be useful for other things such as cdrom_get_cds() in eutils.eclass. (In reply to comment #2) > The license acceptance can be handled via ACCEPT_LICENCE and license groups as > described in bug #152593. However, support for interactive ebuilds may still be > useful for other things such as cdrom_get_cds() in eutils.eclass. I consider this a pretty good flaw in portage-2.2 only because it appears to the end-user that the job has hung. Can portage at least bail out upon detection and say "Hey, you don't accept this license" with an informative message about how to accept the license? I don't think it is a requirement to move the message to the forground and ask for user-input yet. That can come later, if it is possible. Thoughts? For future reference please see this thread: http://thread.gmane.org/gmane.linux.gentoo.devel/57625 *** Bug 234738 has been marked as a duplicate of this bug. *** *** Bug 237589 has been marked as a duplicate of this bug. *** Yesterday the council approved PROPERTIES=interactive for use in ebuilds. I've deployed support on the master rsync mirror for PROPERTIES in the metadata cache, so we can go ahead and start setting it in ebuilds. Created attachment 169362 [details, diff]
force all output to stdio if the merge list contains at least one interactive package
If this patch is saved as /tmp/interactive.patch, then it can be applied as follows:
patch /usr/lib/portage/pym/_emerge/__init__.py /tmp/interactive.patch
This patch forces the output to stdio for all packages in the merge list if at least one of the packages is interactive. I intend to improve support in the future, so that stdio is only used for interactive packages.
This is fixed in 2.2_rc13. Just to let you know, I'm now using PROPERTIES="Interactive" in the vmware-workstation-6.5.0 ebuild... Maybe would be better try to merge interactive package at first (when possible) and, once it's merged and user has interactued, emerge could merge the remaining packages with desired jobs setting. For example, I am seeing this problem while running "emerge -avuDN --jobs=10 world", emerge is using --jobs=1 because there is one interactive package Thanks (Should I open a new bug report for this?) (In reply to comment #11) > For example, I am seeing this problem while running "emerge -avuDN --jobs=10 > world", emerge is using --jobs=1 because there is one interactive package > > Thanks > > (Should I open a new bug report for this?) Yes please file an "enhancement" bug for that. http://bugs.gentoo.org/show_bug.cgi?id=265267 Done, thanks :-) |