Follow-up Comment #3, bug #20095 (project freeciv):
How about also moving them to _LDADD from _LDFLAGS?
(file #16436, file #16437)
___
Additional Item Attachment:
File name: ToluaLDADD.patch Size:0 KB
File name:
Follow-up Comment #3, bug #16976 (project freeciv):
Attached a patch with these fixes, plus bug #19871 and some misc. leader name
changes.
(file #16438)
___
Additional Item Attachment:
File name: leader_name_consistency.patch Size:10 KB
Follow-up Comment #2, bug #19871 (project freeciv):
picked up in bug #16976
___
Reply to this item at:
http://gna.org/bugs/?19871
___
Message sent via/by Gna!
http://gna.org/
Follow-up Comment #3, bug #20097 (project freeciv):
@comment #2:
That's exactly what I've meant by I can't reproduce - this works correctly
here, even if the code isn't really correct.
___
Reply to this item at:
Follow-up Comment #4, bug #20095 (project freeciv):
Strangely enough, still bailing out with the same message. (And yes, I ran
make clean and autogen.sh inbetween...)
Here are a couple of massive logs, if this helps you at all.
(file #16439)
Follow-up Comment #5, bug #20095 (project freeciv):
Too massive for Gna, so now here cropped in half.
(file #16440)
___
Additional Item Attachment:
File name: daniel-make.txtSize:455 KB
Update of patch #3510 (project freeciv):
Status: Ready For Test = Done
Assigned to:None = cazfi
Open/Closed:Open = Closed
Follow-up Comment #6, bug #20095 (project freeciv):
Perhaps a silly question, but does OS X have a *separate* libm ?
(IIRC, that's a valid question for i.e. OpenBSD (or was that just about were
*some* of math functions are ?))
Also, both LUA_AS_DEPENDENCY and LUA_LIBS carry '-lm'.
As for the
URL:
http://gna.org/patch/?3515
Summary: Update stdlib_h.m4 to upstream serial 39
Project: Freeciv
Submitted by: cazfi
Submitted on: Mon 27 Aug 2012 05:38:09 PM EEST
Category: bootstrap
Priority: 5 -
Follow-up Comment #7, bug #20095 (project freeciv):
Looking at tolua-5.1/src/bin/Makefile.am again, *unless* I'm missing
something, after changing _LDFLAGS to _LDADD it would seem tolua_DEPENDENCIES
are redundant - see if removing it helps.
Update of bug #20095 (project freeciv):
Status: Ready For Test = In Progress
___
Follow-up Comment #8:
Also, both LUA_AS_DEPENDENCY and LUA_LIBS carry '-lm'.
Oh, that explains it. LUA_AS_DEPENDENCY
Follow-up Comment #9, bug #20095 (project freeciv):
Fix to not to pollute LUA_AS_DEPENDENCY with -lm. Note that all places that
link against liblua are affected - tolua just happens to be first one.
_LDFLAGS - _LDADD change was just a test, and I don't think it's even right
thing to do (we are
Follow-up Comment #10, bug #20095 (project freeciv):
What about what I've wrote in comment #7 ?
After all, the reason dependencies were not computed correctly before was
exactly that those libs were passed via _LDFLAGS instead of _LDADD.
Follow-up Comment #11, bug #20095 (project freeciv):
...also passing '-lm' to libtolua_la_LIBADD (if it's actually needed there and
not for liblua itself) should make it easier to propagate it down the tree.
___
Reply to this item at:
Update of patch #3464 (project freeciv):
Category:None = general
Status:None = Ready For Test
Planned Release: = 2.5.0
Dear all,
first of all let me thank you for this wonderful game. I
bought Civilization on Amiga, at its first release, and since then I'm
a fan of this saga.
I discovered FreeCiv just an year ago, and I keep
on playing it regularly during lunch time, thanks to its low graphical
requirements
Follow-up Comment #12, bug #20095 (project freeciv):
OK, it's better to write the patch on your own.
This is against trunk r21843.
freeciv-server, freeciv-manual, freeciv-modpack and gtk2/gtk3 clients seems to
build correctly with it.
Please test.
(file #16445)
Follow-up Comment #5, bug #20057 (project freeciv):
Your patch fixes the reported error, thank you. But now I get a different
error:
trunk:
cc1: warnings being treated as errors
map.c: In function ‘startpos_iter_init’:
map.c:1608: warning: type-punning to incomplete type might break
Follow-up Comment #14, bug #20045 (project freeciv):
I'm afraid it keeps failing with the same error. I tested S_4 r21843 that
includes the mentioned patch.
The error appears when I reload the savegame posted here, and also when I
start a new clean game.
1: in unit_virtual_destroy()
19 matches
Mail list logo