Gentoo Archives: gentoo-portage-dev

From: Brian Dolbec <dolsen@g.o>
To: gentoo-portage-dev@l.g.o
Subject: Re: [gentoo-portage-dev] [PATCH] Add missing bind for pkg_nofetch
Date: Fri, 26 Sep 2014 19:04:18
Message-Id: 20140926120355.58e9729b.dolsen@gentoo.org
In Reply to: Re: [gentoo-portage-dev] [PATCH] Add missing bind for pkg_nofetch by Brian Dolbec
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>