Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 2 Feb 2017 10:21:00 -0700
From:      Adam Weinberger <adamw@adamw.org>
To:        Torsten Zuehlsdorff <tz@freebsd.org>
Cc:        Jim Ohlstein <jim@ohlste.in>, ports@FreeBSD.org, linimon@FreeBSD.org
Subject:   Re: Recent update to lang/php70 and lang/php71
Message-ID:  <83055CCD-D150-4D1D-8F4E-A422E1F63D1A@adamw.org>
In-Reply-To: <6d56b335-44c8-5ddb-20c0-ce44ba208ae6@FreeBSD.org>
References:  <15b31c6c-52be-e04f-64bd-3dc5b8cb2332@ohlste.in> <023c7b00-7b06-796f-c091-446c61320222@FreeBSD.org> <8e109406-a056-9043-edc9-d36c92346a4f@ohlste.in> <6d56b335-44c8-5ddb-20c0-ce44ba208ae6@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
> On 2 Feb, 2017, at 7:39, Torsten Zuehlsdorff <tz@freebsd.org> wrote:
>=20
> On 02.02.2017 15:37, Jim Ohlstein wrote:
>> Hello,
>>=20
>> On 02/02/2017 04:35 AM, Torsten Zuehlsdorff wrote:
>>> Hello Jim,
>>>=20
>>>> Could not this semantic change have waited for a new version? It =
forced
>>>> a rebuild of ALL PHP extensions for no reason. They all had a =
dependency
>>>> on devel/pcre via the main port as it was.
>>>=20
>>> Did you use poudriere?
>>=20
>> Yes
>>=20
>>>=20
>>> It wasn't meant to force a rebuild. There was no PORTREVISION bump =
nor a
>>> new dependency. But poudriere detect a new one, so i would say its a
>>> poudriere bug. Or am i wrong?
>>=20
>> Looks like a direct dependency was added. Here's a sample output from =
a
>> jail that runs PHP70:
>>=20
>> Installed packages to be REINSTALLED:
>>    [output]
>=20
> Yes, it look like, but it wasn't. That's the patch:
> =
https://svnweb.freebsd.org/ports/head/lang/php71/Makefile?r1=3D433046&r2=3D=
433045&pathrev=3D433046 =
<https://svnweb.freebsd.org/ports/head/lang/php71/Makefile?r1=3D433046&r2=3D=
433045&pathrev=3D433046>
>=20
> Only one line was moved more upwards. Nothing more. This shouldn't =
have caused such a detection.


It=E2=80=99s not, though, because before it was protected within =
!defined(PKGNAMESUFFIX). Previously, only php itself brought in the pcre =
dependency; now all the extensions bring it in too.

# Adam


=E2=80=94
Adam Weinberger
adamw@adamw.org
http://www.adamw.org




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?83055CCD-D150-4D1D-8F4E-A422E1F63D1A>