Follow-up Comment #12, bug #17236 (project freeciv):
rebased patch; changes:
- simplify usdlg_cmd
With the patch you can focus/center/sentry/ready units with different
selections. Please test!
(file #13928, file #13929)
___
Additional
Update of patch #2247 (project freeciv):
Status:None = Duplicate
Open/Closed:Open = Closed
___
Follow-up Comment #1:
this patch is
Follow-up Comment #1, patch #2480 (project freeciv):
rebased patch
(file #13930)
___
Additional Item Attachment:
File name: 20110819-transport08-allow-recursive-transports.patch Size:6 KB
Follow-up Comment #2, patch #2483 (project freeciv):
updated patch; the last one missed one important change - the sprites were
not displayed at all ...
(file #13931)
___
Additional Item Attachment:
File name:
Follow-up Comment #5, patch #2270 (project freeciv):
I did update the entire patch series. It is working but tests are still
running. Due to changes in unit_move() autogames differ (transported units are
moved after it is sure that the transporter survives). I'm working on an
(optional) patch
Update of bug #17470 (project freeciv):
Status: Ready For Test = Fixed
Open/Closed:Open = Closed
___
Follow-up Comment #3:
fixed - qt has now
Follow-up Comment #6, patch #2270 (project freeciv):
and an all-in-one patch
(file #13932)
___
Additional Item Attachment:
File name: 20110819-transport.patch Size:170 KB
Follow-up Comment #16, patch #2513 (project freeciv):
You make both server and client directly link in
libscriptcore. It should be instead linked to libfreeciv only
(just like libaicore). That affects especially building with
shared libraries enabled.
This is done in patch #2923. With this
Follow-up Comment #2, patch #2923 (project freeciv):
If patch #2513 includes libscriptcore in libfreeciv I have to change the
makefile for the client as additional dependencies are added (lua!). Therefore
I did split it into patch #2513 (basic, smallest number of changes nad _no_
change to the
Follow-up Comment #2, patch #2921 (project freeciv):
I can commit these. But as we discussed long about the style of the commits I
would like to be sure that it is OK.
If it would be simpler for you to do all up to the rename at once, you can
also do it.
Follow-up Comment #18, patch #2513 (project freeciv):
Should I create a patch merging this one with patch #2923?
___
Reply to this item at:
http://gna.org/patch/?2513
___
Nachricht
Update of bug #18407 (project freeciv):
Status: Confirmed = Ready For Test
Planned Release: = 2.3.1
___
Follow-up Comment #3:
It's the search for
Follow-up Comment #4, bug #18407 (project freeciv):
updated patch - now it is the version I did compile ...
(file #13939)
___
Additional Item Attachment:
File name: 0002-limit-number-of-cm-iterations.patch Size:1 KB
Follow-up Comment #4, bug #18439 (project freeciv):
... and the changes to the ruleset? Your patch adds a check for unit gold
upkeep == 0. But this should never happen! The unitlist is created only with
units which have gold upkeep 0 (comment #0).
URL:
http://gna.org/bugs/?18527
Summary: ImageMagick - different types used
Project: Freeciv
Submitted by: syntron
Submitted on: Sa 20 Aug 2011 22:25:28 CEST
Category: general
Severity: 3 - Normal
Hello,
could you check that your config.log says about bz2? The error means that the
bzip2 was found but the devel library was not found (-lbz2). Try to install
libbz2-devel.
For opensuse 11.4 you will get the following error: http://gna.org/bugs/?
18527. I do have a patch but it is not a
Update of bug #18533 (project freeciv):
Category:None = general
Status:None = Ready For Test
Planned Release: = 2.3.1,2.4.0
Follow-up Comment #2, bug #18527 (project freeciv):
ok so it must be something else - perhaps I have a different version of
ImageMagick on the build server?
___
Reply to this item at:
http://gna.org/bugs/?18527
Update of bug #18532 (project freeciv):
Status:None = Confirmed
___
Follow-up Comment #1:
I can confirm this. If it is not an observer the amount of data send seems to
be to big so the
URL:
http://gna.org/bugs/?18535
Summary: fix health in early game (experimental ruleset)
Project: Freeciv
Submitted by: syntron
Submitted on: So 21 Aug 2011 14:27:59 CEST
Category: None
Severity: 3 -
Follow-up Comment #4, bug #18527 (project freeciv):
do you have a 32bit or a 64bit system? On 64bit there is no problem but on
32bit unsigned long int != size_t.
___
Reply to this item at:
http://gna.org/bugs/?18527
Follow-up Comment #2, bug #18535 (project freeciv):
I' not sure how the experimental ruleset is used. The default ruleset is
stable and well balanced but also quit old. The new features are not used. The
experimental ruleset is not that much tested or balanced but it includes all
the nice new
Follow-up Comment #2, bug #18513 (project freeciv):
such a module is available in freeciv as fc_interface.(c|h). I did use it for
tile vision and freeing of color definitions.
___
Reply to this item at:
http://gna.org/bugs/?18513
Follow-up Comment #20, patch #2513 (project freeciv):
What I think should be changed in this patch:
- Link libscriptcode to libfreeciv, not server
It is a chicken or egg problem (at least I dd not have a solution): If
libscriptcode is linked to libfreeciv it is also used in the client.
Update of patch #2513 (project freeciv):
Dependency Removed: = patch #2923
___
Reply to this item at:
http://gna.org/patch/?2513
___
Nachricht
Update of patch #2513 (project freeciv):
Dependency Removed: = patch #2923
___
Reply to this item at:
http://gna.org/patch/?2513
___
Nachricht
Hello Michal,
could you define that exactly you want to do that is not possible at the
moment? What code changes do you use?
Due to time limitations I'm not familiar with the development of longturn. Is
freeciv 2.3.0 OK for games there? Or are additional patches needed?
Matthias (aka syntron)
I did setup something similar to nightly builds on the opensuse build service
(https://build.opensuse.org/project/monitor?project=home%3Asyntron%3Afreeciv).
Besides all releases since 2.2.0 the branches S2_2 (2.2svn), S_3 (2.3.svn),
trunk (2.4svn) and trunk with additional patches (2.3.99plus)
Follow-up Comment #8, patch #2921 (project freeciv):
Sorry, I did only compile test the split of the big patch into small chunks.
One part was to add a new file - tolua_server.pkg - which contains server only
lua functions from all other tolua files. As you have found out, this file was
not
Am Montag, 22. August 2011, 21:42:12 schrieb Michal Mazurek:
On Mon, Aug 22, 2011 at 09:30:19PM +0200, Matthias Pfafferodt wrote:
Hello Michal,
could you define that exactly you want to do that is not possible at the
moment? What code changes do you use?
Due to time limitations I'm
Update of bug #18546 (project freeciv):
Category:None = general
Status:None = Ready For Test
Planned Release: = 2.3.1,2.4.0
Follow-up Comment #9, bug #18499 (project freeciv):
Could you propose a change how to should be done? Perhaps write an example
that should be shown in the research dialog with a short explanation.
___
Reply to this item at:
URL:
http://gna.org/bugs/?18547
Summary: fix number of bulbs for global goal
Project: Freeciv
Submitted by: syntron
Submitted on: Mo 22 Aug 2011 22:54:29 CEST
Category: client
Severity: 3 - Normal
Follow-up Comment #10, bug #18499 (project freeciv):
(comment #1:)
What I'm thinking about are 18 turns for 2731 bulbs - I think it
should be 18 turns for 2731-787=1944 bulbs?!
fixed in bug #18547
___
Reply to this item at:
Am Dienstag, 23. August 2011, 10:06:27 schrieb Michal Mazurek:
I'd like to disable global observers, and disable viewing the map after
a player dies. Are there some options for this?
Global observers you can (dis)allow via the allowtake setting. For the second
question you could check gna
Follow-up Comment #4, bug #18549 (project freeciv):
Could it be that there was a settler trying to irrigate the terrain? That
would result in an ocean tile ... In this case, a check if the tile can be
irrigated (new terrain with TER_NO_CITY?) would be needed.
Or, if the settler was in automode,
Follow-up Comment #3, bug #18546 (project freeciv):
I did try to continue, but without the information from savefile.options
there are to much possibilities and one will kill the server. You can try it
yourself - take a savegame, delete the line and open it ...
Update of bug #16008 (project freeciv):
Status:None = Duplicate
Open/Closed:Open = Closed
___
Follow-up Comment #7:
fixed by patch
Follow-up Comment #5, bug #18546 (project freeciv):
... but you need to add at least a check my_has_capability(), else you get
only this bug. This is my first try of the patch:
if (capstr == NULL) {
return FALSE;
}
___
Reply to this
Follow-up Comment #1, patch #2943 (project freeciv):
I do remember that this was discussed before but I do not find it at the
moment. After the old discussion I did create a patch which removed x,y,nat_x
and nat_y alltogether from the tile struct. I will serach for it and check if
it still is
Follow-up Comment #3, patch #2943 (project freeciv):
The patch was created at a time virtual tiles did not exists ... I have found
it but not tested if it applies to trunk.
___
Reply to this item at:
http://gna.org/patch/?2943
Follow-up Comment #2, patch #2922 (project freeciv):
bugfix and new patch attached
(file #13970, file #13971)
___
Additional Item Attachment:
File name: bugfix_prepare_lua2.diff Size:0 KB
File name: prepare_lua2.diff
Update of patch #2943 (project freeciv):
Status:None = Ready For Test
Planned Release: = 2.4.0
___
Follow-up Comment #4:
OK, here is the
Follow-up Comment #5, patch #2943 (project freeciv):
It was late yesterday - now, one day later you get the patch ...
(file #13973)
___
Additional Item Attachment:
File name: 0001-shrink-tile-struct-by-removing-x-y-nat_x-nat_y.patch
URL:
http://gna.org/bugs/?18564
Summary: fix function prototype for ggz_report_victor()
Project: Freeciv
Submitted by: syntron
Submitted on: Do 25 Aug 2011 22:00:40 CEST
Category: None
Severity: 3 -
Update of bug #18527 (project freeciv):
Status:None = Ready For Test
___
Follow-up Comment #5:
fix compiling of imagemagick
Some magick for ImageMagick - the interface has changed:
Update of bug #18564 (project freeciv):
Status:None = Fixed
Assigned to:None = syntron
Open/Closed:Open = Closed
URL:
http://gna.org/patch/?2947
Summary: new rpm spec file
Project: Freeciv
Submitted by: syntron
Submitted on: So 28 Aug 2011 23:33:54 CEST
Category: bootstrap
Priority: 5 - Normal
Follow-up Comment #8, bug #18549 (project freeciv):
Do you know if there was 'real' ocean at the tile or was it something like a
test if the change would improve the terrain?
___
Reply to this item at:
http://gna.org/bugs/?18549
Follow-up Comment #1, patch #2947 (project freeciv):
updated spec file; changes:
- split freeciv-tools in freeciv-manual and freeciv-modpack
- name space cleanup (dependency were named after non-existend packages)
- longer description (use freeciv header)
the following rpm lint errors are
Update of bug #18527 (project freeciv):
Status: Ready For Test = Fixed
Assigned to:None = syntron
Open/Closed:Open = Closed
Follow-up Comment #6, patch #2943 (project freeciv):
Is this something which would be OK as patch? If yes I would fix the other
clients (sdl, xaw, gtk3; ftwl and win32 are kind of dead).
___
Reply to this item at:
Update of patch #2923 (project freeciv):
Summary: [lua] include libscriptcore in libfreeciv.la =
[lua] fix libscriptcore as part of libfreeciv so that it can be used by the
client
___
Follow-up Comment #4:
For the
Update of patch #2943 (project freeciv):
Assigned to:None = syntron
Summary: Shave 8/24 bytes of struct tile = Remove
x,y,nat_x,nat_y from struct tile
___
Follow-up Comment
Follow-up Comment #15, bug #18549 (project freeciv):
So its the test for the activity result (transform) on a virtual tile on
which a city is located. I think the solution would be to add a check in
tile_transform:
untested patch (check from tile_set_terrain()):
++
static void
The check is implemented like this since 2004 ...Considering patch #2943
(https://gna.org/patch/?2943), the index values should be compared:
bool same_pos(const struct tile *tile1, const struct tile *tile2)
{
fc_assert_ret_val(tile1 != NULL tile2 != NULL, FALSE);
return (tile1-index ==
Am Sonntag, 4. September 2011, 13:10:56 schrieb Jordi Negrevernis i Font:
En/na Matthias Pfafferodt ha escrit:
The check is implemented like this since 2004 ...Considering patch #2943
(https://gna.org/patch/?2943), the index values should be compared:
bool same_pos(const struct tile
Follow-up Comment #8, patch #2943 (project freeciv):
ok, I do have now the one patch version as well as a splitted version (new
macros, changes to server/ai/clients/common+remove entries from struct tile).
Which version should be used to apply this?
Follow-up Comment #9, patch #2943 (project freeciv):
attached are both versions:
-tile.patch.bz2 - all in one file
tile.tar.bz2 - the file splitted into serveral patches:
- 0001-add-helper-function-to-get-x-y-nat_x-nat_y-using-the.patch
-
Follow-up Comment #1, bug #18612 (project freeciv):
I'm doing a lua rewrite (patch #2520). This means, that the lua code will
change.
If you have questions or need help just ask.
___
Reply to this item at:
http://gna.org/bugs/?18612
URL:
http://gna.org/patch/?2955
Summary: access settings from lua
Project: Freeciv
Submitted by: syntron
Submitted on: Mi 07 Sep 2011 23:39:45 CEST
Category: general
Priority: 5 - Normal
Follow-up Comment #3, bug #18612 (project freeciv):
So looks like I should switch to 2.4 branch for codebase rather
than trunk? Or which branch do you suggest.
At the moment trunk and the future 2.4 codebase are identical. If you only
want to work on the lua side you code select any status
Follow-up Comment #5, bug #18612 (project freeciv):
Looks like I'm going to write an event/handler adapter ...
Something like this exists. Do you know
http://freeciv.wikia.com/wiki/Events_Tutorial ? It uses signals to trigger lua
actions.
URL:
http://gna.org/bugs/?18619
Summary: fix city_repair_size
Project: Freeciv
Submitted by: syntron
Submitted on: Do 08 Sep 2011 20:31:30 CEST
Category: None
Severity: 3 - Normal
Follow-up Comment #1, patch #2955 (project freeciv):
fix possible crash due to NULL argument to setting_by_name()
(file #14061)
___
Additional Item Attachment:
File name: 0016-add-lua-function-server.setting.get.patch Size:3 KB
Follow-up Comment #10, patch #2943 (project freeciv):
updated patch - fix error at map generation
(file #14063)
___
Additional Item Attachment:
File name: 20110910-all-tile.patchSize:66 KB
Am Sonntag, 11. September 2011, 13:26:48 schrieb Michal Mazurek:
Calculating a turn on a 500mhz sparc64 with a maximum map, 30 players
takes 1 minute (memory usage jumps from 120MB to 200MB for a moment). I
think this may be caused by saving the game. Clients timeout etc - quite
a problem.
Follow-up Comment #1, patch #2956 (project freeciv):
Could you comment on for that the patch makes more sense? From the code I get
that it does check if the player is alive ...
___
Reply to this item at:
http://gna.org/patch/?2956
Am Sonntag, 11. September 2011, 10:42:24 schrieb Michal Mazurek:
On Sat, Sep 10, 2011 at 08:57:56AM +0200, Matthias Pfafferodt wrote:
Follow-up Comment #10, patch #2943 (project freeciv):
updated patch - fix error at map generation
Do you think the patch is stable enough for including
Follow-up Comment #3, patch #2956 (project freeciv):
OK, that makes sense! I have somethere a patch which was send to gna
implementing a player_iterate_alive macro ...
It is patch #1968 (you have written it). I will check it; perhaps it can be
used?
Am Sonntag, 11. September 2011, 17:21:52 schrieb Michal Mazurek:
On Sun, Sep 11, 2011 at 03:12:17PM +0200, Matthias Pfafferodt wrote:
Am Sonntag, 11. September 2011, 13:26:48 schrieb Michal Mazurek:
Calculating a turn on a 500mhz sparc64 with a maximum map, 30 players
takes 1 minute
Update of patch #1968 (project freeciv):
Status: Ready For Test = Done
Assigned to: cazfi = syntron
Open/Closed:Open = Closed
Update of patch #2956 (project freeciv):
Category:None = general
Planned Release: = 2.4.0
___
Follow-up Comment #6:
Update of patch #1968 (project freeciv):
Status:Done = Ready For Test
Open/Closed: Closed = Open
___
Follow-up Comment #4:
found one bug in
Follow-up Comment #21, bug #16429 (project freeciv):
Ok, this bug is really alive. I suspect it is related to the possibility of
tech loss (tech upkeep; techlost) as it happens only for the experimental
ruleset. (for reference trunk - svn20226).
To reproduce do an autogame with the following
URL:
http://gna.org/patch/?2957
Summary: function to test if a tile is a virtual tile
Project: Freeciv
Submitted by: syntron
Submitted on: So 11 Sep 2011 20:46:05 CEST
Category: general
Priority: 1 -
Update of patch #2943 (project freeciv):
Category:None = general
Status: Ready For Test = Done
Open/Closed:Open = Closed
Update of bug #18087 (project freeciv):
Status:None = Fixed
Open/Closed:Open = Closed
___
Follow-up Comment #23:
Final commits for
Update of bug #18081 (project freeciv):
Category:None = general
Status:None = Fixed
Open/Closed:Open = Closed
Update of bug #18092 (project freeciv):
Category:None = general
Status:None = Fixed
Assigned to:None = syntron
Open/Closed:
Update of bug #18619 (project freeciv):
Category:None = general
Status: Ready For Test = Fixed
Assigned to:None = syntron
Open/Closed:
Follow-up Comment #9, bug #18081 (project freeciv):
I did not consider cpp (qt client). The attached patch gets it to compile
again. I will apply it immediately.
(file #14074)
___
Additional Item Attachment:
File name:
Update of bug #18096 (project freeciv):
Status:None = Duplicate
Open/Closed:Open = Closed
___
Follow-up Comment #1:
fixed in bug #18264
Update of patch #2546 (project freeciv):
Status: Ready For Test = Done
Assigned to:None = syntron
Open/Closed:Open = Closed
Update of patch #2545 (project freeciv):
Status: Ready For Test = Done
Assigned to:None = syntron
Open/Closed:Open = Closed
Update of patch #2544 (project freeciv):
Status: Ready For Test = Done
Open/Closed:Open = Closed
___
Reply to this item at:
Update of bug #18537 (project freeciv):
Status: Ready For Test = Fixed
Assigned to:None = syntron
Open/Closed:Open = Closed
Am Sonntag, 11. September 2011, 10:45:34 schrieb Michal Mazurek:
Would it be possible to create a Longturn branch in the Freeciv
repository? Maintaining my diffs would be a lot easier if I could commit
them somewhere. A branch would be easier because I could update the code
with one svn
Am Montag, 12. September 2011, 10:11:51 schrieb Michal Mazurek:
On Sun, Sep 11, 2011 at 11:23:38PM +0200, Matthias Pfafferodt wrote:
Am Sonntag, 11. September 2011, 10:45:34 schrieb Michal Mazurek:
Would it be possible to create a Longturn branch in the Freeciv
repository? Maintaining my
Am Montag, 12. September 2011, 10:19:18 schrieb Marko Lindqvist:
On 12 September 2011 11:11, Michal Mazurek akf...@jasminek.net wrote:
The diff is here: http://longturn.org/crap/110911.23.diff
The reason i didn't create a local repository is that i want to be able
to update the code easily,
Hello Sveinung,
I do try at the moment to get a lua AI to work. It is not as easy as it may
look. More information, especially about the different types of AIs you can
find at the ticket: https://gna.org/patch/?2143
Some short comments from me:
[...]
2) Access to records of played games
Update of patch #1968 (project freeciv):
Status: Ready For Test = Done
Open/Closed:Open = Closed
___
Reply to this item at:
Follow-up Comment #1, bug #18652 (project freeciv):
could you please post the savegame?
___
Reply to this item at:
http://gna.org/bugs/?18652
___
Nachricht geschickt von/durch Gna!
Follow-up Comment #6, bug #18481 (project freeciv):
The attached patch adds a lua api check if debugging is activated. perhaps
this allows us to find the reason for the lua error ...
(file #14078)
___
Additional Item Attachment:
File
define to the code if debug=yes|checks is
used. Could you please test it? Perhaps there is an error in the lua code?
(see https://gna.org/bugs/index.php?18481)
LTex 2.3 has started, I invite you to join!
I will check the time ...
--
Matthias Pfafferodt - http://www.mapfa.de
URL:
http://gna.org/patch/?2959
Summary: add README.fcdb
Project: Freeciv
Submitted by: syntron
Submitted on: Mo 12 Sep 2011 21:28:45 CEST
Category: docs
Priority: 5 - Normal
Status: In
Follow-up Comment #1, patch #2959 (project freeciv):
and here the file simply as file
(file #14080)
___
Additional Item Attachment:
File name: README.fcdbSize:2 KB
Follow-up Comment #2, bug #18652 (project freeciv):
fix attached - after the game was started it is no longer a scenario ...
(file #14081)
___
Additional Item Attachment:
File name: 0017-fix-saving-of-scenarios-after-start.patch Size:0
Follow-up Comment #3, bug #18652 (project freeciv):
The last patch changes the bug to another place - not the load of the map
generator prints an error message ...
The new patch fixes it at the right place (I hope). The comment in the patch
reads (in start_game()):
+ /* This value defines if
Update of bug #18652 (project freeciv):
Planned Release: = 2.3.1,2.4.0
___
Reply to this item at:
http://gna.org/bugs/?18652
___
Nachricht
801 - 900 of 3081 matches
Mail list logo