Gentoo Archives: gentoo-dev

From: "Tod M. Neidt" <tod@g.o>
To: gentoo-dev@g.o
Subject: Re: [gentoo-dev] xcircuit
Date: Fri, 05 Apr 2002 16:31:43
Message-Id: 1018038295.3095.0.camel@silica.localmosci
In Reply to: Re: [gentoo-dev] xcircuit by Carl-Christian Salvesen
1 On Fri, 2002-04-05 at 05:35, Carl-Christian Salvesen wrote:
2 > On Fri, 2002-04-05 at 11:27, michael wrote:
3 > > could someone make an ebuild for the latest version of xcircuit?
4 > > :)
5 >
6 > Looks like gentoo.org is down. Can you confirm that my ebuild works?
7 > http://struct.homeip.net/gentoo/beta/
8 >
9 > Just unpack it in /usr/portage and emerge xcircuit..
10 Hi!
11
12 Already made one for xcircuit-2.5.4 (the website implies this one is
13 prefereable). I added it to app-misc ( arguable where it should go).
14 Although an official stable one might be nice too :)
15
16 What follows is offered as constructive criticism. If you clean up your
17 ebuild in light of the following comments, I would be more than happy to
18 commit it for you.
19
20 1. Remove all the unnecessary comments leftover from the skel.ebuild.
21
22 2. This application has dependencies. Add them to DEPENDS, not everyone
23 who will emerge this will already have all the dependencies merged. Read
24 the docs, website, and/or 'ldd xcircuit' to figure out the dependencies.
25
26 3. Although it doesn't hurt anything, this app has no info file so the
27 --info= line in the configure can be removed.
28
29 4. xcircuit-2.5.4 had a non-fatal compile error related to parameter.c
30 looking for menudep.h with a parallel compile (emake) and needed to be
31 compiled with a ordinary make to avoid this. Did you confirm that
32 xcircuit-2.3.3 compiled without error? Although the app runs, I imagine
33 that the functionality is impaired.
34
35 Your contributions to Gentoo Linux are appreciated. Tidy up your
36 xcircuit-2.3.3.ebuild and mail it as an attachement to tod@g.o
37 and we'll get it committed into the portage tree.
38
39 Thanks!
40
41 tod
42
43 P.S. You can also refer to app-misc/xcircuit/xcircuit-2.5.4.ebuild