fish: Change 'int argc' to 'size_t argc' throughout.

Cherry picked from commit 6391d1a7cfa10337a75465c72d49df3c9ebc65ca
and backported to stable-1.2 branch.

df: Correctly quote libvirt domain and filesystem in --csv mode (RHBZ#639986).

This fixes virt-df --csv when used with libvirt domains that contain
quotes, spaces, commas and other lesser-used characters.
(cherry picked from commit f7475ec577b7ca810185e0fd70a39ac9adaae620)

Freshen POD (manual pages) stylesheet.
(cherry picked from commit 533aff2879d85a4061b9d31e9b25996ce0af60f5)

appliance: Ignore unreadable dbus service file (Fedora 15).
(cherry picked from commit e319b120acec1962319cca534d4504a8999135db)

fish: In guestfish(1) turn command references into links.

Cherry picked from commit 585fceb3350c17a3d6ed7f2daffd5e2b3e5fc3e1
and backported for stable-1.2 branch.

fish: Correction for online help for 'edit' and 'more' commands.

This corrects commit b5c287bcd456bdb02d8ec0443483df34f4fd6b5d
and commit 639ca1828b167bf59353f0cd3c8c79c6289bbd5d.
(cherry picked from commit 3eb76554ef1f2f218f8920d9a8b8fb4ba6b21d66)

configure: Make "fedora-13" the default repository.
(cherry picked from commit 6b2ae9a8b0efba044f4d409b70211aba450a6899)

fish: Update copyright dates in usage message.
(cherry picked from commit 42b62b8240a9b32f0a198105b03cfadaf97a7a7b)

fish: Remove extraneous space from usage message.
(cherry picked from commit 31a2c006cd6486f0ca5afb11e42bcc412385a074)

syntax: Replace _prohibit_regexp with _sc_search_regexp.
(cherry picked from commit 3d994f3d8e12356223d37c1b45ebe2eb39ef28a6)

daemon: Move 'exists', 'is-file' and 'is-dir' to separate file.

This commit is just code movement.
(cherry picked from commit 3a99114360636806078bbf614c241e89661bcc7f)

generator: Fix incorrect shortdesc in docs for 'is-dir' command.
(cherry picked from commit 55b6e18f95950b1a2ec69d549c9e6c8a5758d166)

generator: Fix documentation for 'is-file' command.
(cherry picked from commit 22aa9268f548eab9763311a080859d8cf2257a93)

fish: Fix 'more' command to work with any file.
(cherry picked from commit 639ca1828b167bf59353f0cd3c8c79c6289bbd5d)

fish: Fix 'edit' command to work with any file.

Cherry picked from commit b5c287bcd456bdb02d8ec0443483df34f4fd6b5d.

guestfs: Reference guestfs-browser architecture in threads documentation.
(cherry picked from commit 13be76168260ee57e3d134177066ca7ceb984092)

guestfs: More accurate documentation for initial message.
(cherry picked from commit e2ef068cb7d18f0c0d4931f97332a38ea4ea2477)

guestfs: Remove traces of documentation for non-existent 'low-level API'
(cherry picked from commit 6d9f8f5c140c3116fb0292fcf73f560812c7d5fb)

guestfs: Fix typo in man page.
(cherry picked from commit f3c05da4f9c226c18476eb135dfcb5875d65bf63)

Version 1.2.13.

test-tool: Add a 'run-test-tool-locally' script.

This can be used to run the test tool without installing it.
It sets the environment variables and command line arguments
up correctly to run the tool from the build directory.
(cherry picked from commit 06c906168c0a96662ad47473cfe5f2fb3c316757)

daemon: Don't warn about 'long long' usage.

This removes a fatal warning (-Werror) which stops compilation
on 32 bit machines.
(cherry picked from commit aa96881a512a144f4611a8e7c9801be9bb7ff9cf)

build: guestfs-structs.h was missing from libguestfs_la_SOURCES.
(cherry picked from commit 5fc69ce3ece5e4d4a3da9d78da244c4fa301b5ac)

perl: Add documentation about testing availability of methods and features.
(cherry picked from commit 0f24424f357e854a9da382de11e4fe81305c8743)

Consistent use of 'void *opaque' to refer to opaque pointer in C API.

We inconsistently used 'void *data' or 'void *opaque' all over to
refer to the same thing.  Use 'void *opaque' in all places in the
published API and documentation.

Cherry picked from commit 867319ec5f9030d3c14c32e3302606f2bf11ac27.

build: Don't add version extra string to the version number.

If this string was non-empty, then it broke a lot of things because
autoconf and other parts of the build system were expecting this
string to contain a simple MAJOR.MINOR.RELEASE version number.

This requires changes to guestfish and guestmount so they use the
guestfs_version API to fetch the version from the library.  (The
Perl tools were already doing it this way).  In a way this is more
accurate, because it's no longer hard-coded in the binary, but
fetched from the dynamically linked libguestfs.so.
(cherry picked from commit 4932fdca3ca1e9002164a1c0b73876f32739d34d)

Version 1.2.12.

Rename global 'xdr_str'.

Two bits of XDR both contained a definition called 'str' which
means that 'xdr_str' was being exported globally twice.  Because
of the linker script this didn't affect us.  But it's best to
rename this global so that conflicts cannot arise.
(cherry picked from commit 27ef6f9171127afd248f38d280cf3d0eae51fdaf)

Add -nodefconfig command line option to qemu.

Without this option, qemu will read some defaults from /etc/qemu/
configuration files.

Cherry picked from commit 6d15d4e7c27477507ba355384a340ce86cec617b
and backported to stable 1.2 branch.

Whitespace change: Add blank line between structures and functions.
(cherry picked from commit 4d2f1632ad655130fafab3bf7fe8fa5cb59705d8)

appliance: Remove some obsolete testing rules from Makefile.am

This haven't been used -- and haven't worked -- for a long time.
(cherry picked from commit c56fa58721a710aeb4a41bd73a472354a2128069)

Build workaround for Python 2.4.x in RHEL 5.

See:
http://www.python.org/dev/peps/pep-0353/#conversion-guidelines
(cherry picked from commit 012b1c71f4eda3ae779da51d7d81c199d90f667d)

Version 1.2.11.

generator: Fix typo in error message for RConstOptString.
(cherry picked from commit 29925244c1be2d1d5f71d46eba205278624a1366)

regressions: Don't print misleading 'Expect error ...' lines.

Some tests in the regressions directory deliberately print
error messages.  As long as they still PASS, this is OK.

However these tests also printed some misleading messages
about what error to expect.  Since error messages were slightly
different across distros, these messages were not accurate.

Therefore remove these messages, and replace with a general
message before all tests telling users not to worry about
errors from the tests as long as the tests don't fail.

Cherry picked from commit e7ee6eebed6ee3a5b56c0482ba2770e697d8febc
and rebased for stable branch.

Move variable initialization close to variable use.
(cherry picked from commit 2fd8c259d3daa88b0cdf98090bb57f3dbd178432)

Revert "add_drive_ro adds readonly=on option if available." (RHBZ#617200).

Adding the readonly=on option is not so clever.  This causes
qemu to present the disk as read-only to the guest.  (The
expected behaviour of snapshots=on,readonly=on was that it
would open the disk O_RDONLY but present a writable disk to
the guest).

Since the guest sees a read-only disk, we are unable to do any
recovery if a filesystem on the disk is inconsistent.  This basically
prevents most accesses to live disk images.

What we really want is a qemu option which presents a writable
disk to the guest, but only opens the disk on the host side with
O_RDONLY, to alleviate the udev bug RHBZ#571714.

This reverts commit 676462684e05dd8341dd695762dd99a87d8ec022.
(cherry picked from commit 799d52be4f08f6c70c0e8ba1aa7367ba4cdd78c4)

generator: Remove unnecessary parameter.

The 'name' parameter is not used on the right hand side of the
match, so it can be removed.

Cherry picked from commit 2e7da2a2f3bbc6d6db148d7dc2ce238bf56f34db
and rebased for stable branch.

Use an unsigned type (size_t) for all loop iterators.

This resolves a warning from gcc 4.5:
  assuming signed overflow does not occur when simplifying
    conditional to constant

This page explains the issues in some detail:
  http://www.airs.com/blog/archives/120
(cherry picked from commit 321ca1ef91a90cec5b94058b84420e8018e3f1d8)

generator: Don't hard-code name in DeviceList check.

Only one function currently uses DeviceList.  The generated code
unfortunately hard-coded the argument name from that function.

Cherry picked from aac51942aab63a9355ad6724345ea923148bf2a9
and rebased for stable branch.

build: Don't warn about 'long long'.

Various language bindings simply need this, so we have to allow
it even though it's a GCC extension.
(cherry picked from commit 0c0976496dafda4d172c5a7fc787d6a87d5bce8d)

Version 1.2.10.

tar: Remove redundant use statement.
(cherry picked from commit 70d27f6e796097630134bd8ebc2b65a65b1bf5c4)

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')