[vote] unity-lens-vm
Andrew Mitchell
ajmitch at ubuntu.com
Thu Jun 14 03:43:30 UTC 2012
On Tue, Jun 12, 2012 at 02:03:30PM -0400, Jonathan Carter (highvoltage) wrote:
> Hi
>
> https://myapps.developer.ubuntu.com/dev/apps/807
>
> Daniel made some fixes to the quickly-generated package and posted it at:
>
> lp:~dholbach/ubuntu-app-reviews/vm-lens
>
> I had a look over it and everything looks fine so I'm putting it to vote.
>
> +1 from me.
>
> -Jonathan
>From my comments on IRC:
I checked over the vm lens, but the first thing that I spotted after building
was that there are several files outside of /opt which aren't namespaced:
Directories like /usr/share/unity/lenses/vm/ should instead be
/usr/share/unity/lenses/extra-unity-lens-vm/ and the D-BUS service file
should be ./usr/share/dbus-1/services/extras-unity-lens-vm.service
Small things, but they're needed to make sure there's no file conflicts.
Andrew
More information about the App-review-board
mailing list