Home | Docs | Forums | Lists | Bugs | Planet | Store | GMN | Get Gentoo!

Full Text Bug Listing

Hey,

This is an update for the ebuild for Eternal Lands, which makes it compatible with the new version. Since the server has switched to 1.0.1 (as of 25th of january) and 1.0.0 has been made obsolete (older clients are not able to connect any more) it is a necissary upgrade.
The ebuild has a few new features, which are accessible through the following USE-flags:

music - Downloads and installs the optional music package (~25MB)
mapeditor - Compiles the mapeditor and copies an introduction to this to the documentation folder. It uses gtk-1.2 currently I figured it was best to make it optional - feel free to change this if you wish; but you can easily play Eternal Lands without having the map editor installed - it's just an add-on like the music.
doc - Generates documentation for the client source using Doxygen and copies it to to the documentation folder.

For now, since the mirrors do not have the 1.0.1-sources, you will have to fetch the source from berlios and put it in your ${DISTDIR}:
ftp://ftp.berlios.de/pub/elc/elc_101.tgz
ftp://ftp.berlios.de/pub/elc/mapedit_101.tgz

Eternal Lands is an open source mmorgp written in C using SDL/OpenGL. 
Go to http://www.eternal-lands.com to read more.

Best regards,
Bj

Hey,

This is an update for the ebuild for Eternal Lands, which makes it compatible with the new version. Since the server has switched to 1.0.1 (as of 25th of january) and 1.0.0 has been made obsolete (older clients are not able to connect any more) it is a necissary upgrade.
The ebuild has a few new features, which are accessible through the following USE-flags:

music - Downloads and installs the optional music package (~25MB)
mapeditor - Compiles the mapeditor and copies an introduction to this to the documentation folder. It uses gtk-1.2 currently I figured it was best to make it optional - feel free to change this if you wish; but you can easily play Eternal Lands without having the map editor installed - it's just an add-on like the music.
doc - Generates documentation for the client source using Doxygen and copies it to to the documentation folder.

For now, since the mirrors do not have the 1.0.1-sources, you will have to fetch the source from berlios and put it in your ${DISTDIR}:
ftp://ftp.berlios.de/pub/elc/elc_101.tgz
ftp://ftp.berlios.de/pub/elc/mapedit_101.tgz

Eternal Lands is an open source mmorgp written in C using SDL/OpenGL. 
Go to http://www.eternal-lands.com to read more.

Best regards,
Bjørn

Reproducible: Always
Steps to Reproduce:
1.
2.
3.

Created an attachment (id=49496) [details]
eternal-lands-1.0.1.ebuild

Initial version of the ebuild for Eternal Lands 1.0.1.

Created an attachment (id=49497) [details]
digest-eternal-lands-1.0.1

The digest for the Eternal Lands 1.0.1 ebuild

Created an attachment (id=49507) [details]
eternal-lands-1.0.1.ebuild (2)

Removes a bug where the post_install would lead to the wrong dir for the
documentation. 
Also this ebuild doesn't copy an obsolete knowledge.lst that resided in ./ (an
issue with the .zip)

Unfortunately the browser.lst that was found in the CVS directory of the map
editor was old, hence I will upload an updated one. I think that this can go in
${FILESDIR} since it's only 9.7KB when it's gzipped, but you'll have to decide
upon that.

Created an attachment (id=49508) [details]
A newer browser.lst - compressed

A compressed version of the browser.lst. To go in ${FILESDIR} if possible.

(From update of attachment 49508 [details])
Forgot the md5sum:
cac8299f8e3e2f85c929e19d90b69b2e

gzipped stuff should never go in files/

Created an attachment (id=49516) [details]
eternal-lands-1.0.1.ebuild (3)

OK, I removed the compressed browser.lst. 
Thanks for the correction, I was not sure on whether it was allowed or not
(there are several ebuilds in the repository that have gzipped files, so it
confused me a bit).

After reviewing the new browser.lst I found that it was not needed with the
current 2d/3d objects - many of the objects included in the new list will only
be used/available in 1.0.2. 
The map editor in this ebuild is now fully up to date and has the same features
as in our windows release.

One last thing that might be an error in that ebuild:

I was in doubt about the IUSE-flags, and whether or not amd64 should be included in it. Since I am using the amd64 USE-flag to add -DX86_64 when compiling the source it would seem logical that it needs to be defined (since you have to do that with other use flags). 
But is this the way you want it, or should IUSE not include amd64?

IUSE="doc music mapeditor amd64"

yeah, arch flags never go in IUSE

Created an attachment (id=49535) [details]
eternal-lands-1.0.1.ebuild (4)

OK, thanks. This ebuild removes amd64 from IUSE.

Tell me if there's anything else in the ebuild that you want changed.

I'd like to see the music installed by default and turned off with a use flag
instead of having to turn it on.

take out the in-line use docs at the top.  take out the messages that get
spewed turning pkg_setup (actually, just take out pkg_setup)

remove some of the extra comments you added from the ebuild in portage right
now.

fix the date in the header

it probably doesn't rdepend on the doc? () stuff does it?

move the OPTIONS, and S_ variables into a function (top of src_unpack)

error check the newgamesbin calls

use dohtml for the documentation instead of cp

prepgamesdirs should be the last thing in src_install

Created an attachment (id=49593) [details]
eternal-lands-1.0.1.ebuild (5)

I made the changes you wanted and changed the filename of music.zip to
el_music_101.zip for future updates. The mirror with the music is horribly slow
right now, so hopefully it'll reach the gentoo mirrors soon :)

Any other problems you see with this ebuild?

Created an attachment (id=49594) [details]
digest-eternal-lands-1.0.1 (2)

Changes the filename of music.zip to el_music_101.zip

Created an attachment (id=49607) [details]
eternal-lands-1.0.1.ebuild (6)

I was only defining S_CLIENT and S_MAPEDITOR in the unpack function which may
create problems if the src_compile() was called directly. It's now defined in
the src_compile as well.

no don't do that.  portage will keep track of the variables between function
calls.

OK, I reversed it - the current ebuild is now eternal-lands-1.0.1.ebuild (5).

doesn't work because of the bad logic wrt mapeditor.

the use mapeditor && ...

needs to look like

if use mapeditor ; then
...
fi

Created an attachment (id=49617) [details]
eternal-lands-1.0.1.ebuild (7)

Ah, sorry about that, a stupid error on my part.

It is fixed now.

thanks for a quick new ebuild but i cant seem to find el_music_101.zip on
berlios, can someone point me to that file. then i will test the ebuild. 
thanks

It's not on berlios. Why don't you just use the ebuild to fetch it? 

Anyway here it is.
http://el.tfm.ro/el_music_101.zip

thanks, after about 45 minutes the ebuild just timed out fetching the music
file

Yea, the mirror is in Romania, so do not expect too much... That's really also
why I made the music an addon in the beginning :)
For now just emerge it with:

games-rpg/eternal-lands nomusic

In your /etc/portage/package.use

version bumped.  Thanks for the work on the ebuild and the bug report.