Summary: | emerge-delta-webrsync stalls when run as non-root user | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Rudi Lippert <gentoo> |
Component: | Current packages | Assignee: | Portage team <dev-portage> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | write2David, zmedico |
Priority: | High | Keywords: | InVCS |
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | 454046 | ||
Bug Blocks: | |||
Attachments: |
possible patch
Handle FEATURES=usersync Check write permission for PORTDIR and DISTDIR |
Description
Rudi Lippert
2007-04-25 18:56:12 UTC
Created attachment 117261 [details, diff]
possible patch
The user check could probably be performed at a more suitable position in the code, but I don't have enough knowledge of it.
Created attachment 321266 [details, diff] Handle FEATURES=usersync (In reply to comment #1) > Created attachment 117261 [details, diff] [details, diff] > possible patch > > The user check could probably be performed at a more suitable position in > the code, but I don't have enough knowledge of it. I don't want to disable running it as non-root, since it's valid to run as non-root with FEAUTURE=usersync if we apply the attached patch. The reason it stalled for you is that rm went into interactive mode while its prompt was redirected to /dev/null. We can fix that by using rm -f (I'll attach a patch for that). Created attachment 321268 [details, diff]
Check write permission for PORTDIR and DISTDIR
These permission checks should make it bail out earlier, so you won't trigger the interactive rm call. I'll add in the rm -f option later.
(In reply to comment #2) > Created attachment 321266 [details, diff] [details, diff] > Handle FEATURES=usersync In git: http://git.overlays.gentoo.org/gitweb/?p=proj/portage.git;a=commit;h=87a769bdab8e54c68260996026c87d3d7845d2e6 (In reply to comment #3) > Created attachment 321268 [details, diff] [details, diff] > Check write permission for PORTDIR and DISTDIR Also in git: http://git.overlays.gentoo.org/gitweb/?p=proj/portage.git;a=commit;h=9807b6646b8f9ce6d67b03cddfe4b9d0b02288bf Use rm -f for non-interactive: http://git.overlays.gentoo.org/gitweb/?p=proj/portage.git;a=commit;h=a695533917d969d66210ce8e2fe8aa88bee42dc7 This is fixed in version 3.6. |