Bug#609228: libpadre-plugin-spellcheck-perl: Failing tests; Tries to create .padre directory for building user

[Salvatore Bonaccorso]

Hi Damyan!

On Sat, Jan 08, 2011 at 02:27:44PM +0200, Damyan Ivanov wrote:
> -=| Salvatore Bonaccorso, Fri, Jan 07, 2011 at 04:48:12PM +0100 |=-
> > Source: libpadre-plugin-spellcheck-perl
> > Version: 1.1.2-1
> > Severity: serious
> > Justification: FTBFS
> 
> Is this with sbuilder settings as on official buildds? Otherwise I'd 
> say it is  of severity:importatnt. (worth fixing in anu case)

I talked about that to the release team. We have for all of them with
arch:all anyway the squeeze-ignore tag, but they should be fixed at
least later on. When Lucas Nussbaum builds the archive, he generally
notes too such problems with unwritable homes IIRC. Official buildds
seems to have something like:

---(mount-defaults.buildd)----------------------------------------------
# mount.defaults: static file system information for chroots.
# Note that the mount point will be prefixed by the chroot path
# (CHROOT_PATH)
#
# <file system> <mount point>   <type>  <options>       <dump>  <pass>
proc            /proc           proc    defaults        0       0
/dev/pts        /dev/pts        none    rw,bind         0       0
tmpfs           /dev/shm        tmpfs   defaults        0       0
/home/buildd/build-trees                /build          none    rw,bind         0       0
------------------------------------------------------------------------


> > Building libpadre-plugin-spellcheck-perl 1.1.2-1 tries to create
> > ..padre directory, which fail:
> > 
> > > PERL_DL_NONLAZY=1 /usr/bin/perl "-MExtUtils::Command::MM" "-e" "test_harness(0, 'blib/lib', 'blib/arch')" t/*.t
> > > Use of uninitialized value $_[0] in join or string at /usr/lib/perl5/File/Spec/Unix.pm line 86.
> > > 
> > > #   Failed test 'use Padre::Plugin::SpellCheck;'
> > > #   at t/0-load.t line 16.
> > > #     Tried to use 'Padre::Plugin::SpellCheck'.
> > > #     Error:  mkdir /.padre: Permission denied at /usr/share/perl5/Padre/Constant.pm line 116
> 
> What would you recommend as a fix? I was thinking about runnung tests 
> in a wrapper that sets HOME to a temporary directory and then wipes it 
> out.

We have already such cases e.g. in libtest-valgrind-perl done by
gregoa. I have documented now at least that way in our debhelper.pod
[1].

 [1]  http://pkg-perl.alioth.debian.org/debhelper.html#running_tests_needing_writable_home

I tried to build with the attached patch, and it works fine now, so we
could apply a similar solution to all of them, what do you think agree
on that way? Or other ideas?

Bests
Salvatore
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 609228.patch
Type: text/x-diff
Size: 698 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/pkg-perl-maintainers/attachments/20110110/ab216886/attachment.patch>
-------------- 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-perl-maintainers/attachments/20110110/ab216886/attachment.pgp>