Page MenuHomeFreeBSD

Mk/Uses/php.mk: change exec and unexec to post*
ClosedPublic

Authored by joneum on Sep 2 2017, 2:09 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Mar 15, 6:15 AM
Unknown Object (File)
Feb 12 2024, 3:02 PM
Unknown Object (File)
Feb 5 2024, 4:21 AM
Unknown Object (File)
Jan 13 2024, 9:27 AM
Unknown Object (File)
Dec 22 2023, 10:17 PM
Unknown Object (File)
Dec 20 2023, 10:52 PM
Unknown Object (File)
Dec 17 2023, 3:39 PM
Unknown Object (File)
Oct 17 2023, 6:19 PM
Subscribers

Details

Summary

When "USES= php:ext" is set into the Makefile, there is a following
warning:"@exec is deprecated, please use @[pre|post][un]exec" [1][2]
After change exec and unexec to post* it works again correctly

[1] https://github.com/freebsd/freebsd-ports/blob/master/CHANGES#L407
[2] https://www.freebsd.org/doc/en/books/porters-handbook/book.html#plist-keywords-base-exec

Approved by: xxx (mentor)
#Differential Revision: https://reviews.freebsd.org/Dxxxxx

Test Plan

before (example: archivers/php-snappy)
http://joneumbox.org/data/10amd64-ports/2017-09-02_11h11m11s/logs/php-snappy-0.1.8.log

after change:
http://joneumbox.org/data/10amd64-ports/2017-09-02_15h54m19s/logs/php-snappy-0.1.8.log

i add ale, because: "PHP_Include_MAINTAIER= ale@FreeBSD.org"

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Why postunexec and not preunexec ?

% pkg info -RF /poudriere/data/packages/head-i386-default/All/php-snappy-0.1.7.txz 
...
scripts: {
    post-install: "echo \\#include \\\"ext/snappy/config.h\\\" >> /usr/local/include/php/ext/php_config.h",
    pre-deinstall: "cp /usr/local/include/php/ext/php_config.h /usr/local/include/php/ext/php_config.h.orig\ngrep -v ext/snappy/config.h /usr/local/include/php/ext/php_config.h.orig > /usr/local/include/php/ext/php_config.h || true\n/bin/rm -f /usr/local/include/php/ext/php_config.h.orig"
}

Why postunexec and not preunexec ?

I was not sure. I will test it with pre*

The output of the pkg info command was with what we currently do in the ports tree.

mv postunexec to preunexec

Mk/Uses/php.mk
282 ↗(On Diff #32611)

typo

This looks ok to me. @antoine, do you think an exp-run is needed ?

In D12215#253529, @mat wrote:

This looks ok to me. @antoine, do you think an exp-run is needed ?

The packages before and after the change don't differ, I don't think an exp-run is needed.

In D12215#253529, @mat wrote:

This looks ok to me. @antoine, do you think an exp-run is needed ?

The packages before and after the change don't differ, I don't think an exp-run is needed.

Great, then...

Macro ssshipit:

This revision is now accepted and ready to land.Sep 9 2017, 4:03 PM
This revision was automatically updated to reflect the committed changes.