Code review comment for lp:~manishsinha/zeitgeist-datasources/reworked-tomboy

Revision history for this message
Michal Hruby (mhr3) wrote :

From a quick look (I'm not using Tomboy):
- 'TomboyUri' should be just application://tomboy.desktop
- please get rid of the install.sh script, everything is now done using Makefiles
- I really don't like the bundled dll library, isn't there another way to check for it?

review: Needs Fixing

« Back to merge proposal