Red Hat Bugzilla – Bug 461454
Review Request: lordsawar - Turn-based strategy game in a fantasy setting
Last modified: 2009-01-14 22:08:53 EST
Spec URL: http://ianweller.fedorapeople.org/SRPMS/lordsawar/0.1.1-1/lordsawar.spec
SRPM URL: http://ianweller.fedorapeople.org/SRPMS/lordsawar/0.1.1-1/lordsawar-0.1.1-1.fc9.src.rpm
This game is meant to be a clone of Warlords II by SSG, but the most
important feature of this game is freedom.
It is not generally acceptable to indicate that some software is a "clone of Foo" where "Foo" is some trademarked entity.
We generally have to handle these things on a case-by-case basis, so I'll block FE-Legal and the folks who know can comment on what needs to be changed.
Even barring legal issues, the package summary could do a better
job at describing what "lordsawar" is.
> Summary : A clone of Warlords II
Summary: Turn-based strategy game in a fantasy setting
To advertise "a clone" is rather dangerous with regard to gamers'
expectations. In particular, here it seems much must be done
before it would look and feel like the original Warlords II.
And still, the typical user with interest in games would search
for terms like "strategy", "fantasy", "role-playing", in a list
of pkg summaries or in a summary of game pkgs. It's likely that
users don't even know what "Warlords II" is - and for those who
do this is not Warlords II yet. Just advertise what this is at
present instead of what it aims at.
I was definitely going to do this the moment Jason talked about the legality of the description and summary, but I got sidetracked. Thanks for the reminder.
Sometime today I'll have new stuff up and I'll remove the FE-Legal blocker. Unless that's a "bad thing".
* Tue Sep 16 2008 Ian Weller <email@example.com> 0.1.1-2
- Summary and description changed
It's unstable. I'd like to advise against publishing this.
#8 <signal handler called>
#9 ArmyBase (this=0xb474d2cc, a=@0x4) at armybase.cpp:36
#10 0x0815411c in ArmyProtoBase (this=0xb474d2cc, a=@0x4)
#11 0x081538bf in ArmyProdBase (this=0xb474d2c8, a=@0x0) at armyprodbase.cpp:34
#12 0x08130551 in Action_Produce::fillData (this=0xb459bdd0, army=0x0,
city=0x9ffa7a8, vectored=false) at action.cpp:1774
#13 0x080e4e2d in Player::cityProducesArmy (this=0x9dad800, city=0x9ffa7a8)
#14 0x080a013d in City::nextTurn (this=0x9ffa7a8) at city.cpp:583
#15 0x080a5033 in Citylist::nextTurn (this=0x9bdf2d0, p=0x9dad800)
#16 0x0821d566 in NextTurnHotseat::startTurn (this=0xb4f00638)
#17 0x0821daaf in NextTurnHotseat::start (this=0xb4f00638)
#18 0x0821d922 in NextTurnHotseat::endTurn (this=0xb4f00638)
#19 0x08084c8f in Game::end_turn (this=0xa00c988) at game.cpp:239
#20 0x0748a392 in Glib::SignalProxyNormal::slot0_void_callback ()
#21 0x00ba18b4 in g_cclosure_marshal_VOID__VOID ()
#22 0x00b9412b in g_closure_invoke () from /lib/libgobject-2.0.so.0
#23 0x00ba8ed8 in ?? () from /lib/libgobject-2.0.so.0
#24 0x00ba9ece in g_signal_emit_valist () from /lib/libgobject-2.0.so.0
#25 0x00baa336 in g_signal_emit () from /lib/libgobject-2.0.so.0
#26 0x004b4ae9 in gtk_button_clicked () from /usr/lib/libgtk-x11-2.0.so.0
#27 0x004b5f4a in ?? () from /usr/lib/libgtk-x11-2.0.so.0
#28 0x09f33960 in ?? ()
#29 0x07ef1700 in gtk_window_set_has_frame () from /usr/lib/libgtkmm-2.4.so.1
#30 0xbfe700c8 in ?? ()
#31 0x07d4c2d9 in Gtk::Button::on_released () from /usr/lib/libgtkmm-2.4.so.1
I was following a bit upstream development (I don't have the time
right now), and though it is indeed unstable, upstream is rather
responsive. So maybe this could be in rawhide only until crashes
are fixed, since otherwise it is playable (at least stand-alone)?
Might be GCC 4.3 issues, since the dev is on Ubuntu. I'll show him this stuff and see what he says.
This bug has been fixed in upstream. For more information see http://savannah.nongnu.org/bugs/?24330 .
I'd like it if somebody would synchronize these bugs with the tracker on savannah so I can react more promptly.
Developer sending patch, will rebuild and test, then post here.
Created attachment 317326 [details]
patch to fix crash
This patch fixes a crash in lordsawar when active neutral cities have produced a preset maximum number of army units.
Spec URL: http://ianweller.fedorapeople.org/SRPMS/lordsawar/0.1.3-1/lordsawar.spec
SRPM URL: http://ianweller.fedorapeople.org/SRPMS/lordsawar/0.1.3-1/lordsawar-0.1.3-1.fc9.src.rpm
* Sat Oct 25 2008 Ian Weller <firstname.lastname@example.org> 0.1.3-1
- Updated upstream
I'm inclined to say "let's make this a test update at least".
It works better and doesn't crash as easily as before.
Issues I've run into (and which may cause users to submit problem
* Music became loud noise after the first fight. Pulse Audio was
still working fine (i.e. I could start a music player as a test), but
in lordsawar it needed several attempts to make the music return.
* At the end of a test game (the remaining two enemies surrendered),
confirming "Quit" did not close the game window. I could choose and
confirm Quit again and again, but that didn't help. Killing the process
The build log contains a few interesting warnings that are worth
gtksdl.c:218: warning: the address of 'SDL_windowhack' will always evaluate as '
playerlist.cpp:367: warning: deprecated conversion from string constant to 'char
player.cpp:862: warning: suggest parentheses around && within ||
Packaging-wise it's fine.
* A desktop menu icon could be added.
* You could use --vendor="" instead of --vendor="fedora" when
installing the .desktop file
oh ok! Thanks, Michael.
New Package CVS Request
Package Name: lordsawar
Short Description: Turn-based strategy game in a fantasy setting
InitialCC: (if you can do email addresses...) email@example.com
sadly, no, we can't do arbitrary email addresses in InitialCC. ;(
If they get a fas account we can easily add them however.
lordsawar-0.1.3-3.fc10 has been submitted as an update for Fedora 10.
lordsawar-0.1.3-3.fc10 has been pushed to the Fedora 10 testing repository. If problems still persist, please make note of it in this bug report.
If you want to test the update, you can install it with
su -c 'yum --enablerepo=updates-testing update lordsawar'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F10/FEDORA-2008-10466
lordsawar-0.1.4-2.fc10 has been submitted as an update for Fedora 10.
lordsawar-0.1.4-2.fc10 has been pushed to the Fedora 10 testing repository. If problems still persist, please make note of it in this bug report.
If you want to test the update, you can install it with
su -c 'yum --enablerepo=updates-testing update lordsawar'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F10/FEDORA-2008-11488
lordsawar-0.1.4-2.fc10 has been pushed to the Fedora 10 stable repository. If problems still persist, please make note of it in this bug report.