svn commit: r305357 - head/net/ssvnc

[Alexey Dokuchaev]

On Mon, Oct 08, 2012 at 04:01:54AM +0000, Alexey Dokuchaev wrote:
> On Sat, Oct 06, 2012 at 11:23:17AM +0000, Baptiste Daroussin wrote:
> > New Revision: 305357
> > URL: http://svn.freebsd.org/changeset/ports/305357
> > 
> > Log:
> >   fix sense of a test
> >   
> > -.if ${PORT_OPTIONS:MULTRAFTP}
> > +.if empty(PORT_OPTIONS:MULTRAFTP)
> 
> That looks weird, and commit message is pretty cryptic as well.  Could you
> explain what went wrong here so others would not fall in the same pitfall?

Err, I read is as ! vs. empty() test.  Still, I've noticved some arguments
on the syntax, would be nice to know if ! ${PORT_OPTIONS:M...} has issues
other than purely stylish.

Thanks,

./danfe