From mboxrd@z Thu Jan 1 00:00:00 1970 X-Spam-Checker-Version: SpamAssassin 4.0.0 (2022-12-14) on finch.gentoo.org X-Spam-Level: * X-Spam-Status: No, score=1.1 required=5.0 tests=DATE_IN_PAST_12_24,DMARC_NONE, INVALID_DATE,MAILING_LIST_MULTI autolearn=no autolearn_force=no version=4.0.0 Received: from mailgw1.netvision.net.il ([194.90.1.14]) by cvs.gentoo.org with esmtp (Exim 3.30 #1) id 15Jf0g-0006th-00 for gentoo-dev@cvs.gentoo.org; Mon, 09 Jul 2001 11:39:50 -0600 Received: from localhost (ras4-p82.rlz.netvision.net.il [62.0.85.210]) by mailgw1.netvision.net.il (8.9.3/8.9.3) with SMTP id UAA21029 for ; Mon, 9 Jul 2001 20:38:59 +0300 (IDT) Content-Type: text/plain; charset="iso-8859-1" From: Dan Armak To: gentoo-dev@cvs.gentoo.org Subject: Re: [gentoo-dev] New version of heroes ebuild X-Mailer: KMail [version 1.2] References: <01070914195900.00558@localhost> <20010709095449.C14752@cvs.gentoo.org> In-Reply-To: <20010709095449.C14752@cvs.gentoo.org> MIME-Version: 1.0 Message-Id: <01070920395701.00654@localhost> Content-Transfer-Encoding: 8bit Sender: gentoo-dev-admin@cvs.gentoo.org Errors-To: gentoo-dev-admin@cvs.gentoo.org X-BeenThere: gentoo-dev@cvs.gentoo.org X-Mailman-Version: 2.0 Precedence: bulk Reply-To: gentoo-dev@cvs.gentoo.org List-Help: List-Post: List-Subscribe: , List-Id: Gentoo Linux development list List-Unsubscribe: , List-Archive: Date: Mon Jul 9 11:40:02 2001 X-Original-Date: Mon, 9 Jul 2001 20:39:57 +0300 X-Archives-Salt: 28dfde69-0dee-4c4b-93c2-ea401117e86d X-Archives-Hash: fca3ea9083f4067a716f3aeeed996e0f On Monday 09 July 2001 18:54, you wrote: > On Mon, Jul 09, 2001 at 02:19:59PM +0300, Dan Armak wrote: > > The original version I posted installed a file into /doc: a very strange > > default on the part of the makefile. The ebuild now passes a special > > paramter to configure to make it install into /usr/share/doc/${P} as > > needed. > > > > DEPEND="virtual/x11" > > > > if [ "`use sdl`" ]; then > > DEPEND="${DEPEND} media-libs/libsdl media-libs/sdl-mixer" > > confopts="${confopts} --with-sdl --with-sdl-mixer" > > else # `use ggi` isn't checked since this is the game's default > > DEPEND="${DEPEND} media-libs/libggi media-libs/libgii > > media-libs/libmikmod" > > confopts="${confopts} --with-ggi --with-mikmod" > > fi > > > > DEPEND="media-libs/libsdl media-libs/sdl-mixer virtual/x11" > > OK, some notes. First, notice that you just overwrote your DEPEND settings > with this line above, so any DEPEND tweaking you did previously was lost. > Did you mean to use RDEPEND instead? If so, it defaults to be the same a > DEPEND if not defined. Terribly sorry! - my mistake. This is a line I thought I'd deleted or commented out. You should just delete it. Then it will be OK. > > Next, you can use our handy DEPEND/RDEPEND syntax to do stuff like this: > > DEPEND="virtual/x11 sdl? ( media-libs/libsdl media-libs/sdl-mixer ) ggi? ( > media-libs/libggi media-libs/libgii )" > Yes but: if the game needs either SDl of libGGI, this syntax doesn't make sure at least something is defined in DEPEND. If neither USE variable is set, niether lib will be installed and the game won't compile as it needs at least 1 frontend lib. > Also, it may not be good to use sdl *or* libggi. Can one compile-in both > options? If not, then the best approach (taking into account the current > functionality of the ebuild system) is to create two packages: heroes-sdl > and heroes-ggi. Right now, there's no good way for Portage to choose > between two existing USE variables; ie. if ggi and sdl are defined, Portage > has no way of knowing which one should be preferred. > Yes but: the heroes game developers themselves prefer SDL over GGI. This is because it is better maintained. (according to the readme file). And no, you can't compile both options. Unless you patch the Makefile to compile 2 different executables. But in that case it's better to make 2 diferent ebuilds - however, they will be mutually exclusive. But it's easy enough to separate into heroes-ggi and heroes-sdl. Do that if you think that's what's needed. > > src_unpack() { > > > > unpack ${A1} ${A2} ${A3} ${A4} > > > > } > > The default src_unpack function should perform this step for you, so you > should be able to remove the src_unpack() function from your ebuild. Also, > the preferred way of doing things now is to remove the A= definition from > your ebuild and just use SRC_URI: I wasn't sure about that. > > SRC_URI="http://prdownloads.sourceforge.net/heroes/heroes-0.11.tar.gz > http://prdownloads.sourceforge.net/heroes/heroes-data-1.1.tar.gz > http://prdownloads.sourceforge.net/heroes/heroes-sound-tracks-1.0.tar.gz > http://prdownloads.sourceforge.net/heroes/heroes-sound-effects-1.0.tar.gz" > > Other than that, it looks good and I look forward to playing it soon! :) OK. I'll learn from my mistakes :-) Dan