Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 10 Jul 2015 16:14:37 +0200
From:      John Marino <freebsd.contact@marino.st>
To:        Dmitry Marakasov <amdmi3@amdmi3.ru>, marino@freebsd.org
Cc:        ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org
Subject:   Re: svn commit: r391626 - head/games/warzone2100
Message-ID:  <559FD34D.2080803@marino.st>
In-Reply-To: <20150710135940.GB9933@hades.panopticon>
References:  <201507090944.t699iqBR039850@repo.freebsd.org> <559E431B.6030708@marino.st> <20150709171246.GB46547@hades.panopticon> <559EAD6F.3050409@marino.st> <20150710135940.GB9933@hades.panopticon>

next in thread | previous in thread | raw e-mail | index | archive | help
On 7/10/2015 3:59 PM, Dmitry Marakasov wrote:
> * John Marino (freebsd.contact@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



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?559FD34D.2080803>