1 |
On Wed, 20 Jun 2018 21:03:44 +0800 |
2 |
Jason Zaman <perfinion@g.o> wrote: |
3 |
|
4 |
> On Wed, Jun 20, 2018 at 02:10:50AM -0500, Marty E. Plummer wrote: |
5 |
> > Use ${EROOT%/} whereever possible, as the tools and directories which |
6 |
> > are used with it are already prefixed with a / |
7 |
> > |
8 |
> > Package-Manager: Portage-2.3.40, Repoman-2.3.9 |
9 |
> > --- |
10 |
> > eclass/xdg-utils.eclass | 10 +++++----- |
11 |
> > 1 file changed, 5 insertions(+), 5 deletions(-) |
12 |
> > |
13 |
> > diff --git a/eclass/xdg-utils.eclass b/eclass/xdg-utils.eclass |
14 |
> > index ac075185d8e..8dba5ed6861 100644 |
15 |
> > --- a/eclass/xdg-utils.eclass |
16 |
> > +++ b/eclass/xdg-utils.eclass |
17 |
> > @@ -66,7 +66,7 @@ xdg_environment_reset() { |
18 |
> > # Updates the .desktop files database. |
19 |
> > # Generates a list of mimetypes linked to applications that can handle them |
20 |
> > xdg_desktop_database_update() { |
21 |
> > - local updater="${EROOT}${DESKTOP_DATABASE_UPDATE_BIN}" |
22 |
> > + local updater="${EROOT%/}${DESKTOP_DATABASE_UPDATE_BIN}" |
23 |
> |
24 |
> Shouldn't things like this be $BROOT since they're being run? $EROOT |
25 |
> might be a different architecture that may or may not run at all on the |
26 |
> build machine. |
27 |
|
28 |
+1 |
29 |
|
30 |
-- |
31 |
James Le Cuirot (chewi) |
32 |
Gentoo Linux Developer |