commit: 58eb01022e604b0a538b9d67bd64daa44da9f94a Author: Zac Medico <zmedico <AT> gentoo <DOT> org> AuthorDate: Mon Nov 19 06:38:21 2018 +0000 Commit: Zac Medico <zmedico <AT> gentoo <DOT> org> CommitDate: Mon Nov 19 06:42:06 2018 +0000 URL: https://gitweb.gentoo.org/proj/portage.git/commit/?id=58eb0102
Revert "fetch: check DISTDIR write access as fetch user (bug 601252)" This reverts commit 4264ac4b07c4555eb5db2ab21288dba05a7f4b6d. Dropping privileges for the access call is not useful by itself, since lock and unlink calls do not drop privileges yet. We might decide to drop privileges for the entire fetch function call. Bug: https://bugs.gentoo.org/601252 Signed-off-by: Zac Medico <zmedico <AT> gentoo.org> lib/portage/package/ebuild/fetch.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/portage/package/ebuild/fetch.py b/lib/portage/package/ebuild/fetch.py index 7ec7fe05e..0431e11ea 100644 --- a/lib/portage/package/ebuild/fetch.py +++ b/lib/portage/package/ebuild/fetch.py @@ -543,7 +543,7 @@ def fetch(myuris, mysettings, listonly=0, fetchonly=0, if can_fetch and \ not fetch_to_ro and \ - _spawn_fetch(mysettings, ['test', '-w', mysettings["DISTDIR"]]) != os.EX_OK: + not os.access(mysettings["DISTDIR"], os.W_OK): writemsg(_("!!! No write access to '%s'\n") % mysettings["DISTDIR"], noiselevel=-1) can_fetch = False
