Newsgroups: php.internals,php.pecl.dev
Path: news.php.net
Xref: news.php.net php.internals:69551 php.pecl.dev:11191
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: unknown (pb1.pair.com: domain php.net does not designate 85.214.73.107 as permitted sender)
To: pecl-dev@lists.php.net, internals@lists.php.net
Content-Type: text/plain; charset="UTF-8"
Date: Mon, 14 Oct 2013 13:34:50 +0200
Message-ID: <1381750491.29722.335.camel@ghost>
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: PECL site improvements
From: ab@php.net (Anatol Belski)

Hi,

there are two improvements just implemented to the PECL website.

I

Every package is required to have LICENSE or COPYING to be present in
the root of the package. The goal is to enforce the conformance with at
least PHP and BSD licenses. Where by no explicit check for exact license
type is done, it's considered as a good practice to include the license.
Also, the licenses will be included into the windows binary packages. To
add that file to the tarball, just add a line like this to the root dir
in the package.xml

<file name="LICENSE" role="doc" />

II

Every package is required to have the macros PHP_MYEXTNAME_VERSION to be
defined in one of the headers in the root of the package. The goal is to
fix discrepancy between phpinfo() and PECL site extension versions
shown. I guess it happened almost to everyone at least once to
unintendingly forget to sync both version strings before the release.

The actual macros name is guessed using <name> and <providesextension>
tags from package.xml. The common definition format is

#define PHP_MYEXTNAME_VERSION "1.2.3"

That macros should be then used in the zend_module_entry defining the
extension module. The macros name has to match the exact pattern, not
MYEXTNAME_VERSION, MYEXTNAME_VERSION_STRING, MYEXT_VERSION_STR or else
because there can be other extension specific macros defined and could
possibly lead to ambiguity. Say the ext can have PHP_MYEXT_VERSION_STR
and PHP_MYEXT_VERSION_ID, so then it's hard to guess.

I'm going to walk through the extensions and fix this definitions in the
next days, but if you'll need to release before it's done, please fix it
yourself.


That's it, please ping me if you think there's a bug in one of those
changes.

Regards

Anatol