Hi all.
Il 03/07/2010 18:40, Giovanni Mascellani ha scritto:
> Il 03/07/2010 18:06, Iain Lane ha scritto:
>> Done, but I couldn't build as it relies on hslogger-prof, which I can't
>> get as uploads aren't being processed this weekend*. But you can build
>> against it using your local copies and upload.
>
> Wonderful, I'll look at it this evening (UTC+0200). These will be my
> first two sponsored packages, so I may take some while to do all the
> checks. ;-)
I had a look at yours packages and some comments. Anyway, about some of
them I'm not completely sure, so I ask other more aged DD to confirm
them or not.
missingh:
* source/format: you're setting it to 1.0; are there any specific
reason for not use 3.0 (quilt)? If not, I'd say 3.0 it's more recommended.
* control: there are some missing substitution variable
(dpkg-gencontrol complains that some of them are not used); TTBOMK, you
should use:
- dev:
Depends:
${haskell:Depends},
${misc:Depends},
${shlibs:Depends}
Suggests: ${haskell:Suggests}
Recommends: ${haskell:Recommends}
Provides: ${haskell:Provides}
- prof:
Depends:
${haskell:Depends},
${misc:Depends}
Suggests: ${haskell:Suggests}
Recommends: ${haskell:Recommends}
Provides: ${haskell:Provides}
- doc:
Depends:
${haskell:Depends},
${misc:Depends}
Suggests: ${haskell:Suggests}
Recommends: ${haskell:Recommends}
Probably, the Haskell Policy should be updated (and not only for this
reason). Maybe I can find some time next week to have a look at it.
* About the fact that libghc6-missingh-doc is substituting
missingh-doc: reading policy 7.6.2, my understanding is that Conflicts
is more appropriate than Breaks (because the old package must be
completely removed). Why are you changing it?
* There are some lintian suggestions that you may want to follow (are
you aware of the -I and --pedantic options? Sometimes they're really too
pedantic, but usually they can give you good hints). All I: and P: are
completely optional, but there are also two W: that you should fix (BSD
is not anymore in common-licenses; I suspect this is due to the fact
that there are many different BSD licenses, so saying "BSD" isn't clear:
it's better to copy the verbatim text).
haskell-configfile:
* There's something strange in the repository: it appears that Marco
has already packaged and tagged (but never updated) a version 1.0.6-2.
You should clean things up.
* source/format: as before
* control: as before, both for Depends & co. and for Breaks vs. Conflicts
* lintian: as before (but without W:s)
* package descriptions: I don't know what you mean with "DHG
consistence", but don't like very much the initial paragraph you have
put in all the descriptions (it could be the last, maybe; the first
paragraphs should give a general idea of what the package do, not where
to find information about the language it is written with). Is it really
a best practice to have such a preamble? (this question, of course, is
for the full mailing list).
* changelog: more generally, I wouldn't put acronyms like "DHG", that
could be meaningless for our users.
* debian/configfile-doc.docs: can be removed.
Other than that, the packages seem to compile correctly (but I haven't
done the cowbuilder test yet). Please, fix the issues I've outlined (or
help me understanding why they're not problems :-) ), and I'll do the
final check.
Cheers, Giovanni (who feels a bit strange of standing for the first time
on the other bank of the DD river!).
--
Giovanni Mascellani <mascellani@poisson.phc.unipi.it>
Pisa, Italy
Web: http://poisson.phc.unipi.it/~mascellani
Jabber: g.mascellani@jabber.org / giovanni@elabor.homelinux.org
Attachment:
signature.asc
Description: OpenPGP digital signature