edit: Clean up temporary files.

Note to self: The 'tempfile' function does *not* default to
removing files with the program exits!

For stable-1.2 branch:
 - Cherry picked from commit 10ea14a3f1adb7023dd0601e4759bd24a030a1c3
 - Rebase.

fish: Don't fail if -m and --listen flags are both given (RHBZ#612178).

Testing this against a Fedora disk image:

$ ./fish/guestfish --ro -a F13.img -m /dev/sda1 --listen
export GUESTFISH_PID=6033
$ ./fish/guestfish --remote=6033 -- ping-daemon
$ ./fish/guestfish --remote=6033 -- ping-daemon
$ ./fish/guestfish --remote=6033 -- exit

Without this fix the first remote command would fail because
qemu would have already been killed.

For stable-1.2 branch:
 - Cherry picked from commit 8161ea9bb046c8450384b5b15e1f4b2f3dca582b
 - Rebased for older guestfish code.

inspector: Improve error message when YAML::Any library is not installed.
(cherry picked from commit 015b7a2ee6b4ba0f1400640fa16e2526a69d081e)

Explicitly depend on e2fsprogs.

See: http://lists.fedoraproject.org/pipermail/devel/2010-June/137953.html
(cherry picked from commit 9928498ad87ed1d7a1c191d5637d36d5f9b09ee4)

Fix gfs2 support by adding required kernel modules.
(cherry picked from commit 61adbba900a4150110d9fcd20097b3a5c249e6c7)

ocaml: Fix thread safety of strings in bindings (RHBZ#604691).

There's a thread safety issue with the current OCaml bindings which
is well explained in the bug report:

https://bugzilla.redhat.com/show_bug.cgi?id=604691

This commit fixes the safety issue by copying strings temporarily
before releasing the thread lock.  Updated code looks like this:

  char *filename = guestfs_safe_strdup (g, String_val (filenamev));
  int r;

  caml_enter_blocking_section ();
  r = guestfs_add_drive_ro (g, filename);
  caml_leave_blocking_section ();
  free (filename);
  if (r == -1)
    ocaml_guestfs_raise_error (g, "add_drive_ro");

Also included is a regression test.

For stable-1.2 branch:
  - cherry picked from commit 1079f74704a06c06996e547fdecf20a8f92799c6
  - generator code rebased

perl: Check all images are defined in first param of open_guest.
(cherry picked from commit e67e216118df3a724482cb698a868ecbaacbf326)

Fix typo in documentation of guestfs_set_launch_done_callback.
(cherry picked from commit 91b00dc092be17a309f14fc35f41dbf1e41c4c58)

virt-df: Disallow -h and --csv options together (RHBZ#600977).

Before this commit, if you used the -h and --csv options together
you would get these warnings from virt-df:

$ virt-df -h --csv Guest
Virtual Machine,Filesystem,Size,Used,Available,Use%
Argument "13.5G" isn't numeric in printf at /home/rjones/d/libguestfs/tools/virt-df line 298.
Argument "4.7G" isn't numeric in printf at /home/rjones/d/libguestfs/tools/virt-df line 298.
Argument "8.1G" isn't numeric in printf at /home/rjones/d/libguestfs/tools/virt-df line 298.
"/dev/vg_trick/RHEL55x64","/dev/VolGroup00/LogVol00",13,4,8,34.8%
Argument "98.7M" isn't numeric in printf at /home/rjones/d/libguestfs/tools/virt-df line 298.
Argument "18.8M" isn't numeric in printf at /home/rjones/d/libguestfs/tools/virt-df line 298.
Argument "74.9M" isn't numeric in printf at /home/rjones/d/libguestfs/tools/virt-df line 298.
"/dev/vg_trick/RHEL55x64","/dev/vda1",98,18,74,19.0%

We could fix this so that the human-readable numbers get written
into the CSV file.  However would probably be wrong for most uses
of the CSV format (databases and spreadsheets) since they would not
be able to interpret these human-readable numbers, or worse could
misinterpret, eg. thinking that "1M" and "1G" are both 1.

Therefore this commit disallows this combination of options.
(cherry picked from commit aaf03a51a26ee501f5dbf2720bbb8a5e3b4e1e6b)

generator: Allow individual tests to depend on daemon features.

Using IfAvailable "featurename" we allow individual tests to
only run if the feature is available in the daemon.

This will allow us to extend testing to a lot more optional
features such as NTFS.
(cherry picked from commit f9d08600c52dc0730e7dad8d9259b59e32aeece2)

tests: Factor out common code into 'is_available' function.

This commit is just code motion.
(cherry picked from commit cbe80b2bcfdee437d195f25aaf6f5d96329ab360)

grub-install: In docs suggest manually creating device.map (RHBZ#484986).
(cherry picked from commit dccd9b8f525cfae49b89c38066d752e6c054c7f6)

Version 1.2.9.

RHEL 6: sr_mod.ko is needed for RHEL 6 to see CD-ROM devices (RHBZ#598807).
(cherry picked from commit ac37f65aaf1f162a2cc57fa8c296a1ff4109adb9)

parted: Check partition number >= 1 in several calls.
(cherry picked from commit cbb026036826f2ab40237aec0b0c5453b859625d)

daemon: Parse output of old parted which didn't support -m option (RHBZ#598309).

This fixes the following commands when run with RHEL 5-era parted:

  get-bootable
  get-parttype
  part-list
(cherry picked from commit 3ab2d089f3eb34562bc2c9ce4310869b46c69d70)

daemon: count_strings function returns size_t
(cherry picked from commit aee7d55fcf754d97d945c318ec06d6387ade067b)

resize2fs: Document this command also works with ext4 (thanks Yufang Zhang).

For stable-1.2 branch:
 - cherry picked from commit 1020b212b189968ead013436cac79019fbd8fdad
 - rebased for 1.2 branch

fish: help command return error for non-existent commands (RHBZ#597145).

With this change, the exit status indicates error for non-existent
commands.

$ guestfish -h foo
foo: command not known, use -h to list all commands
$ echo $?
1
$ guestfish help foo
foo: command not known, use -h to list all commands
$ echo $?
1

For stable-1.2 branch:
 - cherry picked from commit f2b7a8e15c49ebc70c7ea56aefb340362aae5a99
 - rebased for the 1.2 branch

daemon: write-file: Check range of size parameter (RHBZ#597135).

This also adds a regression test.

For stable-1.2 branch:
 - cherry picked from commit 9733d4746988b3a072d8bb1daac4b9795b8f4134
 - modify the regression test to apply against the generator

daemon: Limit label lengths (RHBZ#597118).
(cherry picked from commit 52f9cd4882135910ea06e1e50ac6441d455c9ab1)

daemon: Kill blkid cache to improve reliability of blkid commands.

By killing the cache file, we make blkid work in situations such
as a just-created filesystem.
(cherry picked from commit 21c42e9fabf6cea3d564e338a314479ef120502a)

daemon: Generalize the implementation of vfs-type.

Note that there is no change to the semantics of the code.
(cherry picked from commit 85c71f8fff3e80f549342bf995b686ba7303c2b4)

todo: Should generate -N option.
(cherry picked from commit fc1dc099a1ceeea45d3c21c699daf39618ca714a)

todo: Remove implemented resize functions from TODO file.
(cherry picked from commit 0e42861ccf9e55582db13828ca4eaef213de1f8a)

Rely on new augeas lens for modules.conf and conf.modules

Latest augeas includes a lens for /etc/modules.conf. If this new lens is
present, the code to force the Modprobe lens to try to match /etc/modules.conf
as well results in /etc/modules.conf not being parsed at all. This results in
modprobe_aliases in virt-inspector output being empty.

This change is equivalent to change cfd28d1140393667913689b7b9bcf21c8bfe592c
from virt-v2v.

An effect of this change is that the Modules_conf augeas lens is now required
for correct operation on guests which use /etc/modules.conf.

Fixes RHBZ#596776
(cherry picked from commit 245ed4b8eb076a8c4cc5787f49d21c4f68630f9f)

Fix documentation for vfs-type to reflect reality.
(cherry picked from commit 0f6c6239fe0d1b4624e4e9776559c21486a9c7cf)

Clarify documentation on distro backports in version command.
(cherry picked from commit 11eeb8885c7a8ffb59e12cd5fb2ae0fc324df383)

Add reference to version number documentation to version command.
(cherry picked from commit 8355d3245623c106439ca5ef66f24972c8e09019)

Clarify sparse behaviour of truncate-size command.
(cherry picked from commit d12a702fe1898f3ea421210d92c705440efe07a8)

Fix typo in documentation of guestfs_readlinklist.
(cherry picked from commit 4412fd951ee7766ee9a70bf66b5cf11378a64ed0)

Fix missing word in docuentation of guestfs_readdir.
(cherry picked from commit b8861242d13cc034383a56ba891d3e0b9123a99c)

Fix typo in description of echo-daemon command.
(cherry picked from commit c1e3cff523d72e8110cbc568a3c280e57c2aabca)

fish: Don't eat words when completing case-insensitive paths (RHBZ#582993).

(cherry picked from commit 21bd2db7cf259a17cc3922409937b849e4b83c0f)

fish: Sort returned paths so the list is stable across multiple calls.
(cherry picked from commit 53c3b9d2b03fa5cb0ac7e86a5a51f2a18a2b91c1)

fish: Create a separate FILES section in the manpage.
(cherry picked from commit 7fd6e4bfb11d1f86c585de06fb2146aa5cbd400e)

fish: Move 'EXIT CODE' section to a more logical place in the documentation.
(cherry picked from commit 34a306ab2a7e6d9f468e8194cef8a05624325260)

resize: Refresh the examples in the documentation.

The documentation was previously very intimidating.  Bring some
common, simple examples up to the top of the page in a separate
section.

For stable-1.2 branch:
 - cherry picked from commit 0e28e4104d96bf0bf5b88fb07bb7e5f9f6e6f41f
 - modified instructions slightly to apply to older version of
   virt-list-partitions
 - replace 'truncate' with 'dd'

fish: Make the read/write warning more prominent.

Follow the example on other manual pages by making the warning
more prominent.
(cherry picked from commit 57cc217335ff511e579b094e47b0de57d5c3aaef)

Version 1.2.8.

build: For development releases, print a notice.
(cherry picked from commit 613d041c78e25dafc238696f2fe35b51e443f23b)

guestfs_version: Correct documentation.

Remove reference to 'ELF weak linking tricks' and replace
with suggestion to use dl* functions.
(cherry picked from commit 94030c5ce8b5848330e15e812a01064d5afcc6d0)

TODO: Suggest including undelete utility.
(cherry picked from commit 478e1942a9fb4d86b0c4701b6b71f1eac376d0c1)

Fix name of update-bugs script in output file BUGS.
(cherry picked from commit aecdf3c117874e3dc44282600b988d18350b3ad5)

generator: Some String parameters should be OptString (RHBZ#501894).

I haven't checked the list of functions exhaustively, but
these are the obvious ones.
(cherry picked from commit e715451fae0ba738973af98a4e506b6c5564626a)

fish: Fix build error if built without readline.

fish.c:1447: error: 'add_history_line' defined but not used [-Wunused-function]

(Reported by Matt Booth)

(cherry picked from commit 5af010c96d687a7cf5251107a74770a231f42b77)

appliance: Ignore utempter unreadable binaries (for RHEL 6).
(cherry picked from commit 2dc7bb00337656c00fabed2c1f017aac1fa09da4)

appliance: Change comment about unreadable binaries.

For stable branch, change reference to febootstrap-supermin-helper
to libguestfs-supermin-helper.
(cherry picked from commit 155aa76e57c99594a3724933b6d117f1053544b3)

Update BUGS and PO files.

Update BUGS.

Version 1.2.7.

Generate BUGS file from Red Hat Bugzilla database.

Add a script which generates the 'BUGS' file from Red Hat Bugzilla.
This is run whenever we do 'make dist' and deliberately stored in git.
(cherry picked from commit 002dc21cba1469cc5b1ce5ef4a7984a733b44a1f)

virt-df: Display domains in sorted order.
(cherry picked from commit 5d5565ca24b95719461b0c0aebdb0651dd2671ca)

Update PO files.

Version 1.2.6

Improved error if virt-inspector cannot find OSes in image (RHBZ#591142).

(cherry picked from commit 94e310dcfbcd368cbe02dbc1643ed2ff9821cd48
and modified to apply to the stable branch)

Fix error message in strings-e command (RHBZ#588651).

(cherry picked from commit 287f8957fea3efe411c7ac55595d5d6c7b613e4e
and modified to apply to the stable branch.  Note that the original
commit message contains a typo 'string-e' instead of 'strings-e')

Reduce imported functions in virt-df to ones which are actually used.
(cherry picked from commit 5477d98548661394cf0df3ce48929961e0d2a38f)

Run udev_settle() after swapon/swapoff (RHBZ#516096).
(cherry picked from commit 9d158c3ab132df21d58890f18224cd2fe020de67)

virt-rescue: Refresh documentation.
(cherry picked from commit d0afef23e54f75a521eed8c7261a533776242cfc)

launch: Rearrange the code so config check is first.

Move the config state check first in the guestfs_launch function,
so that we don't reset g->launch_t or calculate the temporary
directory in the case where the launch function will immediately
return with an error.
(cherry picked from commit 55e9707f8591488542da48fc89738234b4d85016)

Ignore test1.img file in directory.
(cherry picked from commit 2064e43bcfc7fc5bde08a0442c646a0a46997875)

fish: Make '-x' option enable traces instead of using separate echo path.

Previously we had separate code paths for echoing commands (-x)
and tracing (guestfs_set_trace).  This just unifies that so that
the guestfish -x option enables tracing.

(cherry picked from commit 8e007581c91a5193e6aec0f6f9aeb379a935c9d4
and commit a96d70113ffcd01f55262c3d0fbe711bee759fec
and commit 87eecb507ee09a62f3a12f5f75d3417a09c0c8d4 which are all
logically part of the same fix)

daemon: gnulib module 'error' is used directly by the daemon.
(cherry picked from commit b52ef0b021fc443d88d4d9c43f57468712e40965)

If qemu dies during launch, set an error message (RHBZ#588851).
(cherry picked from commit fc6dd9daa13ac774156d0822b5aa7830171feb85)

daemon: Fix read-file so it fails gracefully for large files (RHBZ#589039).

Pengzhen Cao noticed that read-file would fail for files
larger than the protocol size; this is *not* the bug.  However
it would also lose protocol synchronization after this.

The reason was that functions which return RBufferOut in the
generator must not 'touch' the *size_r parameter along error
return paths.

I fixed read-file and initrd-cat, and I checked that pread was
doing the right thing.

This also adds regression tests for read-file with various categories
of large file.
(cherry picked from commit 42f59b28f123f53ae038df23a9abee08e959e46b)

Warn instead of dying if grub refers to non-existent kernel
(cherry picked from commit 4839d5142ca50818965866287932f9ded14729e6)

Change network configuration to use macros.

Change the network configuration so everything is set using
some macros at the top of src/guestfs.c.

Also, rename the macros used in the daemon so they are not the
same.  It was a very long time since these sets of macros had to
match the ones defined in src/guestfs.c, despite what the comment
said.

Note that this commit should not change the semantics of the
program at all.
(cherry picked from commit 8a9f2ca65521d093ac14307aca4370d9497ac840)

tests: In verbose mode, print dashes between each test.

Only affects tests when run with LIBGUESTFS_DEBUG=1.  Lets you easily
see when each test starts and ends.
(cherry picked from commit 097266e01cfa28319834bf750649650f3c22006f)

Version 1.2.5

Document that guestfs_mount implies -o sync and performance problem (RHBZ#587582).
(cherry picked from commit fd2d10d6ed91f417a9b99bb61526b45bb7ed3e4e)

contrib: Add dependency diagram of libguestfs ecosystem.
(cherry picked from commit 90b4e24df6f27c02d3b11d8021b27a197cf18b75)

contrib: Add header to README file.
(cherry picked from commit 2f9f3375028a02fdaec62f30e11bacd6524a342c)

lvresize: Use --force so it can make LVs smaller (RHBZ#587484).

This also adds a regression test for this bug.
(cherry picked from commit c24de46d06cc3ecccf00bfaaffb06172659cdd0a)

tools: Fix documentation for CurrentControlSet (thanks Yuval Kashtan).
(cherry picked from commit 25c15261253bbd2fb140e5ca1c902acf714f76a9)

guestfs(3): Documentation on protocol gotchas.
(cherry picked from commit 55641f5d1e9964f8a039005b511b8e688c92cddb)

docs: Routine refresh of the documentation for guestfs(3) and guestfish(1).

For stable branch:
 - cherry picked from commit ce95be8b185ce69
 - updated the patch so it doesn't refer to any new guestfish features
     in the 1.3 branch

Version 1.2.4

todo docs: Discuss shrinking filesystems and specifying sizes in guestfish.
(cherry picked from commit 7c285ea84f63e511f95a3c8bfbb98cbf99b9466b)

daemon: Split out checksum type to program name mapping into function.

This shouldn't change the semantics of the program.

For stable-1.2 this was cherry picked from commit af29c84 and
rebased against the older code in daemon/checksum.c.

todo docs: Integrate with host intrusion detection systems.
(cherry picked from commit 6e269c6e5123c1558589e0a59844dc0187fbd63a)

todo docs: Mapping filesystems to and from disk blocks.

(cherry picked from commit 2ed549e95c7ad75f28b5f0dd4c53849e1607444e)

fish docs: Be consistent about using I<-..> for options.
(cherry picked from commit 89486c50eaeb8b1cc5a6ff388cb47e859f8565d1)

fish docs: Use L</...> for internal links in the man page.
(cherry picked from commit e3e67f63b0a6ce952ba92cb966b8ac5ef239dcf9)

Whitelist the loop kernel module

loop is sometimes a module, for example on RHEL 5.
(cherry picked from commit 75d9c056b2db9ceedef9eca64e67ad3350eabad8)

Don't die during inspection if rpm -qa or dpkg-query fails

If a problem in the package database prevented package enumeration from working,
inspection would die. This change makes it emit a warning and continue.
(cherry picked from commit 39416cdb11bb2005915ed293eafb3cc6d714416d)

Don't die during inspection if initrd doesn't exist

This fixes a problem where inspection would die if grub.conf referenced a
non-existent initrd. Just return an empty initrd instead.
(cherry picked from commit 2b43970c8c97f24b1f45c040c6963d30661fa514)

Remove checks for Test::Pod and Test::Pod::Coverage.

Although these modules are optionally used by the Perl tests,
they aren't necessary and won't break the build if they are not
there.  These modules aren't available in RHEL 5.  Therefore
remove these checks.
(cherry picked from commit c8f302a17453ad58acccd137342438795763606a)

Check user doesn't run configure, make or make check as root.

(Thanks to Yufang Zhang for helping to debug this issue).
(cherry picked from commit 199f218bf431b77c12a728f7072bed1553a0f0af)

Replace 'dd' in tests with use of guestfish 'sparse' command.

'dd' is a very inefficient way to create files.  'truncate' is
better, but unfortunately that command is not available in RHEL 5.
So use the guestfish 'sparse' command instead (which also avoids
allocating disk space).
(cherry picked from commit 175c01d2336c07113f1c2966966957de5c344e3a)

Document new version numbering policy.

See discussion on mailing list:

https://www.redhat.com/archives/libguestfs/2010-April/msg00005.html
https://www.redhat.com/archives/libguestfs/2010-April/msg00057.html
https://www.redhat.com/archives/libguestfs/2010-April/msg00058.html
(cherry picked from commit 54a8f8f30de2f3132da97c54d07ca4639c405de1)

Update PO files.

Prepare for 1.2.3.

Document umask (RHBZ#582548, RHBZ#583554).
(cherry picked from commit c2a8efc548d6f997049cbb3d63bab360b0d946b6)

docs: Clarify documentation for mknod (RHBZ#582948).
(cherry picked from commit 42e43db927dac13da0afe6dba39708295eb06021)

docs: Add virt-resize(1) link to guestfish(1) manpage.
(cherry picked from commit 92d38b9ae3a0b1b4694da5e3d3a3cb155842cd91)

todo documentation: Idea: List, mount filesystems by UUID and label.
(cherry picked from commit e55400960a6a06eb6dc16ea96f3c30ed580d3d3a)

todo documentation: remove 'virt-grow/shrink', see virt-resize
(cherry picked from commit abb7365d4df7b945846a9f6189bbbd70e10122d1)

todo documentation: virt-rescue now uses $TERM from library, remove from todo list
(cherry picked from commit 398e6830a34e916c52cd23ba6d0b4c00602dfc61)

todo documentation: Remove old section on PPC bugs.

No one cares for PPC, and if they do they should file these
as bugs in the bug tracker.
(cherry picked from commit 3b759fe7fe568a3d643ee99e0aef0adaf163d1e5)