From owner-cvs-ports@FreeBSD.ORG  Mon Jun  4 13:35:05 2012
Return-Path: <owner-cvs-ports@FreeBSD.ORG>
Delivered-To: cvs-ports@FreeBSD.org
Received: by hub.freebsd.org (Postfix, from userid 975)
	id E8C3D1065672; Mon,  4 Jun 2012 13:35:05 +0000 (UTC)
Date: Mon, 4 Jun 2012 13:35:05 +0000
From: "Sergey A. Osokin" <osa@FreeBSD.org>
To: Baptiste Daroussin <bapt@FreeBSD.org>
Message-ID: <20120604133505.GP87285@FreeBSD.org>
References: <201206040407.q5447cSC064412@repoman.freebsd.org>
	<20120604132958.GC73254@ithaqua.etoilebsd.net>
Mime-Version: 1.0
Content-Type: text/plain; charset=koi8-r
Content-Disposition: inline
In-Reply-To: <20120604132958.GC73254@ithaqua.etoilebsd.net>
User-Agent: Mutt/1.4.2.1i
Cc: cvs-ports@FreeBSD.org, cvs-all@FreeBSD.org, ports-committers@FreeBSD.org
Subject: Re: cvs commit: ports/www/nginx Makefile ports/www/nginx-devel
	Makefile
X-BeenThere: cvs-ports@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: CVS commit messages for the ports tree <cvs-ports.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-ports>,
	<mailto:cvs-ports-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-ports>
List-Post: <mailto:cvs-ports@freebsd.org>
List-Help: <mailto:cvs-ports-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-ports>,
	<mailto:cvs-ports-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Mon, 04 Jun 2012 13:35:06 -0000

On Mon, Jun 04, 2012 at 03:29:58PM +0200, Baptiste Daroussin wrote:
> On Mon, Jun 04, 2012 at 04:07:38AM +0000, Sergey A. Osokin wrote:
> > osa         2012-06-04 04:07:38 UTC
> > 
> >   FreeBSD ports repository
> > 
> >   Modified files:
> >     www/nginx            Makefile 
> >     www/nginx-devel      Makefile 
> >   Log:
> >   OptionsNGfy.
> >   Most name of variables are shortened, s/_MODULE_//g.
> >   
> >   Thanks to:      bapt
> >   
> >   Revision  Changes    Path
> >   1.457     +456 -379  ports/www/nginx-devel/Makefile
> >   1.330     +456 -379  ports/www/nginx/Makefile
> 
> Btw I have seen something wrong in your options-ng conversion
> the VIDEO single the way you wrote it makes it mandatory but you didn't define
> any default for it.
> 
> I think what you want is to have 0 or only 1 VIDEO options
> in that case just add VIDEO to OPTIONS_DEFINE

It didn't help me.

> if the user check VIDEO then he will have to choses one of the HTTP_MP4_* if
> video is uncheck then the single will just be omitted.

--
ozz