| 1 |
On Wed, Jun 20, 2018 at 06:01:10PM -0500, Marty E. Plummer wrote: |
| 2 |
> On Wed, Jun 20, 2018 at 11:33:53PM +0100, James Le Cuirot wrote: |
| 3 |
> > On Wed, 20 Jun 2018 17:21:09 -0500 |
| 4 |
> > "Marty E. Plummer" <hanetzer@×××××××××.com> wrote: |
| 5 |
> > |
| 6 |
> > > On Wed, Jun 20, 2018 at 09:03:44PM +0800, Jason Zaman wrote: |
| 7 |
> > > > On Wed, Jun 20, 2018 at 02:10:50AM -0500, Marty E. Plummer wrote: |
| 8 |
> > > > > Use ${EROOT%/} whereever possible, as the tools and directories which |
| 9 |
> > > > > are used with it are already prefixed with a / |
| 10 |
> > > > > |
| 11 |
> > > > > Package-Manager: Portage-2.3.40, Repoman-2.3.9 |
| 12 |
> > > > > --- |
| 13 |
> > > > > eclass/xdg-utils.eclass | 10 +++++----- |
| 14 |
> > > > > 1 file changed, 5 insertions(+), 5 deletions(-) |
| 15 |
> > > > > |
| 16 |
> > > > > diff --git a/eclass/xdg-utils.eclass b/eclass/xdg-utils.eclass |
| 17 |
> > > > > index ac075185d8e..8dba5ed6861 100644 |
| 18 |
> > > > > --- a/eclass/xdg-utils.eclass |
| 19 |
> > > > > +++ b/eclass/xdg-utils.eclass |
| 20 |
> > > > > @@ -66,7 +66,7 @@ xdg_environment_reset() { |
| 21 |
> > > > > # Updates the .desktop files database. |
| 22 |
> > > > > # Generates a list of mimetypes linked to applications that can handle them |
| 23 |
> > > > > xdg_desktop_database_update() { |
| 24 |
> > > > > - local updater="${EROOT}${DESKTOP_DATABASE_UPDATE_BIN}" |
| 25 |
> > > > > + local updater="${EROOT%/}${DESKTOP_DATABASE_UPDATE_BIN}" |
| 26 |
> > > > |
| 27 |
> > > > Shouldn't things like this be $BROOT since they're being run? $EROOT |
| 28 |
> > > > might be a different architecture that may or may not run at all on the |
| 29 |
> > > > build machine. |
| 30 |
> > > > |
| 31 |
> > > Good point, but here's a question; if EROOT=${ROOT%/}${EPREFIX}, how do |
| 32 |
> > > we use BROOT here? EBROOT? Or longhand ${BROOT%/}${EPREFIX} ? I think |
| 33 |
> > > that may be a use case that got missed in the EAPI 7 discussions. |
| 34 |
> > |
| 35 |
> > BROOT is already prefixed as BROOT without a prefix would just be /. |
| 36 |
> > |
| 37 |
> I don't follow. Its my understanding that BROOT ~= ROOT for most |
| 38 |
> situations. But consider this setup: |
| 39 |
> Ubuntu amd64 with Gentoo Prefix, emerging a native arm @system to |
| 40 |
> /mnt/arm EPREFIX = /home/user/gentoo. |
| 41 |
> |
| 42 |
> In this situation, ROOT=/mnt/arm, EROOT=/mnt/arm, but what is BROOT? /, |
| 43 |
> or /home/usr/gentoo? |
| 44 |
|
| 45 |
https://dev.gentoo.org/~mgorny/articles/the-ultimate-guide-to-eapi-7.html#broot-variable-for-bdepend |
| 46 |
|
| 47 |
Basically BROOT already contains EPREFIX or BPREFIX or whatever it would |
| 48 |
be called. There is like no need for an un-prefixed BROOT so its just |
| 49 |
merged in. so you should just need "${BROOT}/usr/bin/update-mime-database" |
| 50 |
|
| 51 |
-- Jason |
| 52 |
|
| 53 |
> > -- |
| 54 |
> > James Le Cuirot (chewi) |
| 55 |
> > Gentoo Linux Developer |
| 56 |
> |
| 57 |
> |
| 58 |
> |
Archives