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

Re: Packaging best practice questions (was: Re: Bug#764202: RFP: libiri-perl -- Perl module for Internationalized Resource Identifiers)



Quoting gregor herrmann (2014-10-12 03:19:38)
> On Sun, 12 Oct 2014 02:23:35 +0200, Axel Beckert wrote:
>
>> libiri-perl packaging is now pushed to the common location [..]

Great!


>> 1. IRI has some inc/Module/Install directory which looks a lot like 
>>    an embedded code copy, but is used in Makefile.PL. What's the best 
>>    practice there? Leave it or patch it to use some 
>>    libmodule-install-perl?
>
> Usually we just take a deep breath, sigh, and accept what's in inc/. 
> (In this specific case I'm also not sure if all modules are packaged.)

Right, that's a flaw in Module::Install to include mangled convenience 
code copies.  Module::Install is commonly used among those working with 
RDF, as it expresses its packaging metadata as RDF.  Its successor - 
Dist::Inkt - does not have that flaw but is not yet adopted broadly.

For a draft of what you could put in copyright file (which in my 
experience has been accepted by ftpmasters), have a look at e.g. 
libhtml-html5-microdata-parser-perl.


>> 2. IRI has an empty Changes file, but some XML files at Changes.ttl
>>    and Changes.xml. The former contains the real changelog data, but 
>>    hidden in less readable XML. Anyone knows which tools are needed 
>>    to un-XML-ify that changelog format?
>
> Ugh, this really looks ... interesting.
> Sorry, I have no idea about tools for sanifying those files.

That's an upstream bug:

The *.ttl file is not xml but Turtle, processed by 
Module::Install::DOAPChangeSets - but the author didn't place that file 
at its default location meta/changes.ttl and also didn't write more 
verbosely in Makefile.PL where it is - like this:

  write_doap_changes "Changes.ttl", "Changes", "turtle";
  write_doap_changes_xml "Changes.ttl", "Changes.xml", "turtle";

...and it seems there is something wrong with the data itself (or my 
limited knowledge of these tools) as this produces an empty Changes 
file:

  perl -Minc::Module::Install -MModule::Install::DOAPChangeSets -e 'write_doap_changes "Changes.ttl", "Changes", "turtle"'


I suggest you upload now, without upstream changelog, to have it queued 
for ftpmaster approval while getting upstream to fix that issue.


 - Jonas

-- 
 * Jonas Smedegaard - idealist & Internet-arkitekt
 * Tlf.: +45 40843136  Website: http://dr.jones.dk/

 [x] quote me freely  [ ] ask before reusing  [ ] keep private

Attachment: signature.asc
Description: signature


Reply to: