Date: Tue, 21 Nov 2017 12:55:57 +0100 From: Mathieu Arnold <mat@FreeBSD.org> To: Alexey Dokuchaev <danfe@FreeBSD.org>, Mathieu Arnold <mat@FreeBSD.org> Cc: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: Re: svn commit: r454590 - in head: astro/gpsbabel audio/clementine-player audio/qmpdclient audio/vagalume chinese/libtabe devel/stlink games/spring korean/scim-hangul lang/sagittarius-scheme mail/faces... Message-ID: <eee76be4-71d9-6588-0622-703f0d211200@FreeBSD.org> In-Reply-To: <20171121084122.GA68760@FreeBSD.org> References: <201711210836.vAL8aROu055852@repo.freebsd.org> <20171121084122.GA68760@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Le 21/11/2017 =C3=A0 09:41, Alexey Dokuchaev a =C3=A9crit=C2=A0: > On Tue, Nov 21, 2017 at 08:36:27AM +0000, Mathieu Arnold wrote: >> New Revision: 454590 >> URL: https://svnweb.freebsd.org/changeset/ports/454590 >> >> Log: >> OPTIONS_SUB only cares about being defined, its content is ignored. > Actually I always wanted it to have this feature to avoid extra namespa= ce > pollution when only a fraction of options affects pkg-plist. Would we = be > interested in this? I might crack out a patch. That would be a no, please do not. The code is simple, and works like a charm, I cannot imagine any case where one would *need* to restrict the number of placeholders in PLIST_SU= B. A better solution would be to have fewer options. --=20 Mathieu Arnold
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?eee76be4-71d9-6588-0622-703f0d211200>