svn commit: r391626 - head/games/warzone2100

[John Marino]

On 7/10/2015 3:59 PM, Dmitry Marakasov wrote:
> * John Marino (freebsd.contact at marino.st) wrote:
> Like I've already said, the dependency is absolutely correct and does what
> it should. It's a no-op though on all supported FreeBSD versions,
> so I'm removing it.

I've suggested (based on it's presence) that it should be pulling in
textproc/flex, which is no longer does.

IF this package is supposed to pull in the ports flex for a specific
reason, then the change obviously prevents that.

It's not clear to me why are you stating textproc/flex is a bad
dependency.  Somebody put it there intentionally.  I would have tried to
figure out the exact reason it was put there before declaring it is wrong.

It must have been there for a reason.  The dependency has always been a
"no-op" so removing it based on that seems like weak logic to me.

But -- I raised my concern.  I'm not going to discuss it anymore.  If
warzone is busted because of these changes somebody will raise a PR.

John