Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
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
Summary: >=sys-apps/portage-2.1.10.11 only changes /var/log/portage ownership when upg...
Status: RESOLVED WONTFIX
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Core system (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Portage team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-11-19 11:41 UTC by Jouni Rinne
Modified: 2011-11-19 15:12 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
emerge --info (emerge_info,5.61 KB, text/plain)
2011-11-19 11:41 UTC, Jouni Rinne
Details
portage-2.1.10.35.patch (portage.diff,623 bytes, patch)
2011-11-19 11:42 UTC, Jouni Rinne
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
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 gentoo-dev 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 gentoo-dev 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 gentoo-dev 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 gentoo-dev 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.