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

Full Text Bug Listing

Submitting an ebuild for the sarien-0.7.0 AGI game engine

Reproducible: Always
Steps to Reproduce:

Created an attachment (id=28067) [details]
ebuild

games-engines/sarien added to CVS.  thanks for the bug report and ebuild.

Thanks for adding the ebuild to the CVS, I seem to be having a problem
installing it however.

When I tried installing this morning, compilation failed with a large list of
"undefined references".

main.o(.text+0x7e): In function `main':
: undefined reference to `init_video'
main.o(.text+0xc5): In function `main':
: undefined reference to `report'
main.o(.text+0xca): In function `main':
: undefined reference to `console_init'
main.o(.text+0xd6): In function `main':
: undefined reference to `report'
main.o(.text+0x121): In function `main':
: undefined reference to `report'

<snip>

dobin: making bin/sarien executable...
install: cannot stat `bin/sarien': No such file or directory


However, switching to the work directory and manually running make created the
binary successfully.

> cd /var/tmp/portage/sarien-0.7.0/work/sarien-0.7.0
> make
> ls bin/

total 172K
-rw-r--r--    1 root     root          171 Sep  2  2001 Makefile
-rwxr-xr-x    1 root     root         157K Mar 29 10:44 sarien*
-rw-r--r--    1 root     root         2.7K Sep  2  2001 sarien.ico
-rwxr-xr-x    1 root     root          379 Sep  2  2001 sarien.info*

I couldn't see how make was being called in your updated ebuild, any ideas on
what could be causing compile to fail?

Try it with MAKEOPTS="-j1"

Excellent, that fixed the compilation issues. Thank you.

But may I ask why that change was necessary? I have not experienced any problems compiling any other ebuilds.

It would be a pain if all users had to update their make.conf file just for this ebuild.

Thanks for your help!

The problem is caused by buggy makefiles that don't properly list dependencies
within the build.  I've put a work around into CVS so this shouldn't be a problem
for anyone else.  Thanks for reporting it.