Hi,<div><br></div><div>Wishing to help the ARB start making a dent in some of the apps waiting for review (including mine), I downloaded the tar.gz file for TeaTime from: <a href="https://myapps.developer.ubuntu.com/dev/apps/313/">https://myapps.developer.ubuntu.com/dev/apps/313/</a> </div>
<div><br></div><div>TeaTime is a very simple timer application.  The package works and builds fine with debuild and has a GPL3 license in debian/copyright. It installs into /opt/<a href="http://extras.ubuntu.com/teatime">extras.ubuntu.com/teatime</a> although the developer renamed the source package to teatime-unity (because of another package called teatime; causes some errors in arb-lint).  The output from arb-lint is as follows:</div>
<div><br></div><div><div>ian@ubuntu:~/pythontesting/arb-review/teatime$ arb-lint</div><div>W: The ARB wants only one changelog entry in debian/changelog, so you might</div><div>   want to collate all the relevant information in one changelog entry</div>
<div>   under the last version and date.</div><div>W: This package does not use DEP-5, which provides a machine-readable</div><div>   debian/copyright file. You might want to review</div><div>   <a href="http://www.debian.org/doc/packaging-manuals/copyright-format/1.0/">http://www.debian.org/doc/packaging-manuals/copyright-format/1.0/</a> to</div>
<div>   find out how to accomplish it. <a href="http://bazaar.launchpad.net/~dholbach">http://bazaar.launchpad.net/~dholbach</a></div><div>   /ubuntu-app-reviews/harvestwidget/view/head:/debian/copyright has a live</div><div>
   example for a simple app.</div><div>E: The package seems to contain no files in '/opt/<a href="http://extras.ubuntu.com">extras.ubuntu.com</a></div><div>   /teatime-unity' which indicates a problem with the installation of</div>
<div>   files.</div><div>E: These files are outside opt/<a href="http://extras.ubuntu.com/teatime-unity">extras.ubuntu.com/teatime-unity</a> and</div><div>   shouldn't be:</div><div>   - usr/lib/python2.7/dist-packages/teatime-12.06.egg-info</div>
<div>   - usr/share/pyshared/teatime-12.06.egg-info</div><div>   - opt/<a href="http://extras.ubuntu.com/teatime/window.ui">extras.ubuntu.com/teatime/window.ui</a></div><div>   - opt/<a href="http://extras.ubuntu.com/teatime/teatime.py">extras.ubuntu.com/teatime/teatime.py</a></div>
<div>   - opt/<a href="http://extras.ubuntu.com/teatime/locale/en_GB/LC_MESSAGES/teatime.mo">extras.ubuntu.com/teatime/locale/en_GB/LC_MESSAGES/teatime.mo</a></div><div>   - opt/<a href="http://extras.ubuntu.com/teatime/locale/el/LC_MESSAGES/teatime.mo">extras.ubuntu.com/teatime/locale/el/LC_MESSAGES/teatime.mo</a></div>
</div><div> ..... (more .mo files)</div><div><br></div><div>The python program is pretty simple and stores its saved timers in  ~/.local/share/teatime.js (json format). I don't know if that's ok as the guidelines say configuration data should be stored in ~/.config/teatime</div>
<div><br></div><div>I hope this review is helpful. Please let me know if I should include other information, or if certain things are not needed. Also attached is the output from debbuild and arb-lint in a text file. I plan to do a few reviews over the next couple of weeks.</div>
<div><br></div><div>Best,</div><div>Ian</div>