Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 24 Jun 2016 15:32:22 +0200
From:      Mathieu Arnold <mat@FreeBSD.org>
To:        Adam Weinberger <adamw@adamw.org>
Cc:        ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org
Subject:   Re: svn commit: r417374 - in head: Mk Mk/Uses lang/php55 lang/php56 lang/php70 www/redaxo
Message-ID:  <141BC4CD56168688B595384F@ogg.in.absolight.net>
In-Reply-To: <633619EE-3555-435A-BD17-389F5ED6C6C4@adamw.org>
References:  <201606231319.u5NDJILd062527@repo.freebsd.org> <2400E414-B4B4-479C-BE0F-D804F1E0110A@adamw.org> <83BA65E7C58012B7AA8A7962@atuin.in.mat.cc> <307F483A-1ABE-4E02-A268-16F014D8DC44@adamw.org> <7E036E80F9D532528C3155FB@atuin.in.mat.cc> <633619EE-3555-435A-BD17-389F5ED6C6C4@adamw.org>

next in thread | previous in thread | raw e-mail | index | archive | help
--==========4050975671BE119B3168==========
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

+--On 24 juin 2016 06:33:37 -0600 Adam Weinberger <adamw@adamw.org> wrote:
|> On 24 Jun, 2016, at 1:28, Mathieu Arnold <mat@FreeBSD.org> wrote:
|> 
|> +--On 23 juin 2016 16:37:36 -0600 Adam Weinberger <adamw@adamw.org>
|> wrote:
|> |> On 23 Jun, 2016, at 15:36, Mathieu Arnold <mat@FreeBSD.org> wrote:
|> |> 
|> |> +--On 23 juin 2016 13:46:41 -0600 Adam Weinberger <adamw@adamw.org>
|> |> wrote:
|> |> |> On 23 Jun, 2016, at 7:19, Mathieu Arnold <mat@FreeBSD.org> wrote:
|> |> |> 
|> |> |> Author: mat
|> |> |> Date: Thu Jun 23 13:19:17 2016
|> |> |> New Revision: 417374
|> |> |> URL: https://svnweb.freebsd.org/changeset/ports/417374
|> |> |> 
|> |> |> Log:
|> |> |>  Move bsd.php.mk to Uses/php.mk
|> |> |> 
|> |> |>  PR:		210323
|> |> |>  Submitted by:	mat
|> |> |>  Exp-run by:	antoine
|> |> |>  Sponsored by:	Absolight
|> |> |>  Differential Revision:	https://reviews.freebsd.org/D6867
|> |> | 
|> |> |> # If you are building PHP-based ports in poudriere(8) with ZTS
|> |> |> # enabled, add WITH_MPM=event to /etc/make.conf to prevent build
|> |> |> # failures.
|> |> | 
|> |> | Does USES=php still have the above bug? If not, are there plans to
|> |> | make ZTS detection work without assuming that everybody has apache
|> |> | installed?
|> |> 
|> |> I did not change a thing, I'm modernizing the framework, I'm changing
|> |> how things are done, but not what the things being done do.
|> |> 
|> |> (Also, I have absolutely no idea what ZTS is, or what the difference
|> |> between the different MPM are.)
|> | 
|> | ZTS is a threading model for PHP. For a long time, bsd.php.mk has made
|> | the strange decision to use the output of 'httpd -V' (from apache) to
|> | determine whether PHP was built with ZTS. It uses that to determine
|> | which directory to put modules into.
|> | 
|> | The problem with that bizarre method is that if you don't have apache
|> | installed, it puts modules into the wrong directory, and then all
|> | further builds fail. In order to build PHP ports in poudriere, you
|> | have to trick the system by using that WITH_MPM=event line.
|> | 
|> | There are a couple bugs open about it. 201193 and 207284 are both open
|> | for it. php.mk should either put all PHP modules into one place
|> | (there's really no point to making the module dir change), or figure
|> | out whether PHP was built with ZTS by examining php.ini or the output
|> | of php-config.
|> 
|> I'll have a look at it, I'm refactoring USES=php right now[1].  Though,
|> I'm more inclined to do baby steps, and not change what's in it.  The
|> directory could easily be put in the php.conf file, it would solve all
|> those problems, I think.
|> 
|> 1: https://reviews.freebsd.org/D6936
| 
| That definitely feels like the right place for it.

I really feel it is not. It would change what things are done, which is
what I'm trying to avoid to keep things as simple as possible.

When the conversion is done, anyone can fix the problem you're describing.

-- 
Mathieu Arnold
--==========4050975671BE119B3168==========
Content-Type: application/pgp-signature
Content-Transfer-Encoding: 7bit

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1

iQJ8BAEBCgBmBQJXbTZmXxSAAAAAAC4AKGlzc3Vlci1mcHJAbm90YXRpb25zLm9w
ZW5wZ3AuZmlmdGhob3JzZW1hbi5uZXQzQUI2OTc4OUQyRUQxMjEwNjQ0MEJBNUIz
QTQ1MTZGMzUxODNDRTQ4AAoJEDpFFvNRg85ITXEP/jKVgDPTJ2kkg+AoEaaD9HZ4
Y/niTcRJP94XXOpQoGYTS/kNM2VytE2ErqLz/qQ2SYmWjUQzlR/ntcVAxeDq5FKI
H4ByKbWWE+7HJC78kHcPgL4O/qWX4yGT3luTmtbSo5uQ9MjbPyeDwijmtvCKL1Ba
D02dxVHUjvg3wCLE7iFiD/8rURqF1HUGXXnq0jkSPjWHs721uRAxKade/sBviBfE
VMBryG7aObsJd9XtLe6V2EcEpkqtoUKdh/medLCsmsXK6ef9pxV85IMEyY2y38Td
/QBtIA3ygn0Ra4YpPzObEN28AKuVJ/xLUw8GRd4gdxLYP49CQPAzn4lsUxTT3sEE
pDrj73RYQPAZEeco/g98iDP7/gOiL/hJMZM9QqEoJVqnlrAl+vKn8h8oLjX7x1mg
KBavMDVAwVVlxQuawEIT9hpbUstU2KY200h/mYd1zSrAdltF5V3jFWxyIDSeX4N+
pG6AV8bR7BauVV19E7NhAmZc83oxMwoL6fsEcOymGtVE6NIhxZgwcxhcNgrVrRqd
lMSUCc3amlczcnu2sQP/ZA6pwFgPxhWPEgLXKSp+mRt3ydoSCB1aplcHNBuzgwJs
ArvI+IS7CAp2SjPeP5uznYB79JhmRbAxxa7BO8LbKTMfx8qdQCl7EhxQk1jtVkGf
sVQsr6+QdHiVEd2XyIoJ
=HyU4
-----END PGP SIGNATURE-----

--==========4050975671BE119B3168==========--




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?141BC4CD56168688B595384F>