1 |
On 10/01/2018 01:04 PM, Sergei Trofimovich wrote: |
2 |
> On Mon, 1 Oct 2018 08:19:29 -0700 |
3 |
> Zac Medico <zmedico@g.o> wrote: |
4 |
> |
5 |
>> Hi all, |
6 |
>> |
7 |
>> The ~arch version of portage hs a new QA check that reports installation |
8 |
>> of files outside of directories that have been whitelisted [1]. The |
9 |
>> current whitelist includes: |
10 |
>> |
11 |
>> directories common to / and /usr |
12 |
>> ================================ |
13 |
>> bin lib lib32 lib64 libx32 sbin |
14 |
>> |
15 |
>> top level directories |
16 |
>> ================================ |
17 |
>> boot dev etc opt srv usr var |
18 |
>> |
19 |
>> /usr level directories |
20 |
>> ================================ |
21 |
>> include libexec share src |
22 |
>> |
23 |
>> /usr/share/doc level directories |
24 |
>> ================================ |
25 |
>> /usr/share/doc/${PF} |
26 |
> |
27 |
> As this will break existing ebuilds I'd suggest guarding it against |
28 |
> next EAPI. Out of top of by head the change will break at least |
29 |
> crossdev outright. At least for two reasons: |
30 |
> 1. building a gcc cross-compiler usually refers to $SYSROOT/sys-include. |
31 |
> 'git grep sys-include' will show a bunch of ebuilds that create that |
32 |
> symlinks outside the space. |
33 |
> 2. cross-building target libc is currently done on the host's emerge |
34 |
> and installs into /usr/$CTARGET/. |
35 |
> |
36 |
> I think whitelist would be able to cover these use cases as they are |
37 |
> a function of ${CTARGET} (or $CATEGORY) value. |
38 |
|
39 |
The QA check currently does not die, it only calls eqawarn. So the worst |
40 |
case is a message like this: |
41 |
|
42 |
> QA: other |
43 |
> The ebuild is installing to one or more unexpected paths: |
44 |
> |
45 |
> /usr/share/doc/qt-5.11.1 |
46 |
> |
47 |
> Please fix the ebuild to use correct FHS/Gentoo policy paths. |
48 |
|
49 |
It will be annoying if this message triggers for every package installed |
50 |
by crossdev, so we should do something about that. |
51 |
-- |
52 |
Thanks, |
53 |
Zac |