Subscribe to download
Firing Range

OK, fixed it. Thanks Ishipp (and Revolverman).

Glad to help! : )

Actually never mind, you're right. Which appears to be why this isn't working. Thanks for noticing that.

Nope. There is an odd thing with some of the set-up-to-fire units, though. As near as I can figure, Catapults and Trebuchets (possibly Cannons too) are Siege units, but their late-game upgrades (Artillery, Rocket Artillery) are Gun units.
Also, I think that Gunpowder units can be distinguished from Melee units for situations where you have a Great War Infantry (Gunpowder) fighting against a Longswordsman. If the Great War Infantry has a bonus against other Gunpowder units, he wouldn't get it against the Longswordsman. However, that last paragraph was just speculation.
Another option is that the system may work differently in Brave New World, which I don't have. If you have it, than we could both be right.

Right, but arn't they two diffrent classes when it comes to buffs and the such?

Same thing. Gunpowder units are just mid- to late-game melee units. Remember that the Longswordsman upgrades to Musketman - they are both melee units.

If I recall, they are gunpowder units, not melee units.

If you were thinking it should boost siege units (because of target practice and such), that would be a Proving Ground , which is going to be a component of my next Military Buildings mod.

Because it's unlocked with the Rifling technology, which also unlocks the Rifleman unit.
The Rifleman is the upgrade for the Musketman, which is the upgrade for the Longswordsman, which is a melee unit. So you can't use it like a Barracks to boost just any unit (Warrior, Crossbowman, etc), but only to boost the kinds of units that would actually train using it.
If that was hard to understand, here's the short version: Because Riflemen and Infantry are melee units in Civ5.
Does that help?;)

Why would a Firing Range boost MELEE units?