[Freeciv-Dev] [patch #2776] Hunnic nation
Follow-up Comment #2, patch #2776 (project freeciv): I thought the most convincing is the hypothesis about the Turkish origin of the Huns. A little later the Bulgarian tribes are considered descendants of the Huns. In addition, Chuvash language, having supposedly come from the Bulgarian dialects, has a very primitive characteristics compared to other Turkic languages. Hence, it is believed that he could come from the language of the Huns, because the next wave of expansion of Turkish use the more modern languages. However, very possible that the original language of the Huns was a different language, perhaps Yeniseian, but they could be Turkization still somewhere in Asia. For the names of archaeological sites related to the Huns, I'm still in the course of their search. ___ Reply to this item at: http://gna.org/patch/?2776 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #18323] Governments menu grows without bound, making it difficult/impossible to change government
Update of bug #18323 (project freeciv): Status: Ready For Test = Fixed Open/Closed:Open = Closed Summary: Cannot have any government but despots = Governments menu grows without bound, making it difficult/impossible to change government ___ Reply to this item at: http://gna.org/bugs/?18323 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #2776] Hunnic nation
Follow-up Comment #3, patch #2776 (project freeciv): There's no reason why Chuvash couldn't descend from any other old Turkish language. It's well established that there were Turkic peoples in the region less than a century after the Huns, so that would explain for the supposedly archaic characteristics of Chuvash. From what I know about the claim the Huns are Turkic is that it is mostly based on etymologies for Hun personal names, and a lot of those claims are rather spurious. I mean, basing Atilla on the Turkish word for 'father' as is often claimed is rather silly, the word for father is something like atta or papa in every human language, for the same reason mother is mama or nana, it's just the first sounds babies make. Furthermore, one can just as easily make up Uralic or Iranian etymologies for Hun names. From what I remember from my medieval history classes, ethnic designations in this era are notoriously fuzzy anyway. To begin with, most barbaric empires or kingdoms were political rather than ethnic anyway, so it's likely the Huns were multi-ethnic. Plus several peoples using the same name doesn't mean they are the same people; many Roman and Byzantine authors were too lazy to distinguish between ethnic groups so they just called everything that came out of the steppes 'Scythians' or 'Huns' or whatever. Also it often happened that a nation adopted the name of an ancient people they don't have any connection with for prestige reasons. ___ Reply to this item at: http://gna.org/patch/?2776 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #2780] Xiongnu nation
Update of patch #2780 (project freeciv): Status:None = In Progress Assigned to:None = mixcoatl ___ Reply to this item at: http://gna.org/patch/?2780 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #2773] Backtrace log callback for log levels = error
Follow-up Comment #1, patch #2773 (project freeciv): - Configure check for execinfo.h - Fixed compiler warnings when backtrace functionality not available (file #13508) ___ Additional Item Attachment: File name: RuntimeBacktrace_2773-2.diff Size:7 KB ___ Reply to this item at: http://gna.org/patch/?2773 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #2781] Anhaltian nation
URL: http://gna.org/patch/?2781 Summary: Anhaltian nation Project: Freeciv Submitted by: mixcoatl Submitted on: Mon 11 Jul 2011 03:48:16 PM GMT Category: rulesets Priority: 5 - Normal Status: In Progress Privacy: Public Assigned to: mixcoatl Originator Email: Open/Closed: Open Discussion Lock: Any Planned Release: ___ Details: Anhalt flag by myself ___ File Attachments: --- Date: Mon 11 Jul 2011 03:48:16 PM GMT Name: anhaltian.ruleset Size: 2kB By: mixcoatl http://gna.org/patch/download.php?file_id=13509 --- Date: Mon 11 Jul 2011 03:48:16 PM GMT Name: anhalt.svg Size: 262B By: mixcoatl http://gna.org/patch/download.php?file_id=13510 ___ Reply to this item at: http://gna.org/patch/?2781 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #2782] Tahitian (French Polynesian) nation
URL: http://gna.org/patch/?2782 Summary: Tahitian (French Polynesian) nation Project: Freeciv Submitted by: mixcoatl Submitted on: Mon 11 Jul 2011 03:51:41 PM GMT Category: rulesets Priority: 5 - Normal Status: In Progress Privacy: Public Assigned to: mixcoatl Originator Email: Open/Closed: Open Discussion Lock: Any Planned Release: ___ Details: Tahiti/French Polynesia Flag adapted from Tobias Jakobs, Open Clip Art: http://www.openclipart.org/detail/17757/flag-of-french-polynesia-by-tobias ___ File Attachments: --- Date: Mon 11 Jul 2011 03:51:41 PM GMT Name: french_polynesia.svg Size: 18kB By: mixcoatl http://gna.org/patch/download.php?file_id=13511 --- Date: Mon 11 Jul 2011 03:51:41 PM GMT Name: tahiti.ruleset Size: 2kB By: mixcoatl http://gna.org/patch/download.php?file_id=13512 ___ Reply to this item at: http://gna.org/patch/?2782 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #2783] African nation
URL: http://gna.org/patch/?2783 Summary: African nation Project: Freeciv Submitted by: mixcoatl Submitted on: Mon 11 Jul 2011 03:56:00 PM GMT Category: rulesets Priority: 5 - Normal Status: In Progress Privacy: Public Assigned to: mixcoatl Originator Email: Open/Closed: Open Discussion Lock: Any Planned Release: ___ Details: Africa The African Union flag is poorly visible in a small size, and I couldn't find a svg version of it under a free licence. Another option would be the Pan-African flag but as far as I know that one is more associated with the African diaspora then with Africa itself. So I made a flag with a silhouette of Africa on a green background, with the shape of Africa taken from http://commons.wikimedia.org/wiki/File:Continents.svg ___ File Attachments: --- Date: Mon 11 Jul 2011 03:56:00 PM GMT Name: africa.ruleset Size: 2kB By: mixcoatl http://gna.org/patch/download.php?file_id=13513 --- Date: Mon 11 Jul 2011 03:56:00 PM GMT Name: africa.svg Size: 15kB By: mixcoatl http://gna.org/patch/download.php?file_id=13514 ___ Reply to this item at: http://gna.org/patch/?2783 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #2738] Client - metaserver communication thread
Update of patch #2738 (project freeciv): Planned Release: 2.4.0 = 2.5.0 ___ Follow-up Comment #3: This cannot work with dummy thread implementation. There is no other need for real thread implementation at the moment, and I don't want make them hard requirement for 2.4 just for this. This is not yet so important. 2.4 is first version where threads are used at all, so let's have disabling threads as a workaround for any threads related problems users encounter. ___ Reply to this item at: http://gna.org/patch/?2738 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #2380] Client - metaserver communication with curl
Update of patch #2380 (project freeciv): Planned Release: 2.4.0 = 2.5.0 ___ Reply to this item at: http://gna.org/patch/?2380 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #1845] [metaticket] Use curl for fetching files and pages over network
Update of patch #1845 (project freeciv): Planned Release: 2.4.0 = 2.5.0 ___ Reply to this item at: http://gna.org/patch/?1845 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #2784] Somaliland nation
URL: http://gna.org/patch/?2784 Summary: Somaliland nation Project: Freeciv Submitted by: mixcoatl Submitted on: Mon 11 Jul 2011 04:21:53 PM GMT Category: rulesets Priority: 5 - Normal Status: In Progress Privacy: Public Assigned to: mixcoatl Originator Email: Open/Closed: Open Discussion Lock: Any Planned Release: ___ Details: Somaliland Flag by myself, based on Saudi flag ___ File Attachments: --- Date: Mon 11 Jul 2011 04:21:53 PM GMT Name: somaliland.ruleset Size: 2kB By: mixcoatl http://gna.org/patch/download.php?file_id=13515 --- Date: Mon 11 Jul 2011 04:21:53 PM GMT Name: somaliland.svg Size: 30kB By: mixcoatl http://gna.org/patch/download.php?file_id=13516 ___ Reply to this item at: http://gna.org/patch/?2784 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #18328] Ruleset server settings are case-sensitive
URL: http://gna.org/bugs/?18328 Summary: Ruleset server settings are case-sensitive Project: Freeciv Submitted by: someswede Submitted on: Mon Jul 11 16:46:39 2011 Category: rulesets Severity: 3 - Normal Priority: 5 - Normal Status: None Assigned to: None Originator Email: Open/Closed: Open Release: SVN 19927 Discussion Lock: Any Operating System: None Planned Release: ___ Details: When setting server options in game.ruleset the values are case-sensitive; for instance, having this gives an error: mapsize, player, FALSE Can't read value for setting 'mapsize': In secfile_lookup_enum_data() [registry.c:2499]: secfile 'data/nomodern/game.ruleset' in section 'settings': Entry set0.value: no match for player. whereas this works fine: mapsize, PLAYER, FALSE However, when using the /set server command, it is not case-sensitive: mikaels: 'set mapsize FULLSIZE' Console: 'mapsize' has been set to Number of tiles (FULLSIZE). mikaels: 'set mapsize PlAyeR' Console: 'mapsize' has been set to Tiles per player (PLAYER). mikaels: 'set mapsize fUllsIzE' Console: 'mapsize' has been set to Number of tiles (FULLSIZE). mikaels: 'set mapsize plaYER' Console: 'mapsize' has been set to Tiles per player (PLAYER). ___ Reply to this item at: http://gna.org/bugs/?18328 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #2754] improved Roman nation
Update of patch #2754 (project freeciv): Status: Ready For Test = Done Open/Closed:Open = Closed ___ Reply to this item at: http://gna.org/patch/?2754 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #18328] Ruleset server settings are case-sensitive
Follow-up Comment #1, bug #18328 (project freeciv): something, DISABLED, FALSE does not work either, instead one has to use something, 0, FALSE even if the server command is /set something disabled I'm not sure if this should be considered a bug or just an annoying inconsistency. ___ Reply to this item at: http://gna.org/bugs/?18328 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #18329] Packet receiving code assumes sizeof(enum packet_type) == sizeof(int)
URL: http://gna.org/bugs/?18329 Summary: Packet receiving code assumes sizeof(enum packet_type) == sizeof(int) Project: Freeciv Submitted by: cazfi Submitted on: Mon 11 Jul 2011 08:45:29 PM EEST Category: general Severity: 3 - Normal Priority: 5 - Normal Status: Ready For Test Assigned to: None Originator Email: Open/Closed: Open Release: Discussion Lock: Any Operating System: None Planned Release: 2.3.0, 2.4.0 ___ Details: Packet receiving code depends on sizeof(enum packet_type) == sizeof(int). Fix attached. ___ File Attachments: --- Date: Mon 11 Jul 2011 08:45:29 PM EEST Name: NetworkEnum.diff Size: 792B By: cazfi http://gna.org/bugs/download.php?file_id=13517 --- Date: Mon 11 Jul 2011 08:45:29 PM EEST Name: NetworkEnum-S2_2.diff Size: 794B By: cazfi http://gna.org/bugs/download.php?file_id=13518 ___ Reply to this item at: http://gna.org/bugs/?18329 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #18329] Packet receiving code assumes sizeof(enum packet_type) == sizeof(int)
Update of bug #18329 (project freeciv): Planned Release:2.3.0, 2.4.0 = 2.2.7, 2.3.0, 2.4.0 ___ Reply to this item at: http://gna.org/bugs/?18329 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #18331] Available themes are not listed alphabetically
URL: http://gna.org/bugs/?18331 Summary: Available themes are not listed alphabetically Project: Freeciv Submitted by: someswede Submitted on: Mon Jul 11 18:11:57 2011 Category: client-gtk-2.0 Severity: 2 - Minor Priority: 5 - Normal Status: None Assigned to: None Originator Email: Open/Closed: Open Release: 19927 Discussion Lock: Any Operating System: GNU/Linux Planned Release: ___ Details: Menubar Game Options Local client Graphics: Theme has a drop-down list; the available themes are not listed alphabetically. http://i.imgur.com/HsKS0.png ___ Reply to this item at: http://gna.org/bugs/?18331 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #2786] Use gtk2 instead of plain gtk as example value for --enable-client
URL: http://gna.org/patch/?2786 Summary: Use gtk2 instead of plain gtk as example value for --enable-client Project: Freeciv Submitted by: cazfi Submitted on: Mon 11 Jul 2011 09:59:43 PM EEST Category: docs Priority: 5 - Normal Status: Ready For Test Privacy: Public Assigned to: None Originator Email: Open/Closed: Open Discussion Lock: Any Planned Release: 2.3.0, 2.4.0 ___ Details: Attached patch changes --enable-client documentation to refer to value gtk2 instead of gtk ___ File Attachments: --- Date: Mon 11 Jul 2011 09:59:43 PM EEST Name: GtkIsGtk2.diff Size: 1kB By: cazfi http://gna.org/patch/download.php?file_id=13521 --- Date: Mon 11 Jul 2011 09:59:43 PM EEST Name: GtkIsGtk2-S2_3.diff Size: 1kB By: cazfi http://gna.org/patch/download.php?file_id=13522 ___ Reply to this item at: http://gna.org/patch/?2786 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #2787] Windows icon for freeciv-modpack.exe
URL: http://gna.org/patch/?2787 Summary: Windows icon for freeciv-modpack.exe Project: Freeciv Submitted by: cazfi Submitted on: Mon 11 Jul 2011 11:03:39 PM EEST Category: module installer Priority: 5 - Normal Status: In Progress Privacy: Public Assigned to: None Originator Email: Open/Closed: Open Discussion Lock: Any Planned Release: 2.3.0, 2.4.0 ___ Details: freeciv-modpack.exe should have icon in Windows. Attached untested patch adds that, except that actual gfx is missing. ___ File Attachments: --- Date: Mon 11 Jul 2011 11:03:39 PM EEST Name: MPIcon.diff Size: 2kB By: cazfi http://gna.org/patch/download.php?file_id=13523 ___ Reply to this item at: http://gna.org/patch/?2787 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #2788] Automatically prepend fc_ai_ to module names given for -L
URL: http://gna.org/patch/?2788 Summary: Automatically prepend fc_ai_ to module names given for -L Project: Freeciv Submitted by: cazfi Submitted on: Mon 11 Jul 2011 11:41:50 PM EEST Category: ai Priority: 5 - Normal Status: Ready For Test Privacy: Public Assigned to: None Originator Email: Open/Closed: Open Discussion Lock: Any Planned Release: 2.4.0 ___ Details: This patch makes freeciv to automatically add prefix fc_ai_ to ai module names given for commandline option -L so one can (and should) omit it. ___ File Attachments: --- Date: Mon 11 Jul 2011 11:41:50 PM EEST Name: AutoModPrefix.diff Size: 2kB By: cazfi http://gna.org/patch/download.php?file_id=13524 ___ Reply to this item at: http://gna.org/patch/?2788 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #18333] assertion 'ai-phase_initilized' failed
URL: http://gna.org/bugs/?18333 Summary: assertion 'ai-phase_initilized' failed Project: Freeciv Submitted by: someswede Submitted on: Mon Jul 11 21:09:29 2011 Category: None Severity: 3 - Normal Priority: 5 - Normal Status: None Assigned to: None Originator Email: Open/Closed: Open Release: 19927 Discussion Lock: Any Operating System: GNU/Linux Planned Release: ___ Details: I'm guessing this is related to bugs 17728 and 18095 (for the 2.3.0 betas), which are both marked as closed. Upon turn 5 in my first game with this revision, I got 3 of these (with 6 AI players): [22:54:22] in ai_plr_data_get() [aidata.c::310]: assertion 'ai-phase_initialized' failed. [22:54:22] Please report this message at http://gna.org/projects/freeciv/ ___ Reply to this item at: http://gna.org/bugs/?18333 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #18333] assertion 'ai-phase_initialized' failed
Update of bug #18333 (project freeciv): Release: 19927 = trunk r19927 Planned Release: = 2.4.0 Summary: assertion 'ai-phase_initilized' failed = assertion 'ai-phase_initialized' failed ___ Follow-up Comment #1: It sounds like you're talking about trunk (as there's no file/function named like that in S2_3). Bug #18095 was closed by suppressing the assertion failure on S2_3 only (trunk was unchanged). The failure there was not reproducible on trunk. I had a quick poke at this revision and didn't see anything of the kind after 29 turns with the default ruleset and aifill 7. Can you provide a savegame from which it can be reproduced? Failing that, can you get a backtrace with -F? ___ Reply to this item at: http://gna.org/bugs/?18333 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #2761] Add missing stdarg.h includes
Update of patch #2761 (project freeciv): Status: Ready For Test = Done Assigned to:None = cazfi Open/Closed:Open = Closed ___ Reply to this item at: http://gna.org/patch/?2761 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #18087] server segfault on a map with 1.3M tiles
Follow-up Comment #14, bug #18087 (project freeciv): Oh there is so much stack abuse in freeciv. ...question: is allocating huge arrays on the stack instead of allocating the same arrays on the heap (in the absence of recursion etc) really an abuse? Having read around a bit more, I'm now convinced it's not a good idea. Wikipedia http://en.wikipedia.org/wiki/Stack-based_memory_allocation suggests that a thread's stack size can be as small as a few dozen kilobytes, and more specifically, the Windows documentation http://msdn.microsoft.com/en-us/library/ms686774%28v=vs.85%29.aspx says The default stack reservation size used by the linker is 1 MB. What I think I'd missed with my it's all memory argument is that when memory runs out, malloc() can return NULL (from which the application can theoretically recover), whereas allocating memory on the stack can't fail, so the OS has little option but to kill the entire app. It looks like as a result of that, some (all?) OSes (such as Windows, see previous article) set aside memory for app stacks, so that apps aren't killed for the crime of calling a function when things get tight. Thus stack memory is more reliable than heap memory, thus more precious, thus OSes place limits on it, thus it's not a good idea to try to allocate arbitrarily large chunks of it. Sorry if that was obvious to everyone else. Conclusion is, we shouldn't allocate O(xsize*ysize) memory on the stack. IMO we should take the stack-abuse changes on S2_3, as well as the assign_continent_flood() change that we know we need. I've had a look and they look basically fine to me; more in future comments. ___ Reply to this item at: http://gna.org/bugs/?18087 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #2789] 16bit packet type
URL: http://gna.org/patch/?2789 Summary: 16bit packet type Project: Freeciv Submitted by: cazfi Submitted on: Tue 12 Jul 2011 01:16:20 AM EEST Category: general Priority: 5 - Normal Status: Ready For Test Privacy: Public Assigned to: None Originator Email: Open/Closed: Open Discussion Lock: Any Planned Release: 2.5.0 ___ Details: We discussed already when editor was in development that we may eventually run out of unique packet numbers. We still have some space left, and there's many ways to reuse packet numbers, but just to avoid any uglier solutions I'd like to increase packet number space before this becomes a real problem. Attached patch makes packet type 16 bit value (currently 8 bit). This is still not very much tested, and anyway I'd like to commit it from the beginning of the new cycle (soon after S2_4 branching) rather than late in cycle that has used 8 bit types so far. ___ File Attachments: --- Date: Tue 12 Jul 2011 01:16:20 AM EEST Name: 16BitPacketType.diff Size: 5kB By: cazfi http://gna.org/patch/download.php?file_id=13525 ___ Reply to this item at: http://gna.org/patch/?2789 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #2790] Fix --enable-fcdb help default value
URL: http://gna.org/patch/?2790 Summary: Fix --enable-fcdb help default value Project: Freeciv Submitted by: cazfi Submitted on: Tue 12 Jul 2011 01:23:05 AM EEST Category: general Priority: 5 - Normal Status: Ready For Test Privacy: Public Assigned to: None Originator Email: Open/Closed: Open Discussion Lock: Any Planned Release: 2.4.0 ___ Details: Help for --enable-fcdb claims that default is all when implementation is actually no. In addition to correcting the help, attached patch: - Uses same handling for default value than explicitly set no - Adds check for illegal value ___ File Attachments: --- Date: Tue 12 Jul 2011 01:23:05 AM EEST Name: FcdbConfDocFix.diff Size: 825B By: cazfi http://gna.org/patch/download.php?file_id=13526 ___ Reply to this item at: http://gna.org/patch/?2790 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #2791] CPUHog
URL: http://gna.org/patch/?2791 Summary: CPUHog Project: Freeciv Submitted by: cazfi Submitted on: Tue 12 Jul 2011 01:30:06 AM EEST Category: general Priority: 5 - Normal Status: Ready For Test Privacy: Public Assigned to: None Originator Email: Open/Closed: Open Discussion Lock: Any Planned Release: 2.4.0 ___ Details: To save some CPU-cycles AI does not always calculate everything but uses good enough existing values from different caches etc. I often find AI's bad performance much worse problem than running out of CPU-cycles. This patch adds some stuff to implement CPU-hog AI that hopefully performs a bit better at the cost of using much more computing resources. In this initial version only max number of cm-iterations is increased for CPU-hogs. This patch makes it only easier to make CPU-hog version of freeciv. One cannot change ai behavior runtime, but it still requires code change. CPU-hog framework just makes that required code change one-liner. See comments in player.h for full rationale. ___ File Attachments: --- Date: Tue 12 Jul 2011 01:30:07 AM EEST Name: CPUHog.diff Size: 2kB By: cazfi http://gna.org/patch/download.php?file_id=13527 ___ Reply to this item at: http://gna.org/patch/?2791 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #2792] Devel Stable URLs for freeciv-modpack
URL: http://gna.org/patch/?2792 Summary: Devel Stable URLs for freeciv-modpack Project: Freeciv Submitted by: cazfi Submitted on: Tue 12 Jul 2011 01:45:17 AM EEST Category: module installer Priority: 5 - Normal Status: Ready For Test Privacy: Public Assigned to: None Originator Email: Open/Closed: Open Discussion Lock: Any Planned Release: 2.4.0 ___ Details: Attached patch makes freeciv-modpack compilation to automatically to switch default URLs to gna.org ones for stable releases. Development versions use cazfi.net ones. ___ File Attachments: --- Date: Tue 12 Jul 2011 01:45:17 AM EEST Name: MPDevelStableURLs.diff Size: 898B By: cazfi http://gna.org/patch/download.php?file_id=13528 ___ Reply to this item at: http://gna.org/patch/?2792 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #2427] [metaticket] Add missing function headers
Follow-up Comment #3, patch #2427 (project freeciv): gui-stub seems to be ok, and thus also gui-qt that started as copy of stub. gui-gtk2 was finished before it was copied as base for gui-gtk3. I don't think we need to bother with gui-ftwl and gui-win32 en route to removal until someone starts saving them. This leaves gui-sdl and gui-xaw, as well as server/scripting. ___ Reply to this item at: http://gna.org/patch/?2427 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #18087] server segfault on a map with 1.3M tiles
Follow-up Comment #15, bug #18087 (project freeciv): Attached my version of file #12960, split into three separate patches and tweaked a little. The three patches: * *trunk-S2_3-hugemap-stack-overflow.diff*: The stack overflow fixes. This is what I propose we apply to S2_3 pre RC1, to deal with bug #17962. _I will do so if there are no objections in 36h._ ** I've reviewed this to check that no memory leaks came in with the introduction of heap allocation due to returning without freeing. I only noticed one: in sg_load_map_known()), SAVE_MAP_CHAR() - sg_failure_ret() - sg_check_ret() can return prematurely. This is a leaky pattern and should be dealt with (raise a ticket), but it's an error condition so I think we can live without a fix for RC1. ** I've only reviewed the existing changes for correctness; I haven't looked to see if there are any map-sized stack allocations that have been missed. ** I haven't reviewed the new flooding algorithm in detail. It looks vaguely plausible. ** As a sanity check, I've played a single-player game with this patch (S2_3) for a few tens of turns without the world ending. No autogame testing yet. * *trunk-S2_3-hugemap-colatitude.diff*: This messes with colatitude stuff I don't fully understand, so it seems riskier to me. Unless someone thinks otherwise, I suggest we leave it out for 2.3.0-RC1, since it's not necessary to fix the stack issues. Could add it back to a later 2.3.x. ** This bit of the patch is Matthias; it came in between file #12942 (comment #4) and file #12947 (comment #5). ** Matthias: do you think this patch is useful/necessary for the map size limits we have on S2_3? * *trunk-S2_3-hugemap-biggermaps.diff*: This actually increases the maximum map sizes. I suggest we leave this for trunk only. ** Aside: I think the correct way to express the relationship between MAP_MAX_SIZE and MAX_DBV_LENGTH is to publish the latter in bitvector.h and have a static (compile-time) assert in the map code. But we don't seem to have a static assert mechanism yet. I may look into it. My tweaks were: * Remove memset()s that were redundant due to use of fc_calloc() * Change free() to FC_FREE() throughout, for form's sake * Add 'const' to constant arrays in smooth_int_map() * Rationalise away 'ret' in rand_map_pos_filtered() (file #13529, file #13530, file #13531) ___ Additional Item Attachment: File name: trunk-S2_3-hugemap-stack-overflow.diff Size:16 KB File name: trunk-S2_3-hugemap-colatitude.diff Size:1 KB File name: trunk-S2_3-hugemap-biggermaps.diff Size:2 KB ___ Reply to this item at: http://gna.org/bugs/?18087 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #17962] Inability to generate/load huge maps on Windows
Follow-up Comment #14, bug #17962 (project freeciv): I intend to apply file #13531 (from bug #18087) to try to deal with this before 2.3.0-RC1 is released. Since there isn't much time before RC1, pre-commit testing would be appreciated (especially by Windows users). ___ Reply to this item at: http://gna.org/bugs/?17962 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #17962] Inability to generate/load huge maps on Windows
Update of bug #17962 (project freeciv): Status:None = Ready For Test Assigned to:None = jtn ___ Reply to this item at: http://gna.org/bugs/?17962 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #18283] Comments about map size limits in map.h do not match code
Update of bug #18283 (project freeciv): Status: Need Info = Duplicate Open/Closed:Open = Closed ___ Reply to this item at: http://gna.org/bugs/?18283 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #18334] few dozen typos in 2.2.5
URL: http://gna.org/bugs/?18334 Summary: few dozen typos in 2.2.5 Project: Freeciv Submitted by: kgoetz Submitted on: Tue 12 Jul 2011 08:59:42 AM CST Category: None Severity: 2 - Minor Priority: 5 - Normal Status: None Assigned to: None Originator Email: Open/Closed: Open Release: Discussion Lock: Any Operating System: GNU/Linux Planned Release: ___ Details: Hi guys, While updating the Debian packaging i came across a number of typos in the game - it appears to build and run still, so could you please test and apply? thanks, kk ___ File Attachments: --- Date: Tue 12 Jul 2011 08:59:42 AM CST Name: trivial-typos.diff Size: 6kB By: kgoetz http://gna.org/bugs/download.php?file_id=13532 ___ Reply to this item at: http://gna.org/bugs/?18334 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #18335] please change the colour of the research tab.
URL: http://gna.org/bugs/?18335 Summary: please change the colour of the research tab. Project: Freeciv Submitted by: kgoetz Submitted on: Tue 12 Jul 2011 09:18:09 AM CST Category: client-gtk-2.0 Severity: 3 - Normal Priority: 5 - Normal Status: None Assigned to: None Originator Email: Open/Closed: Open Release: 2.2.5 Discussion Lock: Any Operating System: GNU/Linux Planned Release: ___ Details: Hi, Forwarding this from the debian bts, where i filed it origonally. When a technology is successfully researched, freeciv turns the 'Research' tab red. It would be good if it could use blue (like it does for the messages box) as red is quite hard to see on the background window. ___ Reply to this item at: http://gna.org/bugs/?18335 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #2778] City map iterating macros that don't set _index
Update of patch #2778 (project freeciv): Status: Ready For Test = Done Assigned to:None = cazfi Open/Closed:Open = Closed ___ Reply to this item at: http://gna.org/patch/?2778 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #2793] Update comments to configure.ac
URL: http://gna.org/patch/?2793 Summary: Update comments to configure.ac Project: Freeciv Submitted by: cazfi Submitted on: Tue 12 Jul 2011 07:32:45 AM EEST Category: bootstrap Priority: 5 - Normal Status: Ready For Test Privacy: Public Assigned to: None Originator Email: Open/Closed: Open Discussion Lock: Any Planned Release: 2.4.0 ___ Details: This patch adds some comments to configure.ac to help with cleanup when our minimum tool requirements are increased. ___ File Attachments: --- Date: Tue 12 Jul 2011 07:32:45 AM EEST Name: CacComments.diff Size: 715B By: cazfi http://gna.org/patch/download.php?file_id=13533 ___ Reply to this item at: http://gna.org/patch/?2793 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev