[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#377348: [RFS] php5-xdebug



Having a quick look at this, my first thoughts are that I hate cdbs ;)

Second of all, there seems to be a couple of weird things in there - for
example

./configure --enable-xdebug --with-php-config=/usr/bin/php-config$*

That line is simply going to make them use php-config. While the B-D's
mean that this will use php-config5, because php4 isn't installed, but
it if were, then it would be up to alternatives as to which version to
use.

I also noticed

configure_for_php%:

Which I assume is just a typo, and should be configure_for_php5 (as this
is what is used as a dependency for install/php5-xdebug)

Also, why ucf is listed as a dep for the source I don't know, as that
should be bought in by the dependencies of phpapi-`php-config5 --phpapi`
(php5-cli, php5-cgi, libapache2-mod-php5 all depend on it)

Also, I'd like to see that upon installation, xdebug is enabled, instead
of having to jump through hoops to get it working (yes, removing it
without a purge is a pain, but the postrm can sort that out anyway!)

also, the xdebug.ini should be using zend_extension instead of whatever
it's using.

Looking at the ubuntu package, I'm not too happy with a few of the
things in that, reading through it just doesn't seem clear to me (some
of the code in the rules is completely confusing - and they replace the
config.guess and config.sub for the client, even though it isn't built
(I guess to get rid of the lintian error - an override would have done)

I'm going to do a little work on the package, and conglomerate stuff
from all three packages into one package combining everything nicely.

And then I'll have to uupdate anyway (Thanks Derick! :P)	




Reply to: