Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 612246 - sys-apps/portage: calls 'portageq envvar' which can break paludis
Summary: sys-apps/portage: calls 'portageq envvar' which can break paludis
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Portage team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-03-10 19:23 UTC by Michał Górny
Modified: 2017-03-11 04:42 UTC (History)
0 users

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2017-03-10 19:23:53 UTC
type -P portageq >/dev/null 2>&1 && \
    REPOS_CONF_SYNC=$("$(type -P portageq)" envvar SYNC)

Paludis has a portageq wrapper which dies (i.e. kills ebuild) when 'portageq envvar' is used. This means that in some cases Paludis users won't be able to upgrade their Portage install.
Comment 1 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2017-03-10 19:28:33 UTC
Oh, Arfrever just pointed out that you're working around the shell function wrapper via 'type -P'. Which is both ugly and just asking Paludis devs to make it a normal helper, so it's not worked around like this...
Comment 2 Zac Medico gentoo-dev 2017-03-10 19:34:08 UTC
Realistically, we can drop this code because it has already served its purpose for the vast majority of people upgrading from the relevant portage versions:

	if has_version "<${CATEGORY}/${PN}-2.1.13" || \
		{
			has_version ">=${CATEGORY}/${PN}-2.2_rc0" && \
			has_version "<${CATEGORY}/${PN}-2.2.0_alpha189"
		} ; then

That means we can remove all code involving to the USERPRIV_UPGRADE, USERSYNC_UPGRADE, and REPOS_CONF_UPGRADE variables.
Comment 3 Brian Dolbec (RETIRED) gentoo-dev 2017-03-10 20:38:27 UTC
I'd be fine with cleaning that old code out :)
Comment 4 Zac Medico gentoo-dev 2017-03-11 04:42:46 UTC
I've removed the code from the last 2 releases (2.3.3 and 2.3.4):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=c7a97e5230e117116baad064f70cdeecab480620