PEAR and make install on master

9 years ago by Matteo Beccati — view source

unread

Hi,

so, the removal of deprecated features has started and make install is
now rightfully failing because PEAR is installed by default, although it
is not compatible anymore:

Installing PEAR environment: /home/atlassian/phpNext/lib/php/

Fatal error: Non-static method PEAR_Installer_Role::initializeConfig()
cannot be called statically, assuming $this from incompatible context in
phar:///.../pear/install-pear-nozlib.phar/PEAR/Config.php on line 599
make[1]: *** [install-pear-installer] Error 255
make: *** [install-pear] Error 2

I've tried to look up the pear MLs, but I haven't seen any recent
activity... what's the plan, if any?

Cheers

Matteo Beccati

Development & Consulting - http://www.beccati.com/

9 years ago by Sebastian Bergmann — view source

unread

Am 18.01.2015 um 09:40 schrieb Matteo Beccati:

what's the plan, if any?

Remove it?

9 years ago by Derick Rethans — view source

unread

Am 18.01.2015 um 09:40 schrieb Matteo Beccati:

what's the plan, if any?

Remove it?

Then how would you install extensions? As there is no finished
alternative, PEAR still needs to work.

cheers,
Derick

9 years ago by Pierre Joye — view source

unread

Am 18.01.2015 um 09:40 schrieb Matteo Beccati:

what's the plan, if any?

Remove it?

Then how would you install extensions? As there is no finished
alternative, PEAR still needs to work.

There is, pickle. Pecl tgz or SRC install works out of the box already (or
from VCS), from pecl.php.net, etc.

Binaries install works too as well (windows).

The only reason we did not propose it yet is about which plan will be
chosen for php7 pecl releases and devs.

Cheers,
Pierre

9 years ago by Derick Rethans — view source

unread

Am 18.01.2015 um 09:40 schrieb Matteo Beccati:

what's the plan, if any?

Remove it?

Then how would you install extensions? As there is no finished
alternative, PEAR still needs to work.

There is, pickle. Pecl tgz or SRC install works out of the box already (or
from VCS), from pecl.php.net, etc.

I know what pickle is. But as it's not bundled, it's not "finished" -
exactly the reason why I used that word in my original reply. Please
read what I write.

Derick

9 years ago by Pierre Joye — view source

unread

Am 18.01.2015 um 09:40 schrieb Matteo Beccati:

what's the plan, if any?

Remove it?

Then how would you install extensions? As there is no finished
alternative, PEAR still needs to work.

There is, pickle. Pecl tgz or SRC install works out of the box already
(or
from VCS), from pecl.php.net, etc.

I know what pickle is. But as it's not bundled, it's not "finished" -
exactly the reason why I used that word in my original reply. Please
read what I write.

Please do read what I wrote.

It is finished for what pecl provides. For developers and end users, and
with more.

The only open question is about what we want to do to maintain code base
for 5.x and 7.x. Branches will be a pain. We need to solve that and you,
along many, did not answer this question.

Derick

9 years ago by Nikita Popov — view source

unread

Hi,

so, the removal of deprecated features has started and make install is now
rightfully failing because PEAR is installed by default, although it is not
compatible anymore:

Installing PEAR environment: /home/atlassian/phpNext/lib/php/

Fatal error: Non-static method PEAR_Installer_Role::initializeConfig()
cannot be called statically, assuming $this from incompatible context in
phar:///.../pear/install-pear-nozlib.phar/PEAR/Config.php on line 599
make[1]: *** [install-pear-installer] Error 255
make: *** [install-pear] Error 2

I've tried to look up the pear MLs, but I haven't seen any recent
activity... what's the plan, if any?

That's a good question. As far as I understand we still need PEAR because
it also provides the PECL installer. So it would be good if the installer
is fixed.

As an immediate action I would suggest to disable installation of PEAR by
default, given how it should be irrelevant to most PHP users who don't
install extensions via PECL.

Nikita

9 years ago by Pierre Joye — view source

unread

Hi,

so, the removal of deprecated features has started and make install is now
rightfully failing because PEAR is installed by default, although it is not
compatible anymore:

Installing PEAR environment: /home/atlassian/phpNext/lib/php/

Fatal error: Non-static method PEAR_Installer_Role::initializeConfig()
cannot be called statically, assuming $this from incompatible context in
phar:///.../pear/install-pear-nozlib.phar/PEAR/Config.php on line 599
make[1]: *** [install-pear-installer] Error 255
make: *** [install-pear] Error 2

I've tried to look up the pear MLs, but I haven't seen any recent
activity... what's the plan, if any?

That's a good question. As far as I understand we still need PEAR because
it also provides the PECL installer. So it would be good if the installer
is fixed.

For pecl, we will propose to bundle pickle.phar, which basically
provides 100% bc for existing packages and allow much more, like
install from any URL/VCS based on composer, integration with composer
(composer will call pickle for any php-ext dep). Last but not least,
if one likes to drop "pecl install" support, he won't need to maintain
package.xml, duplicate versions, meta and co but only the sources,
config.* and README.

I am waiting for the discussions+RFC about how to deal with 5 and 7
extensions support. Once we have a decision we will implement what is
necessary for that in pickle.

For the ref here: https://github.com/FriendsOfPHP/pickle

Cheers,

Pierre

@pierrejoye | http://www.libgd.org

9 years ago by Stanislav Malyshev — view source

unread

Hi!

Fatal error: Non-static method PEAR_Installer_Role::initializeConfig()
cannot be called statically, assuming $this from incompatible context in
phar:///.../pear/install-pear-nozlib.phar/PEAR/Config.php on line 599
make[1]: *** [install-pear-installer] Error 255
make: *** [install-pear] Error 2

I've tried to look up the pear MLs, but I haven't seen any recent
activity... what's the plan, if any?

Can't we just fix it? As I understand, it's just making this:

function initializeConfig(&$config)

into this:

static function initializeConfig(&$config)

In PEAR/Installer/Role.php but I'm not sure where to commit the patch or
how to get it into phars (they don't seem to be the latest code in git).
Anybody knows what's the right way here?

--
Stas Malyshev
smalyshev@gmail.com

9 years ago by Sebastian Bergmann — view source

unread

Am 19.01.2015 um 07:58 schrieb Stanislav Malyshev:

Can't we just fix it? As I understand, it's just making this:
function initializeConfig(&$config)
into this:
static function initializeConfig(&$config)
In PEAR/Installer/Role.php but I'm not sure where to commit the patch or
how to get it into phars (they don't seem to be the latest code in git).
Anybody knows what's the right way here?

I checked out pear/pear-core.git but cannot figure out how to run
the test suite. Don't want to commit a change without running the
tests ...

9 years ago by Xinchen Hui — view source

unread

Hey Pear core:

Am 19.01.2015 um 07:58 schrieb Stanislav Malyshev:
Can't we just fix it? As I understand, it's just making this:
function initializeConfig(&$config)
into this:
static function initializeConfig(&$config)
In PEAR/Installer/Role.php but I'm not sure where to commit the patch or
how to get it into phars (they don't seem to be the latest code in git).
Anybody knows what's the right way here?
I checked out pear/pear-core.git but cannot figure out how to run
the test suite. Don't want to commit a change without running the
tests ...

maybe you should involved in recent changes which could impact PEAR?

thanks

--

--
Xinchen Hui
@Laruence
http://www.laruence.com/

9 years ago by Christian Weiske — view source

unread

Hello Xinchen,

In PEAR/Installer/Role.php but I'm not sure where to commit the
patch or how to get it into phars (they don't seem to be the
latest code in git). Anybody knows what's the right way here?

I checked out pear/pear-core.git but cannot figure out how to run
the test suite. Don't want to commit a change without running the
tests ...

We have a "stable" branch that's up to date with the latest release
(1.9.5). You should use that to base your patch on.

--
Regards/Mit freundlichen Grüßen
Christian Weiske

-=≡ Geeking around in the name of science since 1982 ≡=

9 years ago by Ferenc Kovacs — view source

unread

On Mon, Jan 19, 2015 at 9:56 PM, Christian Weiske cweiske@cweiske.de
wrote:

Hello Xinchen,

In PEAR/Installer/Role.php but I'm not sure where to commit the
patch or how to get it into phars (they don't seem to be the
latest code in git). Anybody knows what's the right way here?

I checked out pear/pear-core.git but cannot figure out how to run
the test suite. Don't want to commit a change without running the
tests ...

We have a "stable" branch that's up to date with the latest release
(1.9.5). You should use that to base your patch on.

--
Regards/Mit freundlichen Grüßen
Christian Weiske

-=≡ Geeking around in the name of science since 1982 ≡=-

the issue is resolved for now: I've added a php7 compatible
pear/install-pear-nozlib.phar to the repo and opened a pull request with
the fixes to the pear guys to review/merge:
https://github.com/pear/pear-core/pull/33
as mentioned in another thread: this is just a temporary solution to fix
the immediate problem, and as discussed multiple times, PHP7 would be a
good time to revisit our strategy of bundling the pear/pecl installer with
the core.

--
Ferenc Kovács
@Tyr43l - http://tyrael.hu

9 years ago by Ferenc Kovacs — view source

unread

On Mon, Jan 19, 2015 at 4:07 PM, Sebastian Bergmann sebastian@php.net
wrote:

Am 19.01.2015 um 07:58 schrieb Stanislav Malyshev:
Can't we just fix it? As I understand, it's just making this:
function initializeConfig(&$config)
into this:
static function initializeConfig(&$config)
In PEAR/Installer/Role.php but I'm not sure where to commit the patch or
how to get it into phars (they don't seem to be the latest code in git).
Anybody knows what's the right way here?
I checked out pear/pear-core.git but cannot figure out how to run
the test suite. Don't want to commit a change without running the
tests ...

--

I've started working on it already (had to fix pear-core for php 5.5, so I
have a bit experience with it).

--
Ferenc Kovács
@Tyr43l - http://tyrael.hu