Re: RFS: i-news - GTK-based RSS Ticker
On 08/22/2011 07:47 PM, Gergely Nagy wrote:
> And while I can't sponsor the package (not a DD myself, either), I had a
> quick look at it (at least, on the latest version you posted here):
>
> * In debian/copyright, the paragraph separator dots are unindented - I'm
> not sure whether that's acceptable or not, but it looks a bit awkward.
> * debian/changelog is executable for some reason (o.O)
> * You don't need to list ChangeLog in debian/docs, dh_installchangelogs
> will install it as the upstream changelog by default. But.. do you
> really want to do that? Upstream ChangeLog is zero-length, and thus not
> really useful at this time.
> * And as someone already mentioned, there's common-install-indep:: in
> debian/rules which I believe is a cdbs-ism, and does not have any effect
> in the current packaging.
>
> Pretty minor stuff, but thought I'll have a look and contribute
> something else than nagging for a different package name O:)
Thanks, I just uploaded corrected version.
--
Emmanuel Thomas-Maurin <manutm007@gmail.com>
Reply to: