Follow-up Comment #1, patch #3561 (project freeciv):
A screenshot please.
I don't see that on trident, cimpletoon nor amplio2.
___
Reply to this item at:
http://gna.org/patch/?3561
___
Follow-up Comment #1, patch #3564 (project freeciv):
Oops. I had that correct on S2_4 - not quite sure why wasn't that
cherry-picked there correctly.
Anyway, it seems to be just a matter of a single line.
(file #16764)
___
Additional
Follow-up Comment #2, bug #20291 (project freeciv):
gcc (Debian 4.7.2-4) 4.7.2
GNU ld (GNU Binutils for Debian) 2.22
___
Reply to this item at:
http://gna.org/bugs/?20291
___
Message sent
Follow-up Comment #2, patch #3560 (project freeciv):
As it's a standard button now, it follows the theme.
Uhm. Maybe the whole point of not having it as standard button earlier was
that it's better with glaring indicator of governor state. With this patch it
has just the text.
Can you list
Update of patch #3561 (project freeciv):
Status:None = Ready For Test
___
Follow-up Comment #2:
Turned out that it's not new with the 27/60 patch, but those areas are high in
current TRUNK
Follow-up Comment #1, patch #3559 (project freeciv):
- Rebased on top of patch #3562 to make it work with multiple AI types.
(file #16766)
___
Additional Item Attachment:
File name: Firepower1-2.patch Size:18 KB
Update of patch #3564 (project freeciv):
Status:None = Ready For Test
___
Reply to this item at:
http://gna.org/patch/?3564
___
Message sent
URL:
http://gna.org/patch/?3565
Summary: Use grids in citydlg overview page (patch 29/60)
Project: Freeciv
Submitted by: cazfi
Submitted on: Sat 17 Nov 2012 02:29:40 AM EET
Category: client-gtk-3.0
Follow-up Comment #55, patch #3469 (project freeciv):
29/60 raised as patch #3565 as it depends on patch #3561 and cannot be
committed yet. (I want to go though patches in this ticket in order so every
one is certainly handled)
___
Reply
URL:
http://gna.org/patch/?3566
Summary: Use grids for worklists (patch 31/60)
Project: Freeciv
Submitted by: cazfi
Submitted on: Sat 17 Nov 2012 02:40:52 AM EET
Category: None
Priority: 5 - Normal
Follow-up Comment #57, patch #3469 (project freeciv):
31/60 raised as patch #3566 for bug found in testing
___
Reply to this item at:
http://gna.org/patch/?3469
___
Message sent via/by Gna!
Update of bug #20221 (project freeciv):
Status:None = Ready For Test
___
Follow-up Comment #1:
Patches
(file #16767, file #16768)
___
URL:
http://gna.org/bugs/?20300
Summary: Units lost in paradrop do not count to
score.units_lost
Project: Freeciv
Submitted by: cazfi
Submitted on: Sat 17 Nov 2012 03:41:17 AM EET
Category: general
Follow-up Comment #1, patch #3566 (project freeciv):
Are you sure this patch plays a role here ? It shouldn't affect anything but
packing (and nothing in its content suggests it could).
While at the first I seem to have reproduced the problem, I've noticed a
warning in the terminal:
URL:
http://gna.org/bugs/?20301
Summary: Not all lost units counted in score.units_lost
Project: Freeciv
Submitted by: cazfi
Submitted on: Sat 17 Nov 2012 04:30:21 AM EET
Category: general
Severity: 3 -
Follow-up Comment #4, bug #19821 (project freeciv):
4) is now bug #20301 concerning all cases where units_lost is not properly
increased.
___
Reply to this item at:
http://gna.org/bugs/?19821
Follow-up Comment #3, patch #3561 (project freeciv):
Ahthis thing.
IIRC, when combined with use grids in citydlg overview page patch, the size
goes back to something reasonable - all that's left is (a gap for) the
scrollbar.
___
Reply
Follow-up Comment #3, patch #3560 (project freeciv):
Well, we could just set this to true in the theme.
Note, that there are other places already affected by this setting - that
screenshot in patch #3561 suggests you didn't notice that.
Follow-up Comment #1, bug #20301 (project freeciv):
TRUNK S2_4 versions
(file #16771, file #16772)
___
Additional Item Attachment:
File name: UnitsLostScore.patch Size:20 KB
File name: UnitsLostScore-S2_4.patch Size:20 KB
Update of bug #20301 (project freeciv):
Status:None = Ready For Test
___
Follow-up Comment #2:
This ticket is about fixing all cases.
Err. All cases where wipe_unit() is called since this
Update of bug #20270 (project freeciv):
Status:None = Ready For Test
Planned Release: = 2.3.3, 2.4.0, 2.5.0
___
Follow-up Comment #1:
Fix
(file #16773,
URL:
http://gna.org/patch/?3567
Summary: Automatically focus to zero move_rate units too
Project: Freeciv
Submitted by: cazfi
Submitted on: Sat 17 Nov 2012 05:41:19 AM EET
Category: client
Priority: 5 -
Follow-up Comment #2, patch #3566 (project freeciv):
Are you sure this patch plays a role here?
Well, as you found out that it's not happening reliably, it probably was
coincidence. It didn't happen before I applied the patch, it happened with the
patch, and again didn't happen once I reverted
Follow-up Comment #3, patch #3566 (project freeciv):
I reproduced something similar (but without the assertion) without the patch,
so it's not new with the patch.
It seems that Change Production does not work after you have already done
one thing with the worklist. You can Change Production once
24 matches
Mail list logo
