patchFreeciv - Patches: patch #3805, Allow paratroopers to land on...

 
 
Show feedback again

patch #3805: Allow paratroopers to land on transport

Submitted by:  None
Submitted on:  Wed Mar 27 08:25:30 2013  
 
Category: generalPriority: 5 - Normal
Status: DonePrivacy: Public
Assigned to: Marko Lindqvist <cazfi>Originator Email: -unavailable-
Open/Closed: ClosedPlanned Release: 2.5.0
Contains string changes: None

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

Please log in, so followups can be emailed to you.

 

(Jump to the original submission Jump to the original submission)

Tue Apr 9 21:22:44 2013, SVN revision 22705:

Added ruleset option to allow paratroopers to land on transport.

Patch by Emmet Hikory

See patch #3805

(Browse SVN revision 22705)

Marko Lindqvist <cazfi>
Project AdministratorIn charge of this item.
Sun Apr 7 06:59:20 2013, comment #7:

- Updated against current svn HEAD

(file #17693)

Marko Lindqvist <cazfi>
Project AdministratorIn charge of this item.
Tue Apr 2 01:33:53 2013, comment #6:

Oops, right. Updated patch (transport-paradrop+option+docs.patch) includes that.

(file #17654)

Emmet Hikory <persia>
Project Member
Mon Apr 1 18:29:03 2013, comment #5:

I think enabling this in experimental ruleset makes sense. But that requires update also to doc/README.ruleset_experimental.

Marko Lindqvist <cazfi>
Project AdministratorIn charge of this item.
Mon Apr 1 10:44:47 2013, comment #4:

This updated patch is controlled by game.info.paradrop_to_transport. This is disabled in most rulesets, but enabled in experimental for testing. If the commiter doesn't feel it appropriate to enable it in experimental, please change "TRUE" to "FALSE" pre-commit. Note that this requires a change to the capabilities string for the new value. Some conditional orders have been adjusted to more easily terminate early.

(file #17644)

Emmet Hikory <persia>
Project Member
Mon Apr 1 07:12:56 2013, comment #3:

New boolean setting in game.ruleset [civstyle] for controlling if dropping to transport is allowed should be sufficient.

Marko Lindqvist <cazfi>
Project AdministratorIn charge of this item.
Sun Mar 31 22:18:22 2013, comment #2:

I think it does. While I have no direct experience, the information I have found seems to indicate that civ1 has no paratroopers and civ2 requires the destination to be Land. The best reference that mentions both of these that I could find was http://www.nationmaster.com/encyclopedia/Civilization-II-units

Any recommendations on how this should be set? By another paratrooper-specific unit field? By a game setting? By something else?

Separately, if anyone is able to confirm the behaviour from actual play, that would be helpful: while I suspect the referenced document of being correct, I am uncertain if it is based on extensive playtesting or generated from found documentation. Alternately, if anyone is certain of the authority of nationmaster.com, that works just as well :)

Emmet Hikory <persia>
Project Member
Sun Mar 31 21:27:15 2013, comment #1:

Does this need to be ruleset definable to support civ1/civ2 rulesets? How civ1/civ2 behaved in this respect?

Marko Lindqvist <cazfi>
Project AdministratorIn charge of this item.
Wed Mar 27 08:25:30 2013, original submission:

Currently paratroopers are able to land on tiles that have been made
native by roads, bases, or terrains, but not on tiles that have been made
survivable by the presence of a unit. The attached patch changes this,
so that paratroopers can fly to safe places. I believe it to be client-safe,
so that if a client believes there to be a transport on some tile, the player
can drown their paratrooper from overtrust, and AI-safe, in that H_FOG
AIs will decide they can't land there, and !H_FOG AIs will drop through
to the death test, and use that to decide not to go there.

Anonymous

 

(Note: upload size limit is set to 1024 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

Attached Files
file #17693:  ParatroopTransport.patch added by cazfi (8kB - text/x-diff)
file #17654:  transport-paradrop+option+docs.patch added by persia (9kB - application/octet-stream)
file #17644:  transport-paradrop+ruleset-option.patch added by persia (8kB - application/octet-stream)
file #17559:  transport-paradrop.patch added by None (2kB - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by persia (Posted a comment)
  • -unavailable- added by cazfi (Updated the item)
  • -unavailable- added by None (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Error: not logged in

     

     

    Follow 9 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Tue Apr 9 21:23:08 2013cazfiStatusReady For Test=>Done
      Open/ClosedOpen=>Closed
    Sun Apr 7 06:59:20 2013cazfiAttached File-=>Added ParatroopTransport.patch, #17693
    Tue Apr 2 01:33:53 2013persiaAttached File-=>Added transport-paradrop+option+docs.patch, #17654
    Mon Apr 1 10:44:47 2013persiaAttached File-=>Added transport-paradrop+ruleset-option.patch, #17644
    Wed Mar 27 12:56:21 2013cazfiStatusNone=>Ready For Test
      Assigned toNone=>cazfi
      Planned Release=>2.5.0
    Wed Mar 27 08:25:30 2013NoneAttached File-=>Added transport-paradrop.patch, #17559
    Show feedback again

    Back to the top


    Powered by Savane 3.1-cleanup