Follow-up Comment #3, bug #21718 (project freeciv):
Both patches are in the git patch format (so they should work with git am). I
have a slight preference for alternative 2 since it avoids any regressions.
Alternative 1 is there in case saving work when the Freeciv svn version is
updated is more
URL:
http://gna.org/patch/?4540
Summary: Second pack of Oceanian unknowns nations: Marshall
and Cook Islands
Project: Freeciv
Submitted by: None
Submitted on: lun 24 feb 2014 10:55:48 UTC
Category: rulesets
URL:
http://gna.org/patch/?4541
Summary: third and last oceanian duet: Tuvalu and the Mariana
Islands
Project: Freeciv
Submitted by: None
Submitted on: lun 24 feb 2014 11:49:14 UTC
Category: rulesets
Update of patch #4528 (project freeciv):
Status: Ready For Test = Done
Open/Closed:Open = Closed
___
Reply to this item at:
Update of patch #4529 (project freeciv):
Status: Ready For Test = Done
Open/Closed:Open = Closed
___
Reply to this item at:
Follow-up Comment #1, patch #4539 (project freeciv):
tech_can_be_taken() checks if game.info.tech_trade_allow_holes, but stealing a
tech is not trading them.
I don't know if you need to make allow_holes a parameter to generic
tech_can_be_taken(), or do you need to rename it so that it's clear
Follow-up Comment #2, patch #4539 (project freeciv):
Do you have any ideas to improve handling of already-stolen-from-this-city
-case? It's quite user-unfriendly that one just has to remember where he has
already stolen a tech. Don't know if the current feature that one cannot know
if some third
URL:
http://gna.org/patch/?4542
Summary: Music Style
Project: Freeciv
Submitted by: cazfi
Submitted on: Mon 24 Feb 2014 07:42:24 PM EET
Category: general
Priority: 5 - Normal
Status:
Follow-up Comment #3, patch #4539 (project freeciv):
tech_can_be_taken() checks if game.info.tech_trade_allow_holes, but stealing
a tech is not trading them.
Thank you, Marko. New version renames tech_can_be_taken() to
tech_can_be_stolen() and use game.info.tech_steal_allow_holes. If needed it
Follow-up Comment #4, bug #21718 (project freeciv):
New version of alternative 2 that include the new version of patch #4539.
(file #20180)
___
Additional Item Attachment:
File name: 0001-Alternative-2.patch Size:17 KB
I did a test-migration from svn to github here:
https://github.com/freeciv/freeciv-test
This was done using svn2git:
https://github.com/nirvdrum/svn2git
https://help.github.com/articles/importing-from-subversion
Feel free to test it out. The actual migration process was quite quick and
easy to
Andreas Røsdal writes:
Also, what do you think of moving Freeciv development
from svn to github.com/freeciv?
For me, integration with the ticket tracker is the dealbreaker. It is
still painful having lost our RT history; I'm not not willing to lose
our history again (or linkage to the VCS)
Update of bug #21409 (project freeciv):
Status:None = In Progress
Assigned to:None = jtn
___
Follow-up Comment #4:
Here's a proof of
Looks good to me.
On Tue, Feb 25, 2014 at 1:35 AM, Andreas Røsdal andre...@pvv.ntnu.nowrote:
I did a test-migration from svn to github here:
https://github.com/freeciv/freeciv-test
This was done using svn2git:
https://github.com/nirvdrum/svn2git
Update of bug #21668 (project freeciv):
Status: Ready For Test = Fixed
Assigned to:None = cazfi
Open/Closed:Open = Closed
Update of patch #4487 (project freeciv):
Depends on: = patch #4542
___
Reply to this item at:
http://gna.org/patch/?4487
___
Message sent
Update of patch #4518 (project freeciv):
Status: Ready For Test = Done
Assigned to:None = cazfi
Open/Closed:Open = Closed
Update of patch #4520 (project freeciv):
Status: Ready For Test = In Progress
___
Follow-up Comment #1:
Problem: This checks extra cause to determine if we're trying to build
irrigation extra instead
Follow-up Comment #4, patch #4539 (project freeciv):
Do you have any ideas to improve handling of already-stolen-from-this-city
-case?
Yes. - the defender should receive the same information as the attacker.
Currently only the attacker is told you have already-stolen-from-this-city.
Update of patch #4520 (project freeciv):
Status: In Progress = Ready For Test
___
Follow-up Comment #2:
Make targeted fix to next_extra_for_tile() where the activity being planned is
known instead.
Follow-up Comment #2, patch #4534 (project freeciv):
http://forum.freeciv.org/f/viewtopic.php?f=14t=242#p1478
___
Reply to this item at:
http://gna.org/patch/?4534
___
Message sent via/by
Follow-up Comment #3, patch #4534 (project freeciv):
- Updated against svn
(file #20183)
___
Additional Item Attachment:
File name: Mood-2.patch Size:5 KB
___
22 matches
Mail list logo