Phablet Tools

Code review comment for lp:~jibel/phablet-tools/lp1238839_phablet-screenshot

  1. lp1238839_phablet-screenshot
  2. Merge into trunk
Revision history for this message
Sergio Schvezov (sergiusens) wrote :

imagemagick should be in debian/control

I can't really test this, as I have a maguro. So I've only tested the odd condition of unsupported.
The error message could be nicer though ./phablet-screenshot: 171: ./phablet-screenshot: SUPPORTED: parameter not set

SUPPORTED is not defined anywhere, which explains the wrong error message.

review: Needs Fixing

« Back to merge proposal