Gentoo Archives: gentoo-alt

From: Jason Cooper <gentoo@××××××××××.net>
To: gentoo-alt@l.g.o
Subject: Re: [gentoo-alt] [PATCH] ccze-0.2.1-r3: add ~x64-macos with dylib
Date: Wed, 15 Jan 2014 21:05:36
Message-Id: 20140115210531.GE29184@titan.lakedaemon.net
In Reply to: Re: [gentoo-alt] [PATCH] ccze-0.2.1-r3: add ~x64-macos with dylib by "François Bissey"
1 On Thu, Jan 16, 2014 at 09:55:56AM +1300, François Bissey wrote:
2 > On Wed, 15 Jan 2014 15:53:38 Jason Cooper wrote:
3 > > All,
4 > >
5 > > So swapping out to dylib was easier than I thought. If I don't hear any
6 > > complaints, I'll post this to bugzilla.
7 > >
8 > > thx,
9 > >
10 > > Jason.
11 > >
12 > > --------------->8-----------------------
13 > > --- ccze-0.2.1-r3.ebuild.orig 2012-04-01 06:02:50.000000000 -0400
14 > > +++ ccze-0.2.1-r3.ebuild 2014-01-15 15:48:06.000000000 -0500
15 > > @@ -14,7 +14,7 @@
16 > >
17 > > LICENSE="GPL-2"
18 > > SLOT="0"
19 > > -KEYWORDS="amd64 ppc x86 ~x86-fbsd"
20 > > +KEYWORDS="amd64 ppc x86 ~x86-fbsd ~x64-macos"
21 > > IUSE=""
22 > >
23 > > DEPEND="dev-libs/libpcre
24 > > @@ -36,6 +36,14 @@
25 > > sed -e '/AC_CHECK_TYPE(error_t, int)/d' \
26 > > -i configure.ac || die
27 > >
28 > > + # macos dylib fix
29 > > + if [[ ${CHOST} == *-darwin* ]] ; then
30 > > + sed -e "s:-shared$:-dynamiclib -undefined suppress -flat_namespace
31 > > -install_name ${EPREFIX}/usr/lib/ccze/\$@:" \ + -i
32 > > src/Makefile.in || die
33 > > + sed -e "s:\.so:.dylib:g" \
34 > > + -i src/Makefile.in src/ccze-plugin.c Rules.mk.in || die
35 > > + fi
36 > > +
37 > > eautoreconf
38 > >
39 > > ht_fix_file Rules.mk.in
40 >
41 >
42 > Looks fine. I would have done the dylib bit before the install name bit.
43 > Not that it matters anyway.
44
45 Ok, thanks.
46
47 https://bugs.gentoo.org/show_bug.cgi?id=498200
48
49 On to other packages.
50
51 thx,
52
53 Jason.