Gentoo Archives: gentoo-portage-dev

From: "Michał Górny" <mgorny@g.o>
To: gentoo-portage-dev@l.g.o
Subject: Re: [gentoo-portage-dev] [PATCH] install-qa-check.d: Add a QA check for installing xattrs
Date: Tue, 28 Sep 2021 06:25:57
Message-Id: eeb0ec6807e63cdf09561ad9906c0a9f4eeefaa2.camel@gentoo.org
In Reply to: Re: [gentoo-portage-dev] [PATCH] install-qa-check.d: Add a QA check for installing xattrs by Mike Gilbert
1 On Mon, 2021-09-27 at 21:09 -0400, Mike Gilbert wrote:
2 > On Mon, Sep 27, 2021 at 1:20 PM Michał Górny <mgorny@g.o> wrote:
3 > > + eqawarn
4 > > + eqawarn "It is impossible to reliably guarantee that the extended attributes"
5 > > + eqawarn "will be reliably preserved while merging. Please ensure that any"
6 > > + eqawarn "extended metadata necessary is applied in pkg_postinst() phase,"
7 > > + eqawarn "and that the implementation includes a fallback if necessary."
8 >
9 > This message suggests that applying xattrs in pkg_postinst is
10 > acceptable. However, your patch offers no way to disable the QA
11 > warning for ebuilds that do so.
12
13 We'll cross that bridge when we get there. Ideally, we wouldn't need to
14 silence the check because no packages would do that. If they do, then
15 we'll probably want to work on an eclass like fcaps.eclas.
16
17 --
18 Best regards,
19 Michał Górny

Replies