Re: Recent update to lang/php70 and lang/php71

[Torsten Zuehlsdorff]

On 02.02.2017 18:21, Adam Weinberger wrote:

On 2 Feb, 2017, at 7:39, Torsten Zuehlsdorff  wrote:

On 02.02.2017 15:37, Jim Ohlstein wrote:

Hello,

On 02/02/2017 04:35 AM, Torsten Zuehlsdorff wrote:

Hello Jim,


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.


Did you use poudriere?


Yes



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?


Looks like a direct dependency was added. Here's a sample output from a
jail that runs PHP70:

Installed packages to be REINSTALLED:
   [output]


Yes, it look like, but it wasn't. That's the patch:
https://svnweb.freebsd.org/ports/head/lang/php71/Makefile?r1=433046&r2=433045&pathrev=433046 


Only one line was moved more upwards. Nothing more. This shouldn't have caused 
such a detection.



It’s 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.


Thanks for pointing to this. I reverted the patch.

Greetings,
Torsten
___
freebsd-ports@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-ports
To unsubscribe, send any mail to "freebsd-ports-unsubscr...@freebsd.org"

Re: Recent update to lang/php70 and lang/php71

[Adam Weinberger]

> On 2 Feb, 2017, at 7:39, Torsten Zuehlsdorff  wrote:
> 
> On 02.02.2017 15:37, Jim Ohlstein wrote:
>> Hello,
>> 
>> On 02/02/2017 04:35 AM, Torsten Zuehlsdorff wrote:
>>> Hello Jim,
>>> 
 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.
>>> 
>>> Did you use poudriere?
>> 
>> Yes
>> 
>>> 
>>> 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?
>> 
>> Looks like a direct dependency was added. Here's a sample output from a
>> jail that runs PHP70:
>> 
>> Installed packages to be REINSTALLED:
>>[output]
> 
> Yes, it look like, but it wasn't. That's the patch:
> https://svnweb.freebsd.org/ports/head/lang/php71/Makefile?r1=433046&r2=433045&pathrev=433046
>  
> 
> 
> Only one line was moved more upwards. Nothing more. This shouldn't have 
> caused such a detection.


It’s 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


—
Adam Weinberger
ad...@adamw.org
http://www.adamw.org

___
freebsd-ports@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-ports
To unsubscribe, send any mail to "freebsd-ports-unsubscr...@freebsd.org"

Re: Recent update to lang/php70 and lang/php71

[Torsten Zuehlsdorff]

On 02.02.2017 15:37, Jim Ohlstein wrote:

Hello,

On 02/02/2017 04:35 AM, Torsten Zuehlsdorff wrote:

Hello Jim,


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.


Did you use poudriere?


Yes



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?


Looks like a direct dependency was added. Here's a sample output from a
jail that runs PHP70:

Installed packages to be REINSTALLED:
[output]


Yes, it look like, but it wasn't. That's the patch:
https://svnweb.freebsd.org/ports/head/lang/php71/Makefile?r1=433046&r2=433045&pathrev=433046

Only one line was moved more upwards. Nothing more. This shouldn't have 
caused such a detection.


Greetings,
Torsten
___
freebsd-ports@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-ports
To unsubscribe, send any mail to "freebsd-ports-unsubscr...@freebsd.org"

Re: Recent update to lang/php70 and lang/php71

[Jim Ohlstein]

Hello,

On 02/02/2017 04:35 AM, Torsten Zuehlsdorff wrote:

Hello Jim,


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.


Did you use poudriere?


Yes



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?


Looks like a direct dependency was added. Here's a sample output from a 
jail that runs PHP70:


Installed packages to be REINSTALLED:
php70-zlib-7.0.15 [poudriere-php70] (direct dependency added: pcre)
php70-zip-7.0.15 [poudriere-php70] (direct dependency added: pcre)
php70-xsl-7.0.15 [poudriere-php70] (direct dependency added: pcre)
php70-xmlwriter-7.0.15 [poudriere-php70] (direct dependency added: pcre)
php70-xmlreader-7.0.15 [poudriere-php70] (direct dependency added: pcre)
php70-xml-7.0.15 [poudriere-php70] (direct dependency added: pcre)
php70-wddx-7.0.15 [poudriere-php70] (direct dependency added: pcre)
php70-sqlite3-7.0.15 [poudriere-php70] (direct dependency changed: pcre)
php70-simplexml-7.0.15 [poudriere-php70] (direct dependency added: pcre)
php70-session-7.0.15 [poudriere-php70] (direct dependency added: pcre)
php70-posix-7.0.15 [poudriere-php70] (direct dependency added: pcre)
php70-pdo_sqlite-7.0.15 [poudriere-php70] (direct dependency changed: 
pcre)
php70-pdo_mysql-7.0.15 [poudriere-php70] (direct dependency changed: 
pcre)
php70-pdo-7.0.15 [poudriere-php70] (direct dependency added: pcre)
php70-openssl-7.0.15 [poudriere-php70] (direct dependency added: pcre)
php70-mbstring-7.0.15 [poudriere-php70] (direct dependency changed: 
pcre)
php70-json-7.0.15 [poudriere-php70] (direct dependency added: pcre)
php70-iconv-7.0.15 [poudriere-php70] (direct dependency added: pcre)
php70-hash-7.0.15 [poudriere-php70] (direct dependency added: pcre)
php70-filter-7.0.15 [poudriere-php70] (direct dependency added: pcre)
php70-fileinfo-7.0.15 [poudriere-php70] (direct dependency added: pcre)
php70-exif-7.0.15 [poudriere-php70] (direct dependency added: pcre)
php70-dom-7.0.15 [poudriere-php70] (direct dependency added: pcre)
php70-curl-7.0.15 [poudriere-php70] (direct dependency added: pcre)
php70-ctype-7.0.15 [poudriere-php70] (direct dependency added: pcre)
php70-bz2-7.0.15 [poudriere-php70] (direct dependency added: pcre)



--
Jim Ohlstein
___
freebsd-ports@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-ports
To unsubscribe, send any mail to "freebsd-ports-unsubscr...@freebsd.org"

Re: Recent update to lang/php70 and lang/php71

[Torsten Zuehlsdorff]

Hello Jim,


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.


Did you use poudriere?

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?


Sorry for the rebuild - this wasn't intended. :/

Greetings,
Torsten
___
freebsd-ports@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-ports
To unsubscribe, send any mail to "freebsd-ports-unsubscr...@freebsd.org"

Recent update to lang/php70 and lang/php71

[Jim Ohlstein]

Hello,

Thanks for your hard work.

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.


--
Jim Ohlstein
___
freebsd-ports@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-ports
To unsubscribe, send any mail to "freebsd-ports-unsubscr...@freebsd.org"