stgraber at stgraber.org
Mon Jul 9 17:18:50 UTC 2012
On 07/09/2012 01:10 PM, Stéphane Graber wrote:
> On 07/09/2012 12:31 PM, Paolo Rotolo wrote:
>> Hi all!
>> I did a review of Cuttlefish and I have updated the changes on a branch:
>> It is ready for me :)
> W: cuttlefish source: unused-build-dependency-on-cdbs
> W: cuttlefish source: package-needs-versioned-debhelper-build-depends 8
> W: cuttlefish source: debhelper-overrides-need-versioned-build-depends
> (>= 7.0.50~)
> W: cuttlefish source: ancient-standards-version 3.8.3 (current is 3.9.3)
> The short-description and description could use some rewording, it's
> quite difficult to know what that app is supposed to do.
> The binary package contains: python/runtime.d/
> The directory is empty but it still shouldn't be there.
For that specific point (runtime.d), we agreed that this file can now be
shipped in the package, so removing the rm call should fix the issue.
(The rational being that dh generate a file based on the binary package
name, so is inherently namespaced and won't conflict).
> cuttlefish/indicator.py looks like an executable ("#!/usr/bin/python")
> but isn't marked as executable. I believe the problem here (detected by
> lintian) is that it shouldn't contain the shebang.
> None of these are critical but I'd prefer to have these fixed before voting.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 900 bytes
Desc: OpenPGP digital signature
More information about the App-review-board