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 |