[Pkg-gmagick-im-team] Bug#703616: Missing Depends: html2ps

[Vincent Fourmond]

On Thu, Mar 21, 2013 at 5:48 PM, Mathieu Malaterre <malat at debian.org> wrote:
> On Thu, Mar 21, 2013 at 1:37 PM, Vincent Fourmond <fourmond at gmail.com> wrote:
>> control: severity -1 minor
>> control: tag -1 wontfix
>>
>> On Thu, Mar 21, 2013 at 12:23 PM, Mathieu Malaterre <malat at debian.org> wrote:
>>> identify requires html2ps to be installed on the system. Steps:
>>>
>>> $ echo "<html/>" > t.html
>>> $ identify t.html
>>> sh: html2ps: not found
>>> identify: delegate failed `"html2ps" -U -o "%o" "%i"' @ error/delegate.c/InvokeDelegate/1061.
>>> identify: unable to open image `/tmp/magick-XXAkOyS6': � @ error/blob.c/OpenBlob/2489.
>>> identify: unable to open file `/tmp/magick-XXAkOyS6': ��� @ error/constitute.c/ReadImage/572.
>>
>>   I fail to see what prompts the use of an image identification
>> program on a web page. So, yes, identify does not give you useful
>> information on a web page, but what did you expect anyway ? Use file
>> for general identification of file types.
>
> If I knew this was an image already I would not run identify on it.

  I really think you're using the wrong program here: identify is here
to give you the properties of an image. It will try as hard as
possible to convert what you give it into an image. I would be
surprised if you expected HTML pages to be categorized as "images". If
you want to know if a file is an image, file $file | grep image is the
way to go. Not identify.

> From a pure scripting approach I need to run this identify script.

  s/identify/file/g

> The fact that it return `1` is an issue for me.

  I may sound arrogant on that, and I apologize in advance if I do,
but then you're not using the right program ;-)...

>>   html2ps is already in Suggests; and that's already too much, I think.
>
> Well I would have set html2ps as Recommends, which is barely enough :)

  Not the way I see it.

  Cheers,

      Vincent