Bug 761565 - Missing deps on netpbm-progs and icoutils
Missing deps on netpbm-progs and icoutils
Product: Fedora
Classification: Fedora
Component: libguestfs (Show other bugs)
Unspecified Unspecified
unspecified Severity unspecified
: ---
: ---
Assigned To: Richard W.M. Jones
Fedora Extras Quality Assurance
Depends On:
  Show dependency treegraph
Reported: 2011-12-08 10:51 EST by Matthew Booth
Modified: 2012-11-30 08:08 EST (History)
3 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2012-11-30 08:08:36 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)

  None (edit)
Description Matthew Booth 2011-12-08 10:51:46 EST
Description of problem:
Running virt-inspector against one of my Windows guests output the following to stderr:

# virt-inspector windevel.img > /dev/null
sh: pamcut: command not found
sh: wrestool: command not found
sh: pnmtopng: command not found
sh: bmptopnm: command not found

Installing netpbm-progs and icoutils resolved these dependencies.

Version-Release number of selected component (if applicable):
Comment 1 Richard W.M. Jones 2011-12-08 10:58:50 EST
icoutils requires Perl ...

We'll have to devise a way to make these optional, since
they are only required for icons.
Comment 2 Richard W.M. Jones 2012-11-30 08:08:36 EST
This is fixed.  First of all the dependencies have been
added to the base libguestfs package:

# For core inspection API.
Requires:      libdb-utils
Requires:      netpbm-progs
Requires:      icoutils

Secondly the code was changed so that even if these programs
are not present, we get degraded functionality but no huge
ugly error messages.  At build time:

commit 2c9c0525ebda105f37e6a6eea2307db4cb2f0a73
Author: Richard W.M. Jones <rjones@redhat.com>
Date:   Wed Mar 7 15:07:37 2012 +0000

    build: Make netpbm and icoutils into proper optional dependencies.
    Netpbm and icoutils (wrestool) have always been dependencies.  Since
    they are not always present, make these into optional dependencies
    (which they were, sort of, before).
    Also document these dependencies in the README file.

and at run time the use of the new command "mini-library"
ensures that error output goes into the debug log (and not
to stderr) by default:

commit 019b840e4758ccd254925181f6fbaf3192c7bde1
Author: Richard W.M. Jones <rjones@redhat.com>
Date:   Thu Oct 18 10:55:42 2012 +0100

    inspect: Change icon code to use command mini-lib instead of system(3).

Note You need to log in before you can comment on or make changes to this bug.