Bug#823067: dh-make-perl: please split Debian::Control into a separate binary package

[gregor herrmann]

On Sun, 01 May 2016 13:37:25 +0000, Damyan Ivanov wrote:

> > -rw-r--r-- root/root      9026 2016-04-11 23:04 ./usr/share/perl5/DhMakePerl/Config.pm
> > -rw-r--r-- root/root      7381 2016-04-11 23:04 ./usr/share/perl5/DhMakePerl/Utils.pm
> > -rw-r--r-- root/root      1247 2016-04-11 23:04 ./usr/share/perl5/DhMakePerl/Command/refresh_cache.pm
> > -rw-r--r-- root/root      6695 2016-04-11 23:04 ./usr/share/perl5/DhMakePerl/Command/refresh.pm
> > -rw-r--r-- root/root      1119 2016-04-11 23:04 ./usr/share/perl5/DhMakePerl/Command/dump_config.pm
> > -rw-r--r-- root/root     49429 2016-04-11 23:04 ./usr/share/perl5/DhMakePerl/Command/Packaging.pm
> > -rw-r--r-- root/root     27340 2016-04-11 23:04 ./usr/share/perl5/DhMakePerl/Command/make.pm
> > -rw-r--r-- root/root      2159 2016-04-11 23:04 ./usr/share/perl5/DhMakePerl/Command/locate.pm
> > -rw-r--r-- root/root      1312 2016-04-11 23:04 ./usr/share/perl5/DhMakePerl/Command/help.pm
> > -rw-r--r-- root/root      4508 2016-04-11 23:04 ./usr/share/perl5/DhMakePerl/PodParser.pm
> > -rw-r--r-- root/root      2737 2016-04-11 23:04 ./usr/share/perl5/DhMakePerl.pm
> > 
> > -rw-r--r-- root/root     11784 2016-04-11 23:04 ./usr/share/perl5/Debian/AptContents.pm
> > -rw-r--r-- root/root     10725 2016-04-11 23:04 ./usr/share/perl5/Debian/Rules.pm
> > -rw-r--r-- root/root      2899 2016-04-11 23:04 ./usr/share/perl5/Debian/WNPP/Bug.pm
> > -rw-r--r-- root/root      5123 2016-04-11 23:04 ./usr/share/perl5/Debian/WNPP/Query.pm
> > -rw-r--r-- root/root     12697 2016-04-11 23:04 ./usr/share/perl5/Debian/Dependency.pm
> > -rw-r--r-- root/root      7084 2016-04-11 23:04 ./usr/share/perl5/Debian/Dependencies.pm
> > -rw-r--r-- root/root      4523 2016-04-11 23:04 ./usr/share/perl5/Debian/DpkgLists.pm
> > 
> > -rw-r--r-- root/root      5199 2016-04-11 23:04 ./usr/share/perl5/Debian/Control.pm
> > -rw-r--r-- root/root      2209 2016-04-11 23:04 ./usr/share/perl5/Debian/Control/Stanza/CommaSeparated.pm
> > -rw-r--r-- root/root      2592 2016-04-11 23:04 ./usr/share/perl5/Debian/Control/Stanza/Source.pm
> > -rw-r--r-- root/root      4551 2016-04-11 23:04 ./usr/share/perl5/Debian/Control/Stanza/Binary.pm
> > -rw-r--r-- root/root     19102 2016-04-11 23:04 ./usr/share/perl5/Debian/Control/FromCPAN.pm
> > -rw-r--r-- root/root      7328 2016-04-11 23:04 ./usr/share/perl5/Debian/Control/Stanza.pm
> > 
> > The DhMakeperl::* namespace seems clear; and then we have Debian::Control* but also
> > other Debian::* modules, where we'd need to check high tightly they
> > are tied into dh-make-perl or how generic and usuable they are. (And
> > if we split out more, we need to think about a different name than
> > libdebian-control-perl).
> 
> Debian::Control::FromCPAN uses DhMakePerl::Utils and sometimes 
> Debian::DpkgLists, so perhaps this belongs to dh-make-perl.
> 
> Debian::Control::Stanza uses Debian::Dependencies, I guess the later 
> shall go in the separate package.
> 
> > Sean, do you need only Debian::Control::* o more?
> > 
> > Dam and others, what are your thoughts from the dh-make-perl
> > knowledge side?
> 
> No objections from me. As suggested by the names, the modules were 
> intended to be usable outside of dh-make-perl. 
> Debian::Control::FromCPAN is the only exception that I see.

I found another one:
    lib/Debian/AptContents.pm:use DhMakePerl::Utils qw(find_core_perl_dependency);
and it's (currently) also rather perl-specific.

I've now pushed a branch "split-package" where I, well, started the
split.

What remains to be done, or "Questions and ToDos":
- check which file goes into which package
- short/long description for the new package
- package name for the new package
- check runtime dependencies for the new package (enough?) and the
  old one (probably too many)
- maybe use wildcards in *.install


Cheers,
gregor, waving from the Debian Perl Sprint and waiting for comments    


-- 
 .''`.  Homepage https://info.comodo.priv.at/ - OpenPGP key 0xBB3A68018649AA06
 : :' : Debian GNU/Linux user, admin, and developer -  https://www.debian.org/
 `. `'  Member of VIBE!AT & SPI, fellow of the Free Software Foundation Europe
   `-   
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 949 bytes
Desc: Digital Signature
URL: <http://lists.alioth.debian.org/pipermail/pkg-perl-maintainers/attachments/20160522/cf52f8e6/attachment.sig>