1 |
On Thu, 25 Sep 2014 11:30:53 -0700 |
2 |
Brian Dolbec <dolsen@g.o> wrote: |
3 |
|
4 |
> On Thu, 25 Sep 2014 11:12:51 -0700 |
5 |
> Zac Medico <zmedico@g.o> wrote: |
6 |
> |
7 |
> > On 09/24/2014 10:48 PM, Michał Górny wrote: |
8 |
> > > Spotted-by: Zac Medico <zmedico@g.o> |
9 |
> > > --- |
10 |
> > > bin/phase-functions.sh | 2 ++ |
11 |
> > > 1 file changed, 2 insertions(+) |
12 |
> > > |
13 |
> > > diff --git a/bin/phase-functions.sh b/bin/phase-functions.sh |
14 |
> > > index 9117719..d292ad3 100644 |
15 |
> > > --- a/bin/phase-functions.sh |
16 |
> > > +++ b/bin/phase-functions.sh |
17 |
> > > @@ -807,6 +807,8 @@ __ebuild_phase_funcs() { |
18 |
> > > default_src_compile() |
19 |
> > > { __eapi2_src_compile; } |
20 |
> > > # bind supported phases to the defaults |
21 |
> > > + declare -F pkg_nofetch >/dev/null || \ |
22 |
> > > + pkg_nofetch() { default; } |
23 |
> > > declare -F src_unpack >/dev/null || \ |
24 |
> > > src_unpack() { default; } |
25 |
> > > declare -F src_prepare >/dev/null || \ |
26 |
> > > |
27 |
> > |
28 |
> > LGTM. I'm pretty sure there are no other issues with |
29 |
> > 772ed29fd9e7cf722aed943adbe33a27f250e1ff, so I'd say we should go |
30 |
> > ahead and merge it with these 2 fixes. |
31 |
> |
32 |
> yeah, my thought too. |
33 |
> |
34 |
> It's also why I was bugging him to get this fixed, so we could |
35 |
> re-apply it all. |
36 |
> |
37 |
|
38 |
committed all 3 patches merged into one commit. |
39 |
|
40 |
-- |
41 |
Brian Dolbec <dolsen> |