[Vote] Cuckoo Alarm

Bhavani Shankar R bhavi at ubuntu.com
Mon Sep 10 07:06:05 UTC 2012


On Thu, Jul 19, 2012 at 1:39 PM, Andrew Mitchell <ajmitch at ubuntu.com> wrote:
> On Tue, Jul 10, 2012 at 12:04:11PM -0400, Jonathan Carter (highvoltage) wrote:
>> On 2012-07-10 10:04, Stéphane Graber wrote:
>> >Same comment as with FlashGen:
>> >/opt/extras.ubuntu.com/cuckoo/cuckoo/indicator.py shouldn't have a
>> >shebang as it's not an executable script (just meant to be imported)
>> >
>> >That's minor and the rest looks good here: +1
>> >
>> >(As with all the others, I just did static analysis of the source and
>> >binary, haven't actually installed or run it)
>>
>> Same, +1 from me otherwise.
>>
>> -Jonathan
>
> +1
>
> Andrew
>


Built, Installed, Tested with alarm.ogg (which seems the default file)
and 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