[Build-common-hackers] Bug#288806: Bug#288806: cdbs: support for perl modules that use Module::Build

[Peter Makholm]

Jonas Smedegaard <dr at jones.dk> writes:

> Looks sane to me, generally. But...
>
>> +ifneq ($(DEB_BUILDDIR),$(DEB_SRCDIR))
>> +$(error DEB_BUILDDIR and DEB_SRCDIR must be the same for Perl builds)
>> +endif
>
> If I understand Fr=C3=A9d=C3=A9ric correctly, the above restriction is not
> necessary when using Build directly. I haven't tested, so this is just
> speculation for now...

Wasn't sure about this either. So I just left it in for both systems.


> Another thing is worrying about existing packages - do we not risk
> breaking some current use of CDBS by changing this now?
>
> If so, it might be better to check for Build.PL but instead of setting
> DEB_PERL_BUILDSYSTEM only warn about it.

I would really hate having a long term solution prefering a
compatability layer over using Module::Build directly. It is a problem
that you can't easily override the test som it might be preferable to
make the setting

DEB_PERL_BUILDSYSTEM ?= $(shell test -e Build.PL && echo Module::Build)
DEB_PERL_BUILDSYSTEM ?= MakeMaker

Yes, exisitng packages may fail but it is easily fixed by setting
DEB_PERL_BUILDSYSTEM to MakeMaker (or anything but Module::Build).

//Makholm