Bug 391001 – >=sys-apps/portage-2.1.10.11 only changes /var/log/portage ownership when upgrading from <sys-apps/portage-2.1.10.11

• Product: Gentoo Linux
• Status: RESOLVED WONTFIX

[Description Jouni Rinne 2011-11-19 11:41:22 UTC]

Created attachment 293081 [details]
emerge --info

Although in bug #377177 it is stated that portage-2.1.10.11 and upwards should change the ownership of /var/log/portage to portage:portage, in practice it does not do so. On both my fileserver (x86) and desktop (~amd64) the ownership of /var/log/portage is still set as root:root, and both use portage-2.1.10.35

If I remove the 'has_version'-line, i.e.
 
 	if [[ -d ${ROOT}var/log/portage && \
-		$(ls -ld "${ROOT}var/log/portage") != *" portage portage "* ]] && \
-		has_version '<sys-apps/portage-2.1.10.11' ; then
+		$(ls -ld "${ROOT}var/log/portage") != *" portage portage "* ]] ; then

THEN the ownership is set properly on (re)installing.

[Comment 1 Jouni Rinne 2011-11-19 11:42:23 UTC]

Created attachment 293083 [details, diff]
portage-2.1.10.35.patch

[Comment 2 Zac Medico 2011-11-19 15:00:35 UTC]

The intention of the existing code is to correct the default permissions one time, when upgrading from <sys-apps/portage-2.1.10.11, and to leave them untouched thereafter. If we remove the has_version call, then it will override the user's permissions during ever upgrade, which is not necessarily desirable.

[Comment 3 Zac Medico 2011-11-19 15:02:07 UTC]

Closing a WONTFIX, since the current behavior is intended, as explained in comment #2.

[Comment 4 Jouni Rinne 2011-11-19 15:04:50 UTC]

Well, can you then explain why it didn't work when I upgraded from pre-2.1.10.11 to >=2.1.10.11???

[Comment 5 Zac Medico 2011-11-19 15:10:03 UTC]

It may be that you upgraded to portage-2.1.10.11 before I added the permission changes to the ebuild, which could have happened if you installed ~arch portage between August 12 and 27:

http://sources.gentoo.org/cgi-bin/viewvc.cgi/gentoo-x86/sys-apps/portage/portage-2.1.10.11.ebuild?view=log#rev1.4

[Comment 6 Zac Medico 2011-11-19 15:12:48 UTC]

Actually, the first stabilization was on August 26, so some stable users may have updated before I added the permission change to the ebuild.