Build is green

Updating workspaces.
Build (release)...
Build (debug)...
Running debug tests...
Running cxxtest tests (302 tests)..............................................................................................................................................................................................................................................................................................................OK!

http://jw:8080/job/phabricator/317/ for more details.

Link to the discussion?

In D129#4477, @elexis wrote:

Link to the discussion?

https://wildfiregames.com/forum/index.php?/topic/18003-suggestions-for-0-ad/&page=130#comment-326405

and comments above

The patch is complete as those are all units that can occur derived from the starting entities, and since those are the only files containing the removed characters.

Since the discussion only contained a "why not", I digged up a bit of history, trying to find out why it was removed in the first place:
In rP18024, scythetwirler made it consistent
In rP16878, mimo renamed the template (#3059)
In rP16285, scythetwirlers balancing branch removed the tower from the roman swordsman

Judging from the last diff, I guess the intention of the last commit was to make it consistent (no wooden walls, therefore no wooden tower).

In rP11860 michael "Removed palisades from Roman infantry prod queue." so I guess you know whom to ask if you want to know more about it. Alternatively post a stronger opinon than a "why not" here and you'll get my ack ;-)

I guess, you/I/we will wait a long time so.

edit: (but I'll be positive. It's because first wooden tower was part of palisade and as roman have wooden siege tower, they was punished by not having wooden palisade).

Romans still don't have wooden walls in age 1. Their siege walls occur in age 3.
So if you make those towers available in age 1, you will give them defense buildings in age 1, thus remove the civ specificness, hence leaning towards disagreement with the proposal.

If we have to add the entity because there is a model for it, then it could become an age 2 building perhaps?

In D129#4512, @fatherbushido wrote:

I guess, you/I/we will wait a long time so.

Why? There are some active, qualified and related people on the forums who might become alerted when creating a topic about adding the towers.
Also some of our devs might stumble across this differential proposal and post a comment if they have an opposing opinion.

In D129#4517, @elexis wrote:

Romans still don't have wooden walls in age 1. Their siege walls occur in age 3.
So if you make those towers available in age 1, you will give them defense buildings in age 1, thus remove the civ specificness, hence leaning towards disagreement with the proposal.

If we have to add the entity because there is a model for it, then it could become an age 2 building perhaps?

?

In D129#4512, @fatherbushido wrote:

I guess, you/I/we will wait a long time so.

Why? There are some active, qualified and related people on the forums who might become alerted when creating a topic about adding the towers.
Also some of our devs might stumble across this differential proposal and post a comment if they have an opposing opinion.

If we have to add the entity because there is a model for it, then it could become an age 2 building perhaps?

That refered to LordGoods comment "It's a wasted building otherwise"

A little offtopic, but is it intended, that siege wall and palisade have the same icon?
(Until know I didn't knew rome has a siege wall, because of the palisade icon)

In D129#4512, @fatherbushido wrote:

edit: (but I'll be positive. It's because first wooden tower was part of palisade and as roman have wooden siege tower, they was punished by not having wooden palisade).

Didn't get notified you had add/ited that before answering the questionmark. I asked the original contributors to get closer to the primary source.

In D129#4528, @Imarok wrote:

A little offtopic, but is it intended, that siege wall and palisade have the same icon?
(Until know I didn't knew rome has a siege wall, because of the palisade icon)

Thanks for fixing the reference in https://code.wildfiregames.com/D137