[Vote] Fogger

Andrew Mitchell ajmitch at ubuntu.com
Thu Aug 2 08:30:39 UTC 2012


On Thu, Aug 02, 2012 at 10:51:47AM +0530, Bhavani Shankar R wrote:
> Just went through the package. Looks fine here except few nitpicks in
> d/control as shown by lintian below
> 
> E: fogger: description-starts-with-package-name
> I: fogger: description-synopsis-might-not-be-phrased-properly
> W: fogger: description-too-long
> I: fogger: capitalization-error-in-description linux Linux
> P: fogger: no-homepage-field
> 
> and
> 
> W: fogger: executable-not-elf-or-script
> opt/extras.ubuntu.com/fogger/share/fogger/templates/fogapp.desktop.tmpl
> 
> Looking into the code, seems like a desktop file template and can be
> bypassed I guess
> 
> +1 from me for this now (few nitpicks present but that is minor I believe)
> 
> Adding bzr-builddeb.conf I can build the package using bzr bd now.
> 
> Regards

+1 as the points above are relatively minor & can be tidied up.

Andrew



More information about the App-review-board mailing list