Bug#610509: fai: Build dependencies missing some packages in svn HEAD

2011-01-19 Thread Thomas Lange
> On Wed, 19 Jan 2011 10:57:50 +, Michael Tautschnig  
> said:

> If we prefer not to have that build dependency but still keep the syntax 
checks
> we could also add dummy files (which is what I did on alioth to make the
> experimental versions build).
What if we remove the build dependency, leave perlcheck in the
Makefile, but do not call it on every package build? That way, one
could manually call the perlchecks when he knows the needed packages
are installed, but not everybody needs to install the additional
packages when building the package.

-- 
regards Thomas



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org



Bug#610509: fai: Build dependencies missing some packages in svn HEAD

2011-01-19 Thread Michael Tautschnig
> > On Wed, 19 Jan 2011 10:37:48 +, Michael Tautschnig 
> >  said:
> 
> > I just fixed most of the above this morning in trunk, with any version 
> >= r6270
> > you should be fine. Well, honestly, the perl-tk isn't there yet, thanks 
> for
> > adding it. Should get fixed soon.
> 
> 
> svn r6271 build fine on lenny and squeeze. perl-tk is not needed in
> any build depends!
> 

Try removing it and you'll learn that it is - well, not quite the actual code,
but at least some file named after the actual perl modules being used. It's
needed for the very useful perl syntax checks that Mika recently added. 

If we prefer not to have that build dependency but still keep the syntax checks
we could also add dummy files (which is what I did on alioth to make the
experimental versions build).

Best regards,
Michael



pgpRRgtCAGTP1.pgp
Description: PGP signature


Bug#610509: fai: Build dependencies missing some packages in svn HEAD

2011-01-19 Thread Thomas Lange
> On Wed, 19 Jan 2011 10:37:48 +, Michael Tautschnig  
> said:

> I just fixed most of the above this morning in trunk, with any version >= 
r6270
> you should be fine. Well, honestly, the perl-tk isn't there yet, thanks 
for
> adding it. Should get fixed soon.


svn r6271 build fine on lenny and squeeze. perl-tk is not needed in
any build depends!

-- 
regards Thomas



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org



Bug#610509: fai: Build dependencies missing some packages in svn HEAD

2011-01-19 Thread Michael Tautschnig
> Package: fai
> Version: 4.0.0beta2
> Justification: no longer builds from source
> Severity: normal
> 
> Hi,
> 
> I had two problems when building fai from SVN (r6265) on a ubuntu lucid box:
> 
> 1) first some build deps are missing, see patch below
> 
> 2) during buildpackage perl choked about not finding 
>setup-storage/Init.pm, which I fixed by setting
>export PERL5LIB=.../fai-sources/lib/setup-storage/
>but since I was never a perl hacker, I have no clue
>how to patch that into the package.
> 

[...]

:-)

I just fixed most of the above this morning in trunk, with any version >= r6270
you should be fine. Well, honestly, the perl-tk isn't there yet, thanks for
adding it. Should get fixed soon.

The PERL5LIB thingy is better fixed using a -I command line option, which I
have added as well.

So, consider most of this fixed. But thanks a lot for reporting it and the
perl-tk is great to know, I hadn't yet had the time to figure out which one was
the proper package.

Thanks a lot,
Michael



pgpTg5shfCh3Z.pgp
Description: PGP signature


Bug#610509: fai: Build dependencies missing some packages in svn HEAD

2011-01-19 Thread Steffen Neumann
Package: fai
Version: 4.0.0beta2
Justification: no longer builds from source
Severity: normal

Hi,

I had two problems when building fai from SVN (r6265) on a ubuntu lucid box:

1) first some build deps are missing, see patch below

2) during buildpackage perl choked about not finding 
   setup-storage/Init.pm, which I fixed by setting
   export PERL5LIB=.../fai-sources/lib/setup-storage/
   but since I was never a perl hacker, I have no clue
   how to patch that into the package.

Yours,
Steffen

sneumann@geweniger:/vol/debian/fai/fai-sources $ svn diff
Index: debian/control
===
--- debian/control  (revision 6265)
+++ debian/control  (working copy)
@@ -4,7 +4,7 @@
 Maintainer: Thomas Lange 
 Uploaders: Michael Prokop 
 Standards-Version: 3.9.1.0
-Build-Depends-Indep: asciidoc, asciidoc (>= 8.3) | lynx, w3m, dblatex, 
docbook-xsl, libxml2-utils
+Build-Depends-Indep: asciidoc, asciidoc (>= 8.3) | lynx, w3m, dblatex, 
docbook-xsl, libxml2-utils, libparse-recdescent-perl, liblinux-lvm-perl, 
perl-tk 
 Build-Depends: debhelper (>= 7)
 Vcs-Svn: svn://svn.debian.org/svn/fai/trunk
 Vcs-Browser: http://svn.debian.org/wsvn/fai



bin/install_packages syntax OK
Can't locate Init.pm in @INC (@INC
contains: /usr/share/fai/setup-storage/ /home/neumanns/perl 
/home/neumanns/perl/share/perl/5.8.3/ /etc/perl /usr/local/lib/perl/5.10.1 
/usr/local/share/perl/5.10.1 /usr/lib/perl5 /usr/share/perl5 /usr/lib/perl/5.10 
/usr/share/perl/5.10 /usr/local/lib/site_perl .) at bin/setup-storage line 89.


-- 
IPB HalleAG Massenspektrometrie & Bioinformatik
Dr. Steffen Neumann  http://www.IPB-Halle.DE
Weinberg 3   http://msbi.bic-gh.de
06120 Halle  Tel. +49 (0) 345 5582 - 1470
  +49 (0) 345 5582 - 0
sneumann(at)IPB-Halle.DE Fax. +49 (0) 345 5582 - 1409




-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org