Bug#813904: lintian: Check if upstream/metadata files are well formed

2016-02-07 Thread Jakub Wilk
Hi Petter! IANA Lintian expert, but here's my review: * Petter Reinholdtsen , 2016-02-06, 15:59: +Needs-Info: unpacked, diffstat, file-info, md5sums AFAICS, you only need "unpacked" for this check. +Info: This script checks the upstream/metadata file for problems. ... is

Bug#813904: lintian: Check if upstream/metadata files are well formed

2016-02-07 Thread Petter Reinholdtsen
[Jakub Wilk] > Hi Petter! Hi. Thank you very much for taking the time to review the code. It is very valuable. > AFAICS, you only need "unpacked" for this check. Right. Fixed.n > ... is not strictly necessary. The only consumer of Info from > the first stanza is

Bug#813904: lintian: Check if upstream/metadata files are well formed

2016-02-07 Thread Niels Thykier
Petter Reinholdtsen: > [Jakub Wilk] >> Hi Petter! > > Hi. Thank you very much for taking the time to review the code. It is > very valuable. > Hi, Thanks for working on this and to Jakub for the excellent review! :) >> [...] >>> +use YAML; >> >> Linian already uses YAML::XS in

Bug#813904: lintian: Check if upstream/metadata files are well formed

2016-02-07 Thread Petter Reinholdtsen
[Niels Thykier] > Thanks for working on this and to Jakub for the excellent review! :) Yes, very good. I hope to solve a real problem for Debian Med, where the package maintainers do not notice the upstream/metadata file is broken before the upload and a cron job start failing when the blends

Bug#813904: lintian: Check if upstream/metadata files are well formed

2016-02-06 Thread Petter Reinholdtsen
Package: lintian Version: 2.5.40.2 Tags: patch The Debian Med project is using the upstream/metadata YAML files to store bibliographical references and other stuff, and would like a lintian check to verify that the files are well formed. Here is a draft patch for doing so. Is this the correct