1 |
On 08/07/2018 10:44 AM, Michael Orlitzky wrote: |
2 |
> On 08/07/2018 01:34 PM, Zac Medico wrote: |
3 |
>> |
4 |
>> Why not use ${ED%/} instead of ${D%/} here, so that the output is the |
5 |
>> same regardless of ${EPREFIX}? |
6 |
>> |
7 |
> |
8 |
> We want to show where the executable was actually installed, and |
9 |
> generally that includes EPREFIX. For example, I'd want to see |
10 |
> |
11 |
> /var/tmp/whatever/.../root/prefix/usr/bin/foo |
12 |
> |
13 |
> reported as |
14 |
> |
15 |
> /root/prefix/usr/bin/foo |
16 |
> |
17 |
> rather than |
18 |
> |
19 |
> /usr/bin/foo |
20 |
> |
21 |
> Of course, these checks are now skipped on prefix systems anyway, so |
22 |
> it's a bit of a moot point. But I think it's more future-proof to strip |
23 |
> only the $D. |
24 |
|
25 |
Sounds good. Thanks! Merged: |
26 |
|
27 |
https://gitweb.gentoo.org/proj/portage.git/commit/?id=807ac3d9d6eecead73f59d399b30559e5c731587 |
28 |
-- |
29 |
Thanks, |
30 |
Zac |