fish/: enable -Werror and all of gcc's warning options

* fish/Makefile.am: Use $(WARN_CFLAGS) $(WERROR_CFLAGS).

generator.ml: avoid signed/unsigned-comparison warning in fish/cmds.c

* src/generator.ml (emit_print_list_function): Emit code that doesn't
evoke warnings.  s/int/unsigned int/
(emit print_*_indent): Likewise, s/int/unsigned int/

destpaths.c: avoid signed/unsigned-comparison warning

* fish/destpaths.c (free_words): Change param type: s/int/size_t/.

fish.c: don't perform arithmetic on void* pointers

* fish/fish.c (xwrite): Use char*.

fish.c: avoid signed/unsigned-comparison warning

* fish/fish.c (script): Change type of index to "unsigned int".

fish.c: avoid "assignment discards qualifiers..." warning

* fish/fish.c (main): Cast-away-const.
* fish/fish.h (bad_cast): Define.  Safer than using an actual cast.

tilde.c: avoid a warning

* fish/tilde.c (find_home_for_username): Change param type: s/int/size_t/
(try_tilde_expansion): Adjust caller.

fish.c: avoid warnings

* fish/rc.c (UNIX_PATH_MAX): Remove unused definition.
* fish/fish.h (rc_listen): Declare with __attribute__((noreturn)).

edit.c: avoid warning about signed/unsigned comparison

* fish/edit.c (load_file): Change type of param from int to size_t.
(do_edit): Adjust caller.

Remove virt-v2v.  New repo: gitorious.org/virt-v2v/

daemon: diagnose socket write failure

* daemon/proto.c (send_chunk): Don't ignore socket-write error.
* daemon/proto.c (send_file_end): Return "int", not void,
so we can propagate send_chunk failure to caller.
* daemon/daemon.h (send_file_end): Update prototype.
* daemon/tar.c (do_tar_out, do_tgz_out): Update uses of send_file_end.
* daemon/upload.c (do_download): Likewise.

tests: Found three more references to the squashfs, replaced with ISO.

tests: namemax on ISO devices is 255

Was 256 for the old squashfs device.

Cleanup initialisation of hash entries in Lib.pm

tests: Use ISO instead of squashfs.

The squashfs format changes too much.  Use an ISO file for
the tests instead.

mke2fs: Use e4fsprogs programs if available.

On RHEL 5, mke2fs is ancient, and there is a non-standard "mke4fs"
binary which acts like the more recent mke2fs on Fedora.  Since there
are several annoyances and actual bugs in the ancient RHEL 5 mke2fs,
use mke4fs instead if it's available.

tests: modprobe fat instead of ext2 module.

On RHEL 5, ext2 is compiled into the kernel, so trying to modprobe
it will fail.  I've checked that fat.ko is always a module on the
platforms we care about (RHEL 5, Fedora 11, Debian), so use that
to test instead.

tests: Conditionally skip UUID tests.

More tests introduced which break with old e2fsprogs that didn't
support UUIDs.  Skip those tests when the old platform is
detected.

Also tidy up this code a little.

avoid build failure due to Haskell keyword clash

* src/generator.ml: Fix this particular problem by
renaming the "module" parameter to "modulename".
Avoid the general problem by ensuring that no parameter name is
in the set of nearly all Haskell, OCaml and C reserved words.
(zfile): Adjust one more offender: s/method/meth/.

Export inspect_linux_kernel in Lib.pm

Add boot/grub_fs to output

If present, this gives the mount point of the filesystem which contains grub.
All entries in grub.conf will be relative to this filesystem.

guestfish: detect more failed syscalls

* fish/fish.c (issue_command): Detect/diagnose more failed syscalls.

Prepare for version 1.0.68.

guestfs_launch: Correct checks for dup failure.

Updated PO files.

build: Comment out some unused macros.

GCC 4.4.1 warns about some unused macros.  Comment these out
while they are not used.

build: new configure-time option: --enable-gcc-warnings

* configure.ac: Define/configure it.
* src/Makefile.am: Use new variables.

suppress warnings from -Wmissing-noreturn

Even though these functions are marked as "not implemented yet",
and they will surely return a value once implemented, ...
* src/guestfs.c (select_add_timeout): Declare with noreturn attribute.
(select_remove_timeout): Likewise.

suppress a warning from -Wswitch-default

* src/guestfs.c (guestfs_end_busy): Add a "default:" label.

suppress signed/unsigned-comparison warnings

* src/guestfs.c [struct guestfs_h] (msg_in_size, msg_in_allocated):
(msg_out_size, msg_out_allocated): Change type from int to unsigned int.

build: don't perform arithmetic on void* pointers

* src/guestfs.c (receive_file_data_sync, xread, xwrite): Use char*.

generator.ml: suppress signed/unsigned-compare warnings

* src/generator.ml (check_reply_header): Emit parameter declarations
that are unsigned, so as to avoid signed/unsigned-compare warnings.

build: suppress an ignored-dup-return-value warning

* src/guestfs.c (guestfs_launch): Handle dup failure.

build: suppress an ignored-write-return-value warning

* bootstrap (modules): Add ignore-value.
* src/guestfs.c: Include "ignore-value.h".
(stdout_event): Ignore failure to write to stderr.
Also, prefer STDERR_FILENO over the literal "2".
* src/Makefile.am (libguestfs_la_CPPFLAGS): Include gnulib's .h files.
(libprotocol_la_CFLAGS): Remove -Wall -Wno-unused.

New 'modprobe' command.

Allow kernel modules to be loaded into the appliance.

avoid compiler warnings about unused vars in generated code

* fish/Makefile.am: Compile rc_protocol.c into a convenience library,
so it can have its own CFLAGS, and link that into guestfish.

generator.ml: Use TABs, not spaces for indentation.

generator: Small code rearrangement.

Move code which updates pod2text memo file into a separate function.

generator.ml: do not emit unused print_* functions

* src/generator.ml: Do not emit functions like print_xattr,
print_lvm_vg, print_inotify_event, that are not used.

generator.ml: do not emit unused print_*_list functions

* src/generator.ml (emit_print_list_function): New function.
Emit a function definition only if it will be used.

generator.ml: avoid warnings in generated tests.c

* src/generator.ml: Avoid warnings in generated tests.c, regarding
print_strings, and in exercising any function that takes a StringList
or DeviceList parameter.
(print_strings): Change param type to "char *const *s".
Declare fully-const initializer for each list-taking function use.
Cast *List argument to "(char **)".

avoid compiler warnings about unused vars in generated code

* src/Makefile.am: Compile protocol.c into a convenience library, so it
can have its own CFLAGS, and link that with the destination one.

Implement 'debug ls' and 'debug ll' commands.

These commands can be used to list files in the appliance.

generator.ml: Use TABs, not spaces for indentation.

Ignore manywarnings.m4 / warnings.m4.

java: Small fix to Java bindings.

Missing declaration of local variable 'i'.

HACKING: Document make targets and ./configure --enable-gcc-warnings.

Remove redundant macro.

indent with spaces, not TABs

daemon: enable -Werror and many gcc warnings when --enable-gcc-warnings

* daemon/m4/gnulib-cache.m4: Add two modules: manywarnings, warnings.
* daemon/configure.ac: Implement --enable-gcc-warnings, and selectively
disable a few warning options that are either not useful or that provoke
too many warnings for now.
Define and AC_SUBST WARN_CFLAGS and WERROR_CFLAGS.
* daemon/Makefile.am (guestfsd_CFLAGS): Use $(WARN_CFLAGS)
and $(WERROR_CFLAGS), rather than just -Wall.

xattr.c: avoid warning about comparison between signed and unsigned

* daemon/xattr.c (getxattrs): Use an unsigned index.

guestfsd: don't ignore failed write-to-socket

* daemon/proto.c (reply): Fix typo that would cause us to ignore
failed write-to-socket.

daemon.h: avoid warning about possible noreturn function

* daemon/daemon.h (main_loop): Use "noreturn" attribute.

sfdisk.c, fallocate.c: use a string literal as format

* daemon/fallocate.c (do_fallocate): Format was not a string literal.
* daemon/sfdisk.c (sfdisk): Likewise.

wc, blockdev: avoid warnings about discarding "const" qualifiers

* daemon/wc.c (wc): Make "flag" param const.
* daemon/blockdev.c (call_blockdev): Likewise, for "switc".

daemon/zero: don't ignore write and close errors

* daemon/zero.c (do_zero): Detect write and close errors.

guestfsd.c: don't perform arithmetic on void pointers

* daemon/guestfsd.c (xread, xwrite): Use char* pointers instead.

command.c: avoid shadowing a global function

* daemon/command.c (do_sh_lines, do_sh): Do not shadow global "command".

avoid warning about old-style no-param function definition

* daemon/df.c (do_df, do_df_h): Add "void".
* sync.c (do_sync): Likewise.

generator.ml: emit slightly prettier code

* src/generator.ml: Emit a few omitted newlines.

python: avoid "_POSIX_C_SOURCE" redefinition warning

* src/generator.ml: Include <Python.h> *before* <stdio.h>
to avoid redefinition warning about "_POSIX_C_SOURCE".

adjust const "**" pointers to avoid warnings

Also, ...
* src/generator.ml: Add DeviceList type, and propagate that change
out to all calling/interface code.

test-tool: Recognize '-t timeout' as an option.

New commands: mkfs-b, mke2journal*, mke2fs-J*

mkfs-b: Pass the -b (blocksize) parameter to mkfs.

mke2journal and friends: Lets you create external ext2 journals on
devices.

mke2fs-J and friends: Lets you create ext2/3/4 filesystems with
external journals.

generator: Generate the UUIDs for tests randomly.

Adds a uuidgen function which gets the random UUID from the
uuid program.  Uses this for tests.

Merge branch '226-elide'

build: avoid "make sytnax-check" failure

* daemon/configure.ac: Change a leading TAB to 8 spaces.

generator.ml: don't emit unused functions

* src/generator.ml: Use "rstructs_used" to emit definitions only for
put_TYPE_list functions that are used.

generator.ml: factor out a function

* src/generator.ml (emit_put_list_function): New function.

Work out which RStruct/RStructList structs are really used, and how.

guestfish: Add --selinux option.

Version 1.0.67.

tests: increase likelihood that heap abuse triggers failure

* regressions/Makefile.am (TESTS_ENVIRONMENT): Always set
MALLOC_PERTURB_ to a random value in 1..255.

appliance: Remove /usr/sbin/tcpd - it's unreadable on RHEL 5.4.

appliance: Replace LANG=C with LC_ALL=C

umount: Enable device name translation for device parameter.

mount: Check mountpoints are absolute paths.

selinux: Don't fail if libselinux is not found.

Misc parameters which are String but should be Pathname.

In rmmountpoint, have to explicitly check for ABS_PATH (*not* NEED_ROOT).

inotify-add-watch does not need to explicitly check for ABS_PATH.

cp/cp-a/mv parameters marked as Pathname.

do_equal does not need to explicitly check for NEED_ROOT/ABS_PATH.

generator.ml: convert leading TABs to spaces

sfdisk: guard against buffer overflow

* daemon/sfdisk.c (sfdisk): Don't let outrageous "extra_flag"
or "device" strings overflow a fixed-size buffer.

do_umount: don't use RESOLVE_DEVICE anymore

* daemon/mount.c (do_umount): Don't use RESOLVE_DEVICE here,
now that the caller always invokes REQUIRE_ROOT_OR_RESOLVE_DEVICE.

generator.ml: factor out "pr_args n" function

generator.ml: finish adding Dev_or_Path support

* src/generator.ml: Update all rules to handle Dev_or_Path.
(the above changes to generator.ml are mostly mechanical)
Emit a use of REQUIRE_ROOT_OR_RESOLVE_DEVICE.
* daemon/upload.c (do_download): Remove use of
REQUIRE_ROOT_OR_RESOLVE_DEVICE, now that it's automatically done
in calling code.
* daemon/file.c (do_file): Likewise.

generator.ml: add type: Dev_or_Path

* src/generator.ml (file, download): Use it.

generator.ml: constify do_mkdtemp

* daemon/dir.c (do_mkdtemp): Rewrite for a "const" parameter.
* src/generator.ml (mkdtemp): Declare parameter to be of type Pathname.

generator.ml: move String/Device decls "down" to definition

generator.ml: use new "Pathname" designation

Nearly every file-related function in daemons/*.c is affected:
Remove this pair of statements from each affected do_* function:
-  NEED_ROOT (return -1);
-  ABS_PATH (dir, return -1);
and change the type of the corresponding parameter to "const char *".
* src/generator.ml: Emit NEED_ROOT just once, even when there are two or
more Pathname args.

generator.ml: new type, "Pathname"

* src/generator.ml: Emit NEED_ROOT and ABS_PATH into generated
stubs.c, rather than requiring they be added manually at the start
of each and every do_* function that operates on a "path" parameter.
Update grammar: Pathname is just a String, with the above exception.
Do not update augeas "path" parameters, since they are not file names,
but rather ":"-separated search paths.  Except aug_init, for which
"path" *is* a file name.

generator.ml: emit "const char *" for Device and String params

* src/generator.ml: Emit "const char *", not "char *" for Device and
String params, now that the non-const code is hoisted into the calling
code in stubs.c.

fix daemon.h's use of NEED_ROOT-in-#define

update all NEED_ROOT uses

run this command:
  git grep -l -w NEED_ROOT|xargs perl -pi -e \
    's/(NEED_ROOT) \((.*?)\)/$1 (return $2)/'

* src/generator.ml: Change all `String "device"' to `Device "device"'.

Then update each affected function, removing each uses of RESOLVE_DEVICE,
now that it's generated in caller from stub.c.
* daemon/blockdev.c (call_blockdev): Remove use of RESOLVE_DEVICE.
* daemon/devsparts.c (do_mkfs): Likewise.
* daemon/ext2.c (do_e2fsck_f, do_get_e2label, do_get_e2uuid): Likewise.
(do_resize2fs, do_set_e2label, do_set_e2uuid, do_tune2fs_l): Likewise.
* daemon/fsck.c (do_fsck): Likewise.
* daemon/grub.c (do_grub_install): Likewise.
* daemon/lvm.c (do_lvremove, do_pvcreate, do_pvremove): Likewise.
(do_pvresize): Likewise.
* daemon/mount.c (do_mount_vfs): Likewise.
* daemon/ntfs.c (do_ntfs_3g_probe): Likewise.
* daemon/scrub.c (do_scrub_device): Likewise.
* daemon/sfdisk.c (sfdisk, sfdisk_flag): Likewise.
* daemon/swap.c (do_mkswap, do_mkswap_L, do_mkswap_U): Likewise.
(do_swapoff_device, do_swapon_device): Likewise.
* daemon/zero.c (do_zero): Likewise.
* daemon/zerofree.c (do_zerofree): Likewise.

update generator to emit each RESOLVE_DEVICE call

* src/generator.ml: Emit RESOLVE_DEVICE (device, goto done);
just before each Device-enabled do_$FUNCTION.

s/NEED_ROOT_OR_IS_DEVICE/REQUIRE_ROOT_OR_RESOLVE_DEVICE/

guestfsd.c: correct s/IS_DEVICE/RESOLVE_DEVICE/ in another comment

manually adjust ABS_PATH uses for new semantics

* daemon/file.c (do_cat): fix an ABS_PATH use