Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 444620 - app-admin/eselect - profile.eselect:broken in prefix
Summary: app-admin/eselect - profile.eselect:broken in prefix
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Hosted Projects
Classification: Unclassified
Component: eselect (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo eselect Team
URL:
Whiteboard:
Keywords: InVCS, PATCH
Depends on:
Blocks:
 
Reported: 2012-11-24 21:14 UTC by Greg Turner
Modified: 2013-01-06 19:12 UTC (History)
1 user (show)

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


Attachments
s/ROOT/EROOT/ (fix_eselect_profile_eroot.patch,799 bytes, patch)
2012-11-24 21:16 UTC, Greg Turner
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Greg Turner 2012-11-24 21:14:31 UTC
eselect profile fails in prefix due to usage of ROOT in portageq instead of EROOT.

Reproducible: Always

Steps to Reproduce:
eselect profile list
Comment 1 Greg Turner 2012-11-24 21:16:41 UTC
Created attachment 330498 [details, diff]
s/ROOT/EROOT/

replaces old-style portageq ${ROOT} with portageq ${EROOT}
Comment 2 Greg Turner 2012-11-25 01:04:03 UTC
Note: this leaves the more interesting problem that none of the profiles offered are legitimate prefix profiles and will break your prefix... but that's really a separate bug.
Comment 3 Ulrich Müller gentoo-dev 2012-11-26 14:38:58 UTC
(In reply to comment #1)
> Created attachment 330498 [details, diff] [details, diff]
> s/ROOT/EROOT/
> 
> replaces old-style portageq ${ROOT} with portageq ${EROOT}

profile.eselect shouldn't call portageq in the first place, but use the wrapper functions in libs/package-manager.bash instead (which use EROOT throughout).
Comment 4 Fabian Groffen gentoo-dev 2012-12-05 11:41:30 UTC
so, who's bug is this?
Comment 5 Ulrich Müller gentoo-dev 2012-12-05 14:00:06 UTC
(In reply to comment #4)
> so, who's bug is this?

eselect's.
Comment 6 Ulrich Müller gentoo-dev 2012-12-06 21:05:12 UTC
While get_repositories() exists in libs/package-manager.bash, there's no equivalent of 'portageq get_repo_path "${ROOT:-/}" ${repos}' yet.

Authors of package managers have told be this:
- For paludis, the command would be:
  cave print-repository-metadata --raw-name location --format '%v\n' ${repo}
  However, "it's a bit of a fiddly issue, though, since we don't guarantee
  that location will point to anything you can read" (ciaranm).
- For pkgcore, there's no such command, but it's "trivial to add" (ferringb).


So for the time being, we stay with the Portage-only solution. I've committed the patch to git: <http://git.overlays.gentoo.org/gitweb/?p=proj/eselect.git;a=commit;h=ee66576015a6dc576e76ac1539412ec32ba466d9>
Comment 7 Ulrich Müller gentoo-dev 2013-01-06 19:12:27 UTC
Fixed in eselect-1.3.4.
Thanks for reporting.