[Pkg-bazaar-maint] Proposed change to bzr debian/rules to run more tests

[Jelmer Vernooij]

Hi Max,

On Wed, 2010-11-24 at 00:19 +0000, Max Bowsher wrote:
> I am proposing that in the bzr package, we change:
> 
> common-post-build-indep::
>     ./bzr selftest --no-plugins
> 
> to
> 
> common-post-build-arch::
>     PYTHONPATH=`pwd`/`python -c 'import sys, distutils.util; print
> "build/lib.%s-%s" % (distutils.util.get_platform(), sys.version[0:3])'` \
>                `pwd`/`python -c 'import sys; print "build/scripts-%s" %
> (sys.version[0:3],)'`/bzr selftest --no-plugins
> 
> .
> 
> The problem with the existing invocation is that it tests bzr without
> any of the compiled extensions, so does not truly match what we go on to
> install.
> 
> My proposed change is fairly horrid looking - more elegant ways of
> achieving this are gratefully accepted.
I like the idea of running the tests in an environment that's closer to
what is going to end up in the package.

However, I'm not sure the risk of adding this (and getting it wrong) is
worth the gain. If we run the tests from the build directory then that
means we'll use the extensions but it'll still be different from what
actually gets installed. And we don't run the tests of the plugins that
are bundled yet.

I'd prefer to just run the tests out of debian/tmp, so from the install
prefix and including the plugins, etc. 

Cheers,

Jelmer
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part
URL: <http://lists.alioth.debian.org/pipermail/pkg-bazaar-maint/attachments/20101124/1c3a2b12/attachment.pgp>