Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 52682 - [New game ebuild] battalion-1.4b
Summary: [New game ebuild] battalion-1.4b
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: x86 Linux
: High enhancement (vote)
Assignee: Gentoo Games
URL: http://evlweb.eecs.uic.edu/aej/AndyBa...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-06-01 06:41 UTC by Alexandru Toma
Modified: 2004-06-03 16:31 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
battalion-1.4b.ebuild (battalion-1.4b.ebuild,1.66 KB, text/plain)
2004-06-01 06:43 UTC, Alexandru Toma
Details
battalion-1.4b.ebuild (battalion-1.4b.ebuild,1.67 KB, text/plain)
2004-06-03 02:41 UTC, Alexandru Toma
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Alexandru Toma 2004-06-01 06:41:47 UTC
From The Linux Game Tome:
"If you've ever dreamed of turning into a rampaging monster and trashing small cities while blowing up the helicopters and tanks which are trying to stop you, here's your chance."

Just ignore the "BATTALION ERROR: Could not find the data files" error. It is because the game can't find battalion.sho. I don't know what this is for but, apparently, it is not needed (the game runs fine wothout it) as it is not in any of the Linux souce tarballs. I could've edited it out but I thought another sed line in the ebuild was unnecessary.

Reproducible: Always
Steps to Reproduce:
1.
2.
3.
Comment 1 Alexandru Toma 2004-06-01 06:43:40 UTC
Created attachment 32456 [details]
battalion-1.4b.ebuild
Comment 2 Alexandru Toma 2004-06-03 02:41:40 UTC
Created attachment 32576 [details]
battalion-1.4b.ebuild
Comment 3 Chris Gianelloni (RETIRED) gentoo-dev 2004-06-03 03:49:56 UTC
What category would you put this in?  games-action? games-arcade?
Comment 4 Alexandru Toma 2004-06-03 03:56:15 UTC
I think games-action would be fine.
Comment 5 Chris Gianelloni (RETIRED) gentoo-dev 2004-06-03 06:44:14 UTC
Added to CVS... thanks for the submission!
Comment 6 Alexandru Toma 2004-06-03 15:47:55 UTC
I think there may be a possible mistake in the ebuild I submited. Wouldn't it be better if the "insinto ${GAMES_STATEDIR}" from src_install() was "dodir ${GAMES_STATEDIR}"? I don't know if insinto creates the directory or the doins after it creates (I don't have time to look at the eclasses at the moment). By using dodir we would make sure that directory is actually created if it doesn't exist.
Comment 7 Mr. Bones. (RETIRED) gentoo-dev 2004-06-03 16:00:51 UTC
yep.  fixed.
Comment 8 Chris Gianelloni (RETIRED) gentoo-dev 2004-06-03 16:07:06 UTC
You're kinda right... dodir makes the directories... but you still need insinto if you are going to be doing any doins calls... I just prepended a dodir into the for; do loop and also near the end...

thanks for pointing it out... I completely missed it...
Comment 9 Alexandru Toma 2004-06-03 16:31:54 UTC
I was talking about another insinto... the one above
touch ${D}${GAMES_STATEDIR}/battalion_hiscore
fperms 660 ${GAMES_STATEDIR}/battalion_hiscore

I don't think you need to add a dodir in the for loop since doins already creates the insinto directory... that was the problem ... I didn't know if insinto created the directory on its own and Mr. Bones just confirmed that it doesn't.... doins however, correct me if I'm wrong, but I think it creates the directory