Daniel O'Connor | 20 Apr 2012 17:51
Picon
Gravatar

[PEPr] Proposal for PEAR::PEAR_Exception

Daniel O'Connor (http://pear.php.net/user/doconnor) proposes PEAR::PEAR_Exception.

You can find more detailed information here:
 http://pear.php.net/pepr/pepr-proposal-show.php?id=679

--

-- 
PEAR Development Mailing List (http://pear.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Markus Tacker | 20 Apr 2012 17:57
Picon

[PEPr] Comment on PEAR::PEAR_Exception

> Suggest someone else do a release of the pear installer with an
appropriate explanation of how to fix any upgrade problems (pear upgrade
pear && pear install PEAR_Exception)

Couldn't this just be done automatically by depending PEAR on
PEAR_Exception?

-- 
http://pear.php.net/pepr/pepr-proposal-show.php?id=679

--

-- 
PEAR Development Mailing List (http://pear.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Till Klampaeckel | 20 Apr 2012 18:26
Picon

[PEPr] Comment on PEAR::PEAR_Exception

I agree with this. :) Thanks for putting up this proposal!

-- 
http://pear.php.net/pepr/pepr-proposal-show.php?id=679

--

-- 
PEAR Development Mailing List (http://pear.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Brett Bieber | 20 Apr 2012 18:30
Picon
Gravatar

[PEPr] Comment on PEAR::PEAR_Exception

Very big +1 from me.

-- 
http://pear.php.net/pepr/pepr-proposal-show.php?id=679

--

-- 
PEAR Development Mailing List (http://pear.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Daniel O'Connor | 20 Apr 2012 18:39
Picon
Gravatar

[PEPr] Comment on PEAR::PEAR_Exception

>> Suggest someone else do a release of the pear installer with an
appropriate explanation of how to fix any upgrade problems (pear upgrade
pear && pear install PEAR_Exception)

> Couldn't this just be done automatically by depending PEAR on
PEAR_Exception?

Well, yeah, but I'd have to never set PEAR_Exception's min php version to 5
unfortunately!

I know we've had the discussion a few times around "who uses PHP4 anymore,
anyway?" - but no one has made the call.

This is just one more piece being chipped away.

-- 
http://pear.php.net/pepr/pepr-proposal-show.php?id=679

--

-- 
PEAR Development Mailing List (http://pear.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Picon
Gravatar

[PEPr] Comment on PEAR::PEAR_Exception

I think this RFC is a bit misguided.

I have had this on my plans for a long time but never found the time - I
would love if someone took on this initiative.

The package xml for this is already ready:
https://github.com/pear/pear-core/blob/master/package-Exception.xml - for
the most part.

It didn't help that PEAR had deps on PHP 4, so the only viable option was
to make PEAR_Exception depend on PHP 4 as well. That way PEAR could depend
on PEAR_Exception and not the other way around, thus on the new PEAR update
people would get PEAR_Exception.

If we move PEAR it self to have a min dep on PHP 5 (5.1.6 or 5.2 perhaps)
then PEAR_Exception can dep on PHP 5 as well and everyone is happy. No
confusion.

Package authors can them self do releases (or QA) with a new dep on
PEAR_Exception instead of PEAR, but it won't be required to do a mass
updated on all packages.

I want to see this done but not in the confusing manner proposed.

-- 
http://pear.php.net/pepr/pepr-proposal-show.php?id=679

--

-- 
PEAR Development Mailing List (http://pear.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php
(Continue reading)

Till Klampaeckel | 20 Apr 2012 19:11
Picon

[PEPr] Comment on PEAR::PEAR_Exception

For myself this is not about a PHP4 dependency, it's about all the packages
I have to install when a package depends on "PEAR" just for the exception.

So for example, I depend on HTTP_Request2 which uses PEAR_Exception, in
return I have the following packages installed along with it:

 * xml_util (1.2.1)

 * structures_graph (1.0.4)

 * pear_frontend_web (0.4)

 * pear_frontend_gtk (0.4.0)

 * pear (1.9.4)

 * console_getopt (1.3.1)

 * archive_tar (1.3.10)

Strictly speaking, they are not required for HTTP_Request2, but they are
installed because of the PEAR dependency.

-- 
http://pear.php.net/pepr/pepr-proposal-show.php?id=679

--

-- 
PEAR Development Mailing List (http://pear.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

(Continue reading)

Daniel O'Connor | 20 Apr 2012 19:40
Picon
Gravatar

[PEPr] Comment on PEAR::PEAR_Exception

ab675ef..ae590f3 should be the last few commits to bring back that
package-Exception.xml as package.xml, and make it valid (todo: kill
package-Exception.xml from the pear-core repo)

-- 
http://pear.php.net/pepr/pepr-proposal-show.php?id=679

--

-- 
PEAR Development Mailing List (http://pear.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Till Klampaeckel | 28 Apr 2012 23:43
Picon

[PEPr] Comment on PEAR::PEAR_Exception

Can we vote?

-- 
http://pear.php.net/pepr/pepr-proposal-show.php?id=679

--

-- 
PEAR Development Mailing List (http://pear.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Daniel O'Connor | 29 Apr 2012 14:42
Picon
Gravatar

[PEPr] Call for votes on PEAR::PEAR_Exception

Daniel O'Connor (http://pear.php.net/user/doconnor) has initiated the call for votes on PEAR::PEAR_Exception.

Please review the proposal and give your vote here:
http://pear.php.net/pepr/pepr-proposal-show.php?id=679

--

-- 
PEAR Development Mailing List (http://pear.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Markus Tacker | 29 Apr 2012 14:46
Picon

[PEPr] +1 for PEAR::PEAR_Exception

Markus Tacker (http://pear.php.net/user/tacker) has voted +1 on the proposal for PEAR::PEAR_Exception.

Proposal information:
http://pear.php.net/pepr/pepr-proposal-show.php?id=679
Vote information:
http://pear.php.net/pepr/pepr-vote-show.php?id=679&handle=tacker

--

-- 
PEAR Development Mailing List (http://pear.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Bertrand Mansion | 29 Apr 2012 17:20

[PEPr] +1 for PEAR::PEAR_Exception

Bertrand Mansion (http://pear.php.net/user/mansion) has voted +1 on the proposal for PEAR::PEAR_Exception.

Proposal information:
http://pear.php.net/pepr/pepr-proposal-show.php?id=679
Vote information:
http://pear.php.net/pepr/pepr-vote-show.php?id=679&handle=mansion

--

-- 
PEAR Development Mailing List (http://pear.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Christian Weiske | 29 Apr 2012 20:08
Picon
Gravatar

[PEPr] +1 for PEAR::PEAR_Exception

Christian Weiske (http://pear.php.net/user/cweiske) has voted +1 on the proposal for PEAR::PEAR_Exception.

Proposal information:
http://pear.php.net/pepr/pepr-proposal-show.php?id=679
Vote information:
http://pear.php.net/pepr/pepr-vote-show.php?id=679&handle=cweiske

This vote is conditional. The condition is:

Change the signature to the one of the PHP Exception base class:

>string $message = "" [, int $code = 0 [, Exception $previous = NULL ]]]

Currently, it's PEAR_Exception($message, Exception $prev, $code) which is diametral to the order in Exception.

Also, " <at> version CVS: $Id$" isn't really up to date.

--

-- 
PEAR Development Mailing List (http://pear.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Greg Sherwood | 30 Apr 2012 01:06
Gravatar

[PEPr] +1 for PEAR::PEAR_Exception

Greg Sherwood (http://pear.php.net/user/squiz) has voted +1 on the proposal for PEAR::PEAR_Exception.

Proposal information:
http://pear.php.net/pepr/pepr-proposal-show.php?id=679
Vote information:
http://pear.php.net/pepr/pepr-vote-show.php?id=679&handle=squiz

--

-- 
PEAR Development Mailing List (http://pear.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Till Klampaeckel | 30 Apr 2012 16:42
Picon

[PEPr] +1 for PEAR::PEAR_Exception

Till Klampaeckel (http://pear.php.net/user/till) has voted +1 on the proposal for PEAR::PEAR_Exception.

Proposal information:
http://pear.php.net/pepr/pepr-proposal-show.php?id=679
Vote information:
http://pear.php.net/pepr/pepr-vote-show.php?id=679&handle=till

--

-- 
PEAR Development Mailing List (http://pear.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Alexey Borzov | 3 May 2012 20:36
Picon
Favicon

[PEPr] +1 for PEAR::PEAR_Exception

Alexey Borzov (http://pear.php.net/user/avb) has voted +1 on the proposal for PEAR::PEAR_Exception.

Proposal information:
http://pear.php.net/pepr/pepr-proposal-show.php?id=679
Vote information:
http://pear.php.net/pepr/pepr-vote-show.php?id=679&handle=avb

This vote is conditional. The condition is:

Please do the split in the way that will not require mass package updates and manual mucking around.

Maybe start a PEAR 1.10.x major version with dependency on PHP5 and PEAR_Exception, making 1.9.x branch
the last PHP4-friendly one with occasional backports of fixes from 1.10.x

--

-- 
PEAR Development Mailing List (http://pear.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Michael Gauthier | 3 May 2012 21:48
Favicon
Gravatar

[PEPr] +1 for PEAR::PEAR_Exception

Michael Gauthier (http://pear.php.net/user/gauthierm) has voted +1 on the proposal for PEAR::PEAR_Exception.

Proposal information:
http://pear.php.net/pepr/pepr-proposal-show.php?id=679
Vote information:
http://pear.php.net/pepr/pepr-vote-show.php?id=679&handle=gauthierm

Comment:

If I understand correctly, this is just splitting the existing class into a
separate package.

This proposal does not address the larger problem of having a single point
of class inheritance for exceptions, rather than using interfaces and
extending appropriate SPL exception classes.

I'm fine with that (this package is solving a separate issue), but think it
should be clarified.

--

-- 
PEAR Development Mailing List (http://pear.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php


Gmane