Follow-up Comment #11, bug #17496 (project freeciv):
After latest patches my qt client compiles flawless
___
Reply to this item at:
http://gna.org/bugs/?17496
___
Message sent via/by Gna!
Follow-up Comment #2, bug #16989 (project freeciv):
Freeciv-2.2.4 working fine. You can close this bug.
___
Reply to this item at:
http://gna.org/bugs/?16989
___
Message sent via/by Gna!
Follow-up Comment #3, bug #17495 (project freeciv):
else part should be removed too
The else part does set the variable to 'no'. The following test do onyl check
if the variable is set to 'yes'. Thus, you get the same result if it exists
(='no') or if it is not defined. I think that setting
As I let the AI play against the AI (= autogames) I do not see this problem.
Perhaps it would help to add some debugging messages at the time the love for
the human player is decreased / increased to check _why_ it is done and also
why the patch for bug #16813 does not solve the problem?
Am
Follow-up Comment #12, bug #17496 (project freeciv):
It is working out of the box with the patch!
___
Reply to this item at:
http://gna.org/bugs/?17496
___
Nachricht geschickt von/durch
Update of patch #2185 (project freeciv):
Status: Ready For Test = Done
Open/Closed:Open = Closed
___
Reply to this item at:
Follow-up Comment #2, patch #2185 (project freeciv):
r18959 breaks the trunk build due to a typo. Patch attached; committing
immediately.
(file #11914)
___
Additional Item Attachment:
File name: trunk-muskovite.diff Size:0 KB
Follow-up Comment #3, patch #2185 (project freeciv):
Committed as r18960 (unfortunately it doesn't show up here as I mistakenly
said gna bug #2185 instead of gna patch #2185 in the change description).
___
Reply to this item at:
URL:
http://gna.org/bugs/?17504
Summary: Translator comments needed
Project: Freeciv
Submitted by: jcreus
Submitted on: Sun Jan 16 16:46:29 2011
Category: None
Severity: 2 - Minor
Update of bug #17504 (project freeciv):
Assigned to:None = jtn
Summary: Translator comments needed = Translator comments
needed for base help
___
Follow-up Comment #1:
Update of patch #2355 (project freeciv):
Status: In Progress = Ready For Test
___
Additional Item Attachment:
File name: trunk-S2_3-techupkeep-help.diff Size:1 KB
Update of patch #1893 (project freeciv):
Depends on: = patch #2355
___
Reply to this item at:
http://gna.org/patch/?1893
___
Message sent
Update of patch #1893 (project freeciv):
Status:Done = In Progress
Assigned to:None = jtn
Open/Closed: Closed = Open
Update of bug #17504 (project freeciv):
Status:None = Ready For Test
Planned Release: = 2.3.0-beta2
___
Additional Item Attachment:
File name:
Update of bug #17301 (project freeciv):
Status: Ready For Test = Need Info
Planned Release: 2.3.0 =
___
Reply to this item at:
Update of bug #17504 (project freeciv):
Status: Ready For Test = None
Planned Release: 2.3.0-beta2 =
___
Follow-up Comment #2:
I can see it now,
Follow-up Comment #3, bug #17334 (project freeciv):
Attached a slightly tweaked version of the patch whose strings are aligned
with the default ruleset as much as possible, to reduce the number of
translations required.
(file #11918)
___
Update of bug #16989 (project freeciv):
Status:None = Works For Me
Open/Closed:Open = Closed
___
Follow-up Comment #3:
Perhaps the fix for
URL:
http://gna.org/patch/?2356
Summary: Abyssinian nation
Project: Freeciv
Submitted by: dmarks
Submitted on: Sun 16 Jan 2011 11:07:05 AM PST
Category: rulesets
Priority: 5 - Normal
Follow-up Comment #2, patch #2349 (project freeciv):
While the language is referred to Chinookan, i thought the people were simply
referred to as Chinook?
___
Reply to this item at:
http://gna.org/patch/?2349
Update of patch #2349 (project freeciv):
Summary:Chinookan nation = Chinook nation
___
Follow-up Comment #3:
Good point - I believe you are correct. Corrected ruleset attached.
(file #11921)
URL:
http://gna.org/bugs/?17506
Summary: Server segfault in map_change_own_seen() when
loading savegame
Project: Freeciv
Submitted by: jtn
Submitted on: Sun Jan 16 20:40:41 2011
Category: None
Severity: 4
Follow-up Comment #3, bug #17459 (project freeciv):
See also bug #17506, which looks very similar (may be the same root cause).
___
Reply to this item at:
http://gna.org/bugs/?17459
___
Follow-up Comment #1, bug #17506 (project freeciv):
As for bug #17459, this savegame has borders=SEE_INSIDE.
___
Reply to this item at:
http://gna.org/bugs/?17506
___
Message sent via/by
Follow-up Comment #3, bug #17504 (project freeciv):
I guess Joan's point is true for most abbreviations used throughout the game
(HP = hit point, FP = firepower, A = attack, W = worker, etc.). They're handy
in many contexts and usually it's possible to guess the meaning, but maybe an
additions
Follow-up Comment #1, patch #2355 (project freeciv):
Thanks for doing the documentation work. The patch looks good!
___
Reply to this item at:
http://gna.org/patch/?2355
___
Nachricht
Follow-up Comment #14, bug #17496 (project freeciv):
Thanks for all the comments. This makes it easier to make additional fixes on
top of my patch. I don't think I'll use Matthias's patch. From the commet #13
alone (not reading the patch itself) it seems to make things wrong way:
- fix variable
URL:
http://gna.org/bugs/?17507
Summary: Oceanic terrain class name not localized
Project: Freeciv
Submitted by: jcreus
Submitted on: Sun Jan 16 21:15:35 2011
Category: None
Severity: 3 - Normal
in map_get_player_tile() to check if the map was initialised
(file #11924)
___
Additional Item Attachment:
File name: 20110116-fix-server-segfault-in-shared_vision_change_seen.patch
Size:1 KB
Follow-up Comment #1, bug #17465 (project freeciv):
could you check if patch #2344 does fixes this bug?
___
Reply to this item at:
http://gna.org/bugs/?17465
___
Nachricht geschickt
Follow-up Comment #3, bug #17506 (project freeciv):
I'll try your patch. In the meantime:
2.3.0-beta1 (and S2_3 at the branch point) don't show the problem. A git
bisect run on S2_3 suggests that r18812 (bug #17372) is when the segfaults
started. I haven't looked at the changes at all; does
Update of bug #17502 (project freeciv):
Status:None = Invalid
Assigned to:None = cazfi
Open/Closed:Open = Closed
Follow-up Comment #1, bug #17507 (project freeciv):
I think terrain class names are never shown to users, but are only used in
rulesets. So no translation needed.
___
Reply to this item at:
http://gna.org/bugs/?17507
Follow-up Comment #2, bug #17465 (project freeciv):
Seems so. I downgraded to r18889 and applied patch #2344 on top of that and
error message did not appear.
___
Reply to this item at:
http://gna.org/bugs/?17465
Follow-up Comment #1, patch #2311 (project freeciv):
This patch seems to cause autogames to differ. I'm not 100% sure autogames
are currently totally reproducible (there has been some problems with that in
past couple of years, and I don't know if reason has ever been found and
fixed). Anyway, I
Follow-up Comment #15, bug #17496 (project freeciv):
- fix variable name (GUI_qt_CPPFLAGS = GUI_qt_CXXFLAGS)
It should be CPPFLAGS, not CXXFLAGS. I assume proper fix is
needed elsewhere.
At the moment one (GUI_qt_CPPFLAGS) is used in qt-client.m4 while the other
(GUI_qt_CXXFLAGS) is used
Update of patch #2306 (project freeciv):
Status: Ready For Test = Done
Assigned to:None = cazfi
Open/Closed:Open = Closed
Follow-up Comment #16, bug #17496 (project freeciv):
At the moment one ...
This is trunk with file #11910
___
Reply to this item at:
http://gna.org/bugs/?17496
___
Nachricht geschickt
Follow-up Comment #4, bug #17506 (project freeciv):
2.3.0-beta1 (and S2_3 at the branch point) don't show the
problem. A git bisect run on S2_3 suggests that r18812
(bug #17372) is when the segfaults started. I haven't looked at
the changes at all; does that sound plausible?
Sounds possible
Follow-up Comment #17, bug #17496 (project freeciv):
GUI_qt_CXXFLAGS) is used in the makefile and configure.ac - so I
selected the wrong point to change it ...
There is actually long (probably from the very beginning of Freeciv project
15 years ago) standing bug in how CPPFLAGS, CFLAGS (and
Follow-up Comment #5, bug #17506 (project freeciv):
Sounds possible (not tested).
Some background: at the moment there are two types of savegames which use a
different order of loading the data. A previous patch (bug #17158) fixed new
savegames but broke loading of old savegames. In bug
Follow-up Comment #2, bug #17507 (project freeciv):
I've seen it in client/helpdata.c:532, the help for bases mentioned in bug
#17504. The string is Requires %s terrain class on the tile.n. For the Buoy,
this terrain class is Oceanic.
There's also Requires %s terrain class on an adjacent tile.n
URL:
http://gna.org/bugs/?17508
Summary: Loading savegames for Britain/Europe scenarios
gives many errors and server segfault
Project: Freeciv
Submitted by: jtn
Submitted on: Sun Jan 16 22:43:19 2011
Category: None
Update of bug #17459 (project freeciv):
Status:None = Duplicate
Open/Closed:Open = Closed
___
Follow-up Comment #5:
I could reproduce
Follow-up Comment #6, bug #17506 (project freeciv):
I confirm that your patch fixes the segfault for me, thanks.
___
Reply to this item at:
http://gna.org/bugs/?17506
___
Message sent
Update of bug #17507 (project freeciv):
Status:None = In Progress
Assigned to:None = jtn
___
Reply to this item at:
URL:
http://gna.org/bugs/?17509
Summary: Qualify string Land for i18n (unit vs terrain
class)
Project: Freeciv
Submitted by: jtn
Submitted on: Sun Jan 16 23:54:48 2011
Category: None
Severity: 2 - Minor
Update of bug #17507 (project freeciv):
Status: In Progress = Ready For Test
Planned Release: = 2.3.0-beta2
___
Follow-up Comment #3:
Looks like
Additional Item Attachment, bug #17509 (project freeciv):
File name: trunk-S2_3-land-unitclass-qual.diff Size:9 KB
File name: S2_2-land-unitclass-qual.diff Size:5 KB
___
Reply to this item at:
http://gna.org/bugs/?17509
Follow-up Comment #1, bug #17509 (project freeciv):
That's just fine, thanks.
___
Reply to this item at:
http://gna.org/bugs/?17509
___
Message sent via/by Gna!
http://gna.org/
Follow-up Comment #4, bug #17504 (project freeciv):
Isn't MP for movement points a bit ambiguous for users,
though? This is clear in the translator comments, but users
don't see that. This is what we usually call worker-turns,
isn't it?
It is the settler-turns or worker-turns concept, but
Update of bug #17504 (project freeciv):
Status:None = Ready For Test
Planned Release: = 2.3.0-beta2
___
Reply to this item at:
Follow-up Comment #5, bug #17504 (project freeciv):
Re: abbreviations. Tooltip seems even better. Anything that would expand the
abbreviation to the user, I guess.
___
Reply to this item at:
http://gna.org/bugs/?17504
URL:
http://gna.org/bugs/?17510
Summary: Invalid lightning bolt expression in the help
system
Project: Freeciv
Submitted by: hubkow
Submitted on: Mon Jan 17 01:48:36 2011
Category: None
Severity: 2 -
54 matches
Mail list logo