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 27 Mar 2013 08:25:30 AM UTC  
 
Category: generalPriority: 5 - Normal
Status: DonePrivacy: Public
Assigned to: Marko Lindqvist <cazfi>Originator Email: -unavailable-
Open/Closed: ClosedPlanned Release: 2.5.0

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 09 Apr 2013 09:22:44 PM UTC, 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 07 Apr 2013 06:59:20 AM UTC, comment #7:

- Updated against current svn HEAD

(file #17693)

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

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

(file #17654)

Emmet Hikory <persia>
Project Member
Mon 01 Apr 2013 06:29:03 PM UTC, 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 01 Apr 2013 10:44:47 AM UTC, 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 01 Apr 2013 07:12:56 AM UTC, 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 31 Mar 2013 10:18:22 PM UTC, 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 31 Mar 2013 09:27:15 PM UTC, 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 27 Mar 2013 08:25:30 AM UTC, 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.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 9 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Tue 09 Apr 2013 09:23:08 PM UTCcazfiStatusReady For Test=>Done
      Open/ClosedOpen=>Closed
    Sun 07 Apr 2013 06:59:20 AM UTCcazfiAttached File-=>Added ParatroopTransport.patch, #17693
    Tue 02 Apr 2013 01:33:53 AM UTCpersiaAttached File-=>Added transport-paradrop+option+docs.patch, #17654
    Mon 01 Apr 2013 10:44:47 AM UTCpersiaAttached File-=>Added transport-paradrop+ruleset-option.patch, #17644
    Wed 27 Mar 2013 12:56:21 PM UTCcazfiStatusNone=>Ready For Test
      Assigned toNone=>cazfi
      Planned Release=>2.5.0
    Wed 27 Mar 2013 08:25:30 AM UTCNoneAttached File-=>Added transport-paradrop.patch, #17559
    Show feedback again

    Back to the top


    Powered by Savane 3.1-cleanup