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

Re: Bug#423807: libsvn-notify-mirror-perl: FTBFS: Can't locate SVN/Notify.pm in @INC

On Tue, May 15, 2007 at 06:50:54PM +0200, gregor herrmann wrote:

> On Mon, 14 May 2007 19:02:43 -0500, Gunnar Wolf wrote:
> > > Heh, strange, I didn't have neither File::Rsync nor Net::SSH installed
> > > on my machine! Anyway, I have to believe you, and right now I'm busy
> > > to start debugging - Building and uploading ;-)
> > Grmbl... With your packages, the module requires human intervention to
> > be built (there are some questions regarding whether to test Net::SSH
> > and File::Rsync). 
> Hm, in my pbuilder chroot they are answered automagically.

Right. If File::Rsync or Net::SSH are installed, Build.PL will
ask whether to run the corresponding tests. If there's no terminal,
as when building with pbuilder, the default answer, 'no', will be 

These tests need an ssh connection to localhost, so they are not
appropriate when autobuilding. Currently, the rsync tests aren't actually
enabled even when answering 'yes' due to a bug in the Debian modification
to Build.PL.

So, I think we should leave libnet-ssh-perl and libfile-rsync-perl out of
the build dependencies. The build system will still prompt when building
interactively and the modules are present, but we can work around
this by running 'perl Build.PL < /dev/null' from debian/rules and
preserve the interactive behaviour for those building 'manually'.

> t/001_basic....Cannot find sendmail and no "smtp" parameter specified at /usr/share/perl5/SVN/Notify.pm line 644.

This happens because SVN::Notify::new() wants a sendmail even if it's
not going to use it (which is the case here.) I had success with faking

    SENDMAIL=/dev/null $(PERL) Build test

I just committed these changes into the SVN repository; could some
DD please have a look and upload?

Niko Tyni   ntyni@iki.fi

Reply to: