[php-maint] PHPUnit testers wanted (2nd query)

[Arthur Gautier]

On Thu, Mar 03, 2011 at 01:47:50AM +0800, Thomas Goirand wrote:
> On 03/02/2011 11:35 PM, Arthur Gautier wrote:
> > I switched debian/rules to cdbs, there is no configure target anymore
> 
> Outch ! That was really not needed.
> 
> I said it previously, but I will say it again: if you switch to CDBS,
> then I wont review any package, and I don't think you'll be able to find
> anyone here that will want to review packaging (you can ask others if
> you want still, but my bet is that they wont be able to help). Here's why.
> 
> Considering the issues found using debhelper, I would wild guess that
> you know even less about CDBS that I do. There's a lot to know about
> CDBS before you can really use it, you can't just "hope" that it will
> work. The authors of CDBS themselves are saying that it's for experts at
> least (you can ask Jonas that maintains it if you want). So in short,
> unless you can say that you know CDBS very well, and don't just trust it
> blindly, don't use it.
> 
> Of course, it's your choice of maintainer. The only thing is that I will
> not review anything because I'm not confident for my skills with CDBS
> (and I believe that *many* DD have the same reaction).
>
There we go, few changes i'd just committed to alioth removes cdbs dependency
and gets back to a more regular packaging. 
Changes done could be seen here:
  - http://git.debian.org/?p=pkg-php/php-token-stream.git
  - http://git.debian.org/?p=pkg-php/php-text-template.git
  - http://git.debian.org/?p=pkg-php/php-codecoverage.git
  - http://git.debian.org/?p=pkg-php/php-timer.git

> [...]
> >>> - You shouldn't be using debian/install, but pear to install stuff.
> >>> Something like this would do:
> >>>
> >>> $(PEAR) install -f -n -R debian/$(package) $(pear_pkg)/package.xml
> >>>
> > To work around pear.phpunit.de channel issue, i needed to change the
> > php_dir options in pear so i can't use pear install.
> 
> Why would you need to change php_dir ??? Shouldn't the PEAR modules that
> you are installing go in the same folder? And by the way, pear
> config-get and pear config-set should be enough for you to work with if
> you needed to.
Because phpunit dependencies uses their own pear channel, and we need to
have discovered it in our environment. I can't modify system environment,
and don't want to modify user environment neither. So i have to create a
new environment for packaging.


> 
> > By the way, one should use $(PEAR) install -P (and not -R) as describe
> > in the pear documentation.
> 
> Interesting. Where did you read that? URL?
Here is an extract of pear help install :
  -R DIR, --installroot=DIR
        root directory used when installing files (ala PHP's INSTALL_ROOT), use packagingroot for RPM
  -P DIR, --packagingroot=DIR
        root directory used when packaging files, like RPM packaging

I'm not using rpm, but -P plays well with debian too :)
> > Could you please retest again packages ?
> 
> Nop. CDBS... Sorry for that.
Now cdbs is out. May i ask you to retest them ?

Thanks
-- 
Arthur Gautier
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.alioth.debian.org/pipermail/pkg-php-maint/attachments/20110303/2d7cfe34/attachment-0001.pgp>