Andreas Tille:
> Package: debhelper
> Severity: normal
> 
> Hi,
> 
> the package dh-r contains a script dh-make-R which creates a debian/
> dir.  This script calls an R module which contains:
> 
>    use base 'Debian::Debhelper::Buildsystem';
> 
> Since debhelper 11.2 I get:
> 
> $ dh-make-R 
> [...]
> 
> 
> I went versions from snapshot.d.o down to 11.1.6 which works nicely and
> does not show this error.  Could you please revert the change from
> debhelper 11.1.6 to 11.2 which makes the existence of debian/control
> required?
> 
> Thanks a lot for maintaining debhelper
> 
>       Andreas.
> 
> 
> [...]
Hi Andreas,

I am sorry to hear that debhelper/11.2 broke dh-make-R.

It is not going to be trivial as trivial as I would like to "undo" this
issue on the debhelper side.  I would basically have to revert the
support for "cmake+ninja" (etc.) or lock debhelper into never using
compat at load/"compile" time of the Buildsystem class hierarchy (which
will require some needlessly complicated tricks and I fear it will grow
unwieldy over time).

I have then analysed dh-make-R and it turns out that dh-make-R (and
dh-update-R) only needs the "parse_depends" function from
"Debian::Debhelper::Buildsystem::R".  Said function seems like it can
easily be extracted into a separate Perl package and imported from there
into dh-make-R, dh-update-R and "Debian::Debhelper::Buildsystem::R".

 * Would this be an option for you?

This would avoid hacks on my side and make dh-make-R + dh-update-R less
sensitive to "unrelated" changes to the Debhelper buildsystem ABI.

Thanks,
~Niels

PS: The "use Debian::Debhelper::Buildsystem::R qw(parse_deps)" line in
dh-make-R and dh-update-R appear to reference a non-existing an unused
parse_deps instead of parse_depends.

Reply via email to