[Vote] Modem Manager GUI
Bhavani Shankar R
bhavi at ubuntu.com
Sun Sep 9 11:36:18 UTC 2012
On Mon, Aug 20, 2012 at 7:47 PM, Bhavani Shankar R <bhavi at ubuntu.com> wrote:
> On Mon, Aug 13, 2012 at 2:04 PM, Andrew Mitchell <ajmitch at ubuntu.com> wrote:
>> On Wed, Jul 18, 2012 at 05:42:25PM -0400, Jonathan Carter (highvoltage) wrote:
>>> On 2012-07-16 16:24, Michael Hall wrote:
>>> >I have reviewed this app and made some changes to fix lintian and
>>> >arb-lint warnings.
>>> >
>>> >The code with changes is here:
>>> >
>>> >https://code.launchpad.net/~mhall119/ubuntu-app-reviews/modem-manager-gui/
>>>
>>> It doesn't seem to access any system files in any way or do anything
>>> weird that escalates to root, packaging also looks good.
>>>
>>> +1
>>>
>>> -Jonathan
>>
>> +1, though I did switch it back to 3.0 (native) for it to build.
>>
>> Andrew
>>
>
> +1 to Andrew's comment. Apart from that I found the AUTHORS file
> missing from the source and couple of minor lintian warnings:
>
> W: modem-manager-gui: possible-unindented-list-in-extended-description
> I: modem-manager-gui: capitalization-error-in-description dbus D-Bus
>
> These should be straightforward to fix, So +1
>
Pulled in the latest version from developers' PPA. Fixed the above
except the lintian warning (as it can be ignored I think) and uploaded
to extras PPA for some basic testing.
Regards,
--
Bhavani Shankar
Ubuntu Developer | www.ubuntu.com
https://launchpad.net/~bhavi
More information about the App-review-board
mailing list