Gentoo Archives: gentoo-portage-dev

From: Zac Medico <zmedico@g.o>
To: "Michał Górny" <mgorny@g.o>
Cc: gentoo-portage-dev@l.g.o, Michael Palimaka <kensington@g.o>
Subject: Re: [gentoo-portage-dev] [PATCH 1/3] bin/misc-functions.sh: Introduce eqalog and eqawarnlog functions.
Date: Sat, 25 Oct 2014 20:32:41
Message-Id: 544C08DF.10504@gentoo.org
In Reply to: Re: [gentoo-portage-dev] [PATCH 1/3] bin/misc-functions.sh: Introduce eqalog and eqawarnlog functions. by "Michał Górny"
1 On 10/25/2014 01:26 PM, Micha³ Górny wrote:
2 > Dnia 2014-10-25, o godz. 12:53:15
3 > Zac Medico <zmedico@g.o> napisa³(a):
4 >
5 >> On 10/25/2014 09:15 AM, Michael Palimaka wrote:
6 >>> +eqalog() {
7 >>> + local tag=$1 x
8 >>> + shift
9 >>> + for x in "$@" ; do
10 >>> + echo "${tag}" "${x}" >> "${T}"/qa.log
11 >>> + done
12 >>> +}
13 >>> +
14 >>> +eqawarnlog() {
15 >>> + eqalog "$@"
16 >>> + shift
17 >>> + for x in "$@" ; do
18 >>> + eqawarn " $x"
19 >>> + done
20 >>> +}
21 >>> +
22 >>
23 >> These functions are internals, so they need to be prefixed with __ like
24 >> __eqalog and __eqawarnlog.
25 >
26 > eqawarnlog shouldn't be internal since we support adding QA checks
27 > in repositories. In fact, I am planning to move some Gentoo-specific QA
28 > checks out of portage code.
29
30 It's a PMS thing. If it's not in PMS and the package manager provides
31 it, it's supposed to be prefixed.
32 --
33 Thanks,
34 Zac

Replies