[vote] Cuttlefish
Bhavani Shankar R
bhavi at ubuntu.com
Fri Aug 31 06:44:08 UTC 2012
On Tue, Jul 10, 2012 at 9:07 PM, Stéphane Graber <stgraber at ubuntu.com> wrote:
> On 07/10/2012 11:30 AM, Stéphane Graber wrote:
>> On 07/10/2012 10:26 AM, Stéphane Graber wrote:
>>> On 07/10/2012 07:03 AM, Paolo Rotolo wrote:
>>>> Hi,
>>>> Stephane, thanks for your corrections.
>>>> I've re-uploaded the fixes in the branch:
>>>> https://code.launchpad.net/~paolorotolo/ubuntu-app-reviews/cuttlefisht
>>>>
>>>> Regards,
>>>> Paolo
>>>
>>> W: cuttlefish: script-not-executable
>>> opt/extras.ubuntu.com/cuttlefish/cuttlefish/indicator.py
>>>
>>> ^ that's the usual, "file contains shebang but isn't meant to be an
>>> executable" error, so ideally this file should be kept non-executable
>>> but the shebang removed from its header.
>>>
>>>
>>> As for the other ones I voted on, this isn't critical, so +1 regardless.
>>
>> Withdrawing that +1 until debian/copyright is a valid DEP5 copyright
>> file (including valid link to the specification).
>
> This has now been fixed, thanks.
>
> +1
>
>
>
Built locally and tested. Uploaded to extras PPA
Regards
--
Bhavani Shankar
Ubuntu Developer | www.ubuntu.com
https://launchpad.net/~bhavi
More information about the App-review-board
mailing list