svn commit: r424223 - in head/www: nginx nginx-devel

[Adam Weinberger]

> On 21 Oct, 2016, at 17:00, Sergey A. Osokin <osa at FreeBSD.org> wrote:
> 
> On Fri, Oct 21, 2016 at 10:37:09AM +0200, Mathieu Arnold wrote:
>> Le 21/10/2016 ?? 01:23, Sergey A. Osokin a ??crit :
>>> On Wed, Oct 19, 2016 at 09:13:53AM +0200, Mathieu Arnold wrote:
>>>> Le 19/10/2016 ?? 01:10, Sergey A. Osokin a ??crit :
>>>>> Author: osa
>>>>> Date: Tue Oct 18 23:10:35 2016
>>>>> New Revision: 424223
>>>>> URL: https://svnweb.freebsd.org/changeset/ports/424223
>>>>> 
>>>>> Log:
>>>>>  Use ImageMagick-nox11 instead of original one as it's more
>>>>>  convenient for the server-based product.
>>>> Please revert that, It will mess up systems who already have
>>>> ImageMagick. I you really feel strongly about it, provide a new option
>>>> to depend on the -nox11 version.
>>> What kind of the mess up?  Could you please provide a bit more details.
>> 
>> Well, just pkg install one port that needs ImageMagick, and then pkg
>> install nginx, installing nginx will install ImageMagick-nox11, that
>> will deinstall ImageMagick, that will remove the other port you needed.
> 
> Where can I get that package with disabled by default third-party module?
> Are you talking about the "custom" package, not an "official" one from
> FreeBSD repo, that probably has been built by user?
> 
>> Like I said, please revert that commit.
> 
> Again, I see no reason for this.  Hope you can provide a bit more arguments
> for the action.

Hi Sergey,

The problem is that, for users with the option enabled, it forcibly uninstalls the X11-enabled ImageMagick. It's possible for ImageMagick users to build without x11 if they want, but it's impossible for ImageMagick-nox11 users to build *with* x11 if they wanted it.

Forcing a dependency on IM-nox11 means that people can never have nginx (with those options enabled) and IM with x11 support at the same time. Having nginx depend on plain old ImageMagick means that people have a choice.

# Adam


-- 
Adam Weinberger
adamw at adamw.org
http://www.adamw.org