libguestfs.git
11 years agoUpdate PO files (no functional change).
Richard Jones [Tue, 15 Sep 2009 16:13:24 +0000 (17:13 +0100)]
Update PO files (no functional change).

11 years agoPrepare for version 1.0.70. 1.0.70
Richard Jones [Tue, 15 Sep 2009 10:13:54 +0000 (11:13 +0100)]
Prepare for version 1.0.70.

11 years agoUpdate PO files.
Richard Jones [Tue, 15 Sep 2009 10:12:45 +0000 (11:12 +0100)]
Update PO files.

11 years agoSet minimum gettext version to 0.17.
Richard Jones [Tue, 15 Sep 2009 10:10:20 +0000 (11:10 +0100)]
Set minimum gettext version to 0.17.

This fixes the problem that the 1.0.69 tarball would fail in
the po/ subdirectory when installing:

make[1]: Entering directory `/builddir/build/BUILD/libguestfs-1.0.69/po'
/bin/sh @MKINSTALLDIRS@ /builddir/build/BUILDROOT/libguestfs-1.0.69-1.fc11.x86_64/usr/share
/bin/sh: @MKINSTALLDIRS@: No such file or directory
make[1]: Leaving directory `/builddir/build/BUILD/libguestfs-1.0.69/po'

RHEL 5.3 has gettext 0.14.6.  However the tarball will still
build (since it includes all the required code).  This may cause
a problem if RHEL users want to build from the git repository.

11 years agoFix comparison between signed and unsigned (for RHEL 5). 1.0.69
Richard Jones [Tue, 15 Sep 2009 08:23:49 +0000 (09:23 +0100)]
Fix comparison between signed and unsigned (for RHEL 5).

11 years agoMove decl out for C99 compliance.
Richard Jones [Mon, 14 Sep 2009 19:14:47 +0000 (20:14 +0100)]
Move decl out for C99 compliance.

11 years agoUpdate PO files.
Richard Jones [Mon, 14 Sep 2009 18:35:58 +0000 (19:35 +0100)]
Update PO files.

11 years agoMove guestfs-internal-actions.h to EXTRA_DIST.
Richard Jones [Mon, 14 Sep 2009 18:35:46 +0000 (19:35 +0100)]
Move guestfs-internal-actions.h to EXTRA_DIST.

11 years agoPrepare for 1.0.69.
Richard Jones [Mon, 14 Sep 2009 18:28:14 +0000 (19:28 +0100)]
Prepare for 1.0.69.

11 years agoRemove main loop.
Richard Jones [Fri, 11 Sep 2009 19:00:30 +0000 (20:00 +0100)]
Remove main loop.

This commit removes the external main loop, which never worked
and caused a number of bugs.  Requests are now done synchronously,
and if the user wants to have requests issued in the background
or to have a responsive GUI, then they'll just have to use threads.

The big change is to push all reads and writes through two
functions called send_to_daemon (for writes) and recv_from_daemon
(for reads) which operate synchronously.  These functions
read/write whole messages, and also handle checking for EOF
(ie. daemon died) and asynchronous log message events from
qemu (eg. from debug / dmesg printed by the guest).  A more
complete description of how these work can be found in the code.

This code passes a complete run of the tests.

Bugs believed to be fixed by this commit:

  https://bugzilla.redhat.com/show_bug.cgi?id=501888
  internal error: reply callback called twice

  https://bugzilla.redhat.com/show_bug.cgi?id=504418
  In virt-inspector: "download: guestfs_download reply failed, see earlier error messages"

I have tried to avoid reintroducing this:

  https://bugzilla.redhat.com/show_bug.cgi?id=508713
  libguestfs: error: write: Broken pipe (guestfish only)

One other benefit of this is that 'set_busy/end_busy' calls
no longer appear in traces.

11 years agoDon't enable trace in set_trace test.
Richard Jones [Mon, 14 Sep 2009 15:21:04 +0000 (16:21 +0100)]
Don't enable trace in set_trace test.

This leaves trace enabled afterwards, resulting in some
ugly test messages.

11 years agoFix type punning warning about use of CMSG_DATA in Rawhide.
Richard Jones [Mon, 14 Sep 2009 14:43:23 +0000 (15:43 +0100)]
Fix type punning warning about use of CMSG_DATA in Rawhide.

11 years agoguestfish: Enable grouping in string lists
Matthew Booth [Fri, 11 Sep 2009 08:27:57 +0000 (09:27 +0100)]
guestfish: Enable grouping in string lists

This change adds the ability to group entries in a string list with single
quotes. So the string:
  "'foo bar'"
becomes 1 token rather than 2. Consequently single quotes must now be escaped:
  "\'"
resolves to a literal single quote.

Incidentally, this change also alters another, probably unintentional behaviour
of the previous implementation, in that tokens are separated by any amount of
whitespace rather than a single whitespace character. I.e.:
  "a  b"
resolves to:
  'a' 'b'
rather than:
  'a' '' 'b'
That last syntax can be used if an empty argument is still desired. Whitespace
is now also defined to include tabs.

parse_string_list can also now fail if it contains an unmatched open quote.

11 years agoIgnore localrepo/ directory.
Richard Jones [Mon, 14 Sep 2009 12:41:02 +0000 (13:41 +0100)]
Ignore localrepo/ directory.

This can be used by people building from source and packagers for
their own purposes.

11 years agoAdd command trace functionality.
Richard Jones [Wed, 9 Sep 2009 16:48:30 +0000 (17:48 +0100)]
Add command trace functionality.

Enable this by calling guestfs_trace (handle, 1) or by
setting the LIBGUESTFS_TRACE=1 environment variable.

11 years agoNon-daemon actions indirect through generated code.
Richard Jones [Wed, 9 Sep 2009 11:34:13 +0000 (12:34 +0100)]
Non-daemon actions indirect through generated code.

Previously non-daemon actions were called directly by
user code, eg:

/* Non-generated */
int
guestfs_set_verbose (guestfs_h *g, int v)
{
  g->verbose = !!v;
  return 0;
}

This changes these actions so they go indirectly via
some generated code, eg:

/* Generated */
int guestfs_set_verbose (guestfs_h *g,
int verbose)
{
  return guestfs__set_verbose (g, verbose);
}

/* Non-generated */
int
guestfs__set_verbose (guestfs_h *g, int v)
{
  g->verbose = !!v;
  return 0;
}

The aim is to have a place in the generated code where
we can add debug or tracing information for these non-
daemon calls.

11 years agoguestfish: Redirect stdout when executing remote commands
Matthew Booth [Fri, 11 Sep 2009 16:04:13 +0000 (17:04 +0100)]
guestfish: Redirect stdout when executing remote commands

guestfish --listen necessarily redirects its stdout to /dev/null so as not to
interfere with eval. The remote protocol doesn't contain any other provision for
collecting stdout for the caller, so executing guestfish --remote will never
generate any output.

This patch fixes that by forwarding the caller's STDOUT to the listener over the
unix socket connection. The listener redirects its STDOUT to the caller's STDOUT
for the duration of the command, then closes it again.

11 years agoUpdate TODO list
Richard Jones [Sat, 12 Sep 2009 08:45:44 +0000 (09:45 +0100)]
Update TODO list

11 years agoAdd diffutils package.
Richard Jones [Fri, 11 Sep 2009 19:21:37 +0000 (20:21 +0100)]
Add diffutils package.

On Fedora 12, /usr/bin/cmp is not pulled in unless we explicitly
add the diffutils package.

11 years agoAdd echo_daemon command
Matthew Booth [Fri, 11 Sep 2009 13:27:54 +0000 (14:27 +0100)]
Add echo_daemon command

echo_daemon is a simple echo which can be used to test connectivity between the
client and daemon.

11 years agogenerator.ml: Fix string list memory leak
Matthew Booth [Fri, 11 Sep 2009 09:33:32 +0000 (10:33 +0100)]
generator.ml: Fix string list memory leak

Parsed string lists are allocated by malloc, but were never freed.

11 years agoIgnore some more m4 files.
Richard Jones [Wed, 9 Sep 2009 11:03:10 +0000 (12:03 +0100)]
Ignore some more m4 files.

11 years agoDon't include gnulib files in POTFILES.in.
Richard Jones [Wed, 9 Sep 2009 11:01:42 +0000 (12:01 +0100)]
Don't include gnulib files in POTFILES.in.

11 years agoavoid build-from-scratch failure due to missing daemon/configure
Jim Meyering [Fri, 4 Sep 2009 16:00:23 +0000 (18:00 +0200)]
avoid build-from-scratch failure due to missing daemon/configure

* bootstrap: Don't use autoreconf's --norecursive
option.  We require the default --recursive behavior in order
to create daemon/configure.  Reported by Matthew Booth.

11 years agomaint: guestfs.c: avoid warning about possible NULL deref from llvm/clang
Jim Meyering [Mon, 31 Aug 2009 18:29:08 +0000 (20:29 +0200)]
maint: guestfs.c: avoid warning about possible NULL deref from llvm/clang

* src/guestfs.h (guestfs_abort_cb): Declare with attribute noreturn.

11 years agomaint: guestfs.c: remove unnecessary initialization
Jim Meyering [Mon, 31 Aug 2009 17:51:46 +0000 (19:51 +0200)]
maint: guestfs.c: remove unnecessary initialization

* src/guestfs.c (guestfs__receive_file_sync): Don't set "r",
only to ignore it.

11 years agogenerator.ml: avoid a warning about signed overflow in tests.c
Jim Meyering [Fri, 28 Aug 2009 12:07:35 +0000 (14:07 +0200)]
generator.ml: avoid a warning about signed overflow in tests.c

* src/generator.ml: Emit "unsigned long int n_failed;" rather than
"int failed;", to avoid warning from gcc about "assuming signed
overflow does not occur when simplifying conditional to constant".

11 years agogenerator.ml: avoid defined-but-not-used warnings in guestfs_c_actions.c
Jim Meyering [Fri, 28 Aug 2009 11:53:36 +0000 (13:53 +0200)]
generator.ml: avoid defined-but-not-used warnings in guestfs_c_actions.c

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

11 years agogenerator.ml: avoid warnings in generated ocaml/guestfs_c_actions.c
Jim Meyering [Fri, 28 Aug 2009 11:43:16 +0000 (13:43 +0200)]
generator.ml: avoid warnings in generated ocaml/guestfs_c_actions.c

* src/generator.ml: Emit prototypes for ocaml_guestfs_* functions,
to avoid warnings from gcc -Wmissing-prototypes.  Normally we'd put
these somewhere else, but in this unusual case, they're not needed
anywhere else.  Handle the >5-argument case, too, for these:
ocaml_guestfs_test0_byte, ocaml_guestfs_sfdisk_byte,
ocaml_guestfs_sfdisk_N_byte.

11 years agobuild: enable gcc warnings in capitests/ and ocaml/
Jim Meyering [Mon, 24 Aug 2009 09:39:42 +0000 (11:39 +0200)]
build: enable gcc warnings in capitests/ and ocaml/

* capitests/Makefile.am: Use $(WARN_CFLAGS) and $(WERROR_CFLAGS).
* ocaml/Makefile.am:: Likewise.

11 years agoocaml/guestfs_c.c: avoid warning about missing prototypes
Jim Meyering [Thu, 27 Aug 2009 11:23:20 +0000 (13:23 +0200)]
ocaml/guestfs_c.c: avoid warning about missing prototypes

* ocaml/guestfs_c.c (ocaml_guestfs_create, ocaml_guestfs_close): Declare.

11 years agoocaml/guestfs_c.c: avoid warning about initialization discarding "const"
Jim Meyering [Thu, 27 Aug 2009 10:36:25 +0000 (12:36 +0200)]
ocaml/guestfs_c.c: avoid warning about initialization discarding "const"

* ocaml/guestfs_c.c (guestfs_custom_operations): Add a cast.

11 years agoocaml/guestfs_c.c avoid signed/unsigned-comparison warning
Jim Meyering [Thu, 27 Aug 2009 10:34:38 +0000 (12:34 +0200)]
ocaml/guestfs_c.c avoid signed/unsigned-comparison warning

* ocaml/guestfs_c.c (ocaml_guestfs_strings_val): Declare index as
unsigned int.

11 years agobuild: use only one m4/ directory
Jim Meyering [Tue, 25 Aug 2009 14:21:48 +0000 (16:21 +0200)]
build: use only one m4/ directory

* Makefile.am (ACLOCAL_AMFLAGS): Specify only one include dir: m4.
* bootstrap: Tell gnulib-tool to put .m4 files in m4/, not gnulib/m4.
* autogen.sh: Move autoreconf from here into...
* bootstrap: ...here, so that it is run only when gnulib-tool is.
Also, tell it to skip the usual autopoint and libtoolize runs.
* m4/.gitignore: Update.

11 years agobuild: invoke autopoint with --force
Jim Meyering [Tue, 25 Aug 2009 07:10:21 +0000 (09:10 +0200)]
build: invoke autopoint with --force

* bootstrap: Invoke autopoint with --force, to avoid warning
about existing build-aux/config.rpath.
Invoke libtoolize before gnulib-tool, to avoid spurious warnings.
* autogen.sh: Add comments.
Remove build-aux/config.rpath before running autoreconf.

11 years agobuild: update gnulib submodule to latest
Jim Meyering [Tue, 25 Aug 2009 07:43:04 +0000 (09:43 +0200)]
build: update gnulib submodule to latest

* .gnulib: Update to latest.

11 years agobuild: don't define _GNU_SOURCE manually
Jim Meyering [Fri, 21 Aug 2009 17:44:36 +0000 (19:44 +0200)]
build: don't define _GNU_SOURCE manually

Now that we're using gnulib in earnest, any manual definition
would provoke a redefinition warning.
* fish/fish.c (_GNU_SOURCE): Don't define.
* fish/destpaths.c (_GNU_SOURCE): Likewise.
* src/guestfs.c (_GNU_SOURCE): Likewise.
* bootstrap (modules): Add asprintf, strchrnul, strerror, strndup
and vasprintf.
* fish/fish.c (main): Set argv[0] to sanitized program_name, so
functions like getopt_long that use argv[0] use the clean name.

11 years agoguestfish: diagnose stdout write failure
Jim Meyering [Fri, 21 Aug 2009 17:19:39 +0000 (19:19 +0200)]
guestfish: diagnose stdout write failure

Use gnulib's closeout module to ensure any failure to write to
stdout is detected and reported.
* fish/fish.c: Include "closeout.h".
(main): Call atexit (close_stdout);
* bootstrap (modules): Add closeout.

11 years agoguestfish: don't try to diagnose getopt failure
Jim Meyering [Mon, 24 Aug 2009 09:32:17 +0000 (11:32 +0200)]
guestfish: don't try to diagnose getopt failure

* fish/fish.c: ... getopt_long already does that.  Instead, suggest
"Try `guestfish --help' for more information."

11 years agoguestfish: write --help to stdout, use gnulib's progname module
Jim Meyering [Fri, 21 Aug 2009 17:05:20 +0000 (19:05 +0200)]
guestfish: write --help to stdout, use gnulib's progname module

* fish/fish.c: Include "progname.h".
(main): Call set_program_name to initialize.
Don't hard-code guestfish everywhere.  Use program_name.
However, be careful when modifying argv[0], since it is used
in the hopes that it is an absolute file name.
(usage): Don't spew all of --help for a mis-typed option.
Split long lines.

11 years agobuild: avoid some autoconf warnings
Jim Meyering [Mon, 24 Aug 2009 09:34:22 +0000 (11:34 +0200)]
build: avoid some autoconf warnings

* configure.ac: Move gl_EARLY and gl_INIT to be earlier.

11 years agoTodo: ntfsclone.
Richard Jones [Mon, 24 Aug 2009 10:45:49 +0000 (11:45 +0100)]
Todo: ntfsclone.

11 years agoAdd waitpid along guestfs_close path (RHBZ#518747).
Richard Jones [Sat, 22 Aug 2009 09:40:42 +0000 (10:40 +0100)]
Add waitpid along guestfs_close path (RHBZ#518747).

11 years agofish/: enable -Werror and all of gcc's warning options
Jim Meyering [Thu, 20 Aug 2009 11:56:47 +0000 (13:56 +0200)]
fish/: enable -Werror and all of gcc's warning options

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

11 years agogenerator.ml: avoid signed/unsigned-comparison warning in fish/cmds.c
Jim Meyering [Fri, 21 Aug 2009 13:04:32 +0000 (15:04 +0200)]
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/

11 years agodestpaths.c: avoid signed/unsigned-comparison warning
Jim Meyering [Fri, 21 Aug 2009 13:02:16 +0000 (15:02 +0200)]
destpaths.c: avoid signed/unsigned-comparison warning

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

11 years agofish.c: don't perform arithmetic on void* pointers
Jim Meyering [Fri, 21 Aug 2009 12:59:49 +0000 (14:59 +0200)]
fish.c: don't perform arithmetic on void* pointers

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

11 years agofish.c: avoid signed/unsigned-comparison warning
Jim Meyering [Fri, 21 Aug 2009 12:58:20 +0000 (14:58 +0200)]
fish.c: avoid signed/unsigned-comparison warning

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

11 years agofish.c: avoid "assignment discards qualifiers..." warning
Jim Meyering [Fri, 21 Aug 2009 12:55:39 +0000 (14:55 +0200)]
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.

11 years agotilde.c: avoid a warning
Jim Meyering [Fri, 21 Aug 2009 12:50:23 +0000 (14:50 +0200)]
tilde.c: avoid a warning

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

11 years agofish.c: avoid warnings
Jim Meyering [Fri, 21 Aug 2009 12:45:13 +0000 (14:45 +0200)]
fish.c: avoid warnings

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

11 years agoedit.c: avoid warning about signed/unsigned comparison
Jim Meyering [Fri, 21 Aug 2009 12:41:49 +0000 (14:41 +0200)]
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.

11 years agoRemove virt-v2v. New repo: http://gitorious.org/virt-v2v/
Richard Jones [Thu, 20 Aug 2009 15:28:35 +0000 (16:28 +0100)]
Remove virt-v2v.  New repo: gitorious.org/virt-v2v/

11 years agodaemon: diagnose socket write failure
Jim Meyering [Thu, 20 Aug 2009 10:29:46 +0000 (12:29 +0200)]
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.

11 years agotests: Found three more references to the squashfs, replaced with ISO.
Richard W.M. Jones [Wed, 19 Aug 2009 20:37:56 +0000 (21:37 +0100)]
tests: Found three more references to the squashfs, replaced with ISO.

11 years agotests: namemax on ISO devices is 255 1.0.68
Richard Jones [Wed, 19 Aug 2009 16:51:40 +0000 (17:51 +0100)]
tests: namemax on ISO devices is 255

Was 256 for the old squashfs device.

11 years agoCleanup initialisation of hash entries in Lib.pm
Matthew Booth [Wed, 19 Aug 2009 14:51:27 +0000 (15:51 +0100)]
Cleanup initialisation of hash entries in Lib.pm

11 years agotests: Use ISO instead of squashfs.
Richard Jones [Wed, 19 Aug 2009 14:09:27 +0000 (15:09 +0100)]
tests: Use ISO instead of squashfs.

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

11 years agomke2fs: Use e4fsprogs programs if available.
Richard W.M. Jones [Wed, 19 Aug 2009 13:03:35 +0000 (14:03 +0100)]
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.

11 years agotests: modprobe fat instead of ext2 module.
Richard W.M. Jones [Wed, 19 Aug 2009 13:02:08 +0000 (14:02 +0100)]
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.

11 years agotests: Conditionally skip UUID tests.
Richard W.M. Jones [Wed, 19 Aug 2009 13:01:13 +0000 (14:01 +0100)]
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.

11 years agoavoid build failure due to Haskell keyword clash
Jim Meyering [Wed, 19 Aug 2009 12:30:02 +0000 (14:30 +0200)]
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/.

11 years agoExport inspect_linux_kernel in Lib.pm
Matthew Booth [Wed, 19 Aug 2009 11:52:16 +0000 (12:52 +0100)]
Export inspect_linux_kernel in Lib.pm

11 years agoAdd boot/grub_fs to output
Matthew Booth [Wed, 19 Aug 2009 10:42:46 +0000 (11:42 +0100)]
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.

11 years agoguestfish: detect more failed syscalls
Jim Meyering [Wed, 19 Aug 2009 08:01:07 +0000 (10:01 +0200)]
guestfish: detect more failed syscalls

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

11 years agoPrepare for version 1.0.68.
Richard Jones [Wed, 19 Aug 2009 08:44:51 +0000 (09:44 +0100)]
Prepare for version 1.0.68.

11 years agoguestfs_launch: Correct checks for dup failure.
Richard Jones [Wed, 19 Aug 2009 08:37:44 +0000 (09:37 +0100)]
guestfs_launch: Correct checks for dup failure.

11 years agoUpdated PO files.
Richard Jones [Wed, 19 Aug 2009 08:10:24 +0000 (09:10 +0100)]
Updated PO files.

11 years agobuild: Comment out some unused macros.
Richard Jones [Wed, 19 Aug 2009 08:09:47 +0000 (09:09 +0100)]
build: Comment out some unused macros.

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

11 years agobuild: new configure-time option: --enable-gcc-warnings
Jim Meyering [Tue, 18 Aug 2009 12:56:48 +0000 (14:56 +0200)]
build: new configure-time option: --enable-gcc-warnings

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

11 years agosuppress warnings from -Wmissing-noreturn
Jim Meyering [Tue, 18 Aug 2009 13:39:26 +0000 (15:39 +0200)]
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.

11 years agosuppress a warning from -Wswitch-default
Jim Meyering [Tue, 18 Aug 2009 13:35:32 +0000 (15:35 +0200)]
suppress a warning from -Wswitch-default

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

11 years agosuppress signed/unsigned-comparison warnings
Jim Meyering [Tue, 18 Aug 2009 13:32:40 +0000 (15:32 +0200)]
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.

11 years agobuild: don't perform arithmetic on void* pointers
Jim Meyering [Tue, 18 Aug 2009 13:23:35 +0000 (15:23 +0200)]
build: don't perform arithmetic on void* pointers

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

11 years agogenerator.ml: suppress signed/unsigned-compare warnings
Jim Meyering [Tue, 18 Aug 2009 13:12:26 +0000 (15:12 +0200)]
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.

11 years agobuild: suppress an ignored-dup-return-value warning
Jim Meyering [Tue, 18 Aug 2009 13:03:55 +0000 (15:03 +0200)]
build: suppress an ignored-dup-return-value warning

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

11 years agobuild: suppress an ignored-write-return-value warning
Jim Meyering [Tue, 18 Aug 2009 12:55:59 +0000 (14:55 +0200)]
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.

11 years agoNew 'modprobe' command.
Matthew Booth [Tue, 18 Aug 2009 14:06:20 +0000 (15:06 +0100)]
New 'modprobe' command.

Allow kernel modules to be loaded into the appliance.

11 years agoavoid compiler warnings about unused vars in generated code
Jim Meyering [Tue, 18 Aug 2009 08:15:57 +0000 (10:15 +0200)]
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.

11 years agogenerator: Small code rearrangement.
Richard Jones [Tue, 18 Aug 2009 08:16:56 +0000 (09:16 +0100)]
generator: Small code rearrangement.

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

11 years agogenerator.ml: do not emit unused print_* functions
Jim Meyering [Mon, 17 Aug 2009 18:44:01 +0000 (20:44 +0200)]
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.

11 years agogenerator.ml: do not emit unused print_*_list functions
Jim Meyering [Mon, 17 Aug 2009 18:28:34 +0000 (20:28 +0200)]
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.

11 years agogenerator.ml: avoid warnings in generated tests.c
Jim Meyering [Mon, 17 Aug 2009 16:47:46 +0000 (18:47 +0200)]
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 **)".

11 years agoavoid compiler warnings about unused vars in generated code
Jim Meyering [Mon, 17 Aug 2009 16:44:37 +0000 (18:44 +0200)]
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.

11 years agoImplement 'debug ls' and 'debug ll' commands.
Richard Jones [Tue, 18 Aug 2009 07:56:18 +0000 (08:56 +0100)]
Implement 'debug ls' and 'debug ll' commands.

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

11 years agogenerator.ml: Use TABs, not spaces for indentation.
Jim Meyering [Mon, 17 Aug 2009 19:14:34 +0000 (21:14 +0200)]
generator.ml: Use TABs, not spaces for indentation.

11 years agoIgnore manywarnings.m4 / warnings.m4.
Richard W.M. Jones [Mon, 17 Aug 2009 11:27:43 +0000 (12:27 +0100)]
Ignore manywarnings.m4 / warnings.m4.

11 years agojava: Small fix to Java bindings.
Richard W.M. Jones [Mon, 17 Aug 2009 11:26:58 +0000 (12:26 +0100)]
java: Small fix to Java bindings.

Missing declaration of local variable 'i'.

11 years agoHACKING: Document make targets and ./configure --enable-gcc-warnings.
Richard W.M. Jones [Mon, 17 Aug 2009 11:23:32 +0000 (12:23 +0100)]
HACKING: Document make targets and ./configure --enable-gcc-warnings.

11 years agoRemove redundant macro.
Richard W.M. Jones [Mon, 17 Aug 2009 11:23:16 +0000 (12:23 +0100)]
Remove redundant macro.

11 years agoindent with spaces, not TABs
Jim Meyering [Mon, 17 Aug 2009 09:56:10 +0000 (11:56 +0200)]
indent with spaces, not TABs

11 years agodaemon: enable -Werror and many gcc warnings when --enable-gcc-warnings
Jim Meyering [Mon, 17 Aug 2009 08:35:05 +0000 (10:35 +0200)]
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.

11 years agoxattr.c: avoid warning about comparison between signed and unsigned
Jim Meyering [Mon, 17 Aug 2009 08:30:52 +0000 (10:30 +0200)]
xattr.c: avoid warning about comparison between signed and unsigned

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

11 years agoguestfsd: don't ignore failed write-to-socket
Jim Meyering [Mon, 17 Aug 2009 08:25:42 +0000 (10:25 +0200)]
guestfsd: don't ignore failed write-to-socket

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

11 years agodaemon.h: avoid warning about possible noreturn function
Jim Meyering [Mon, 17 Aug 2009 07:51:52 +0000 (09:51 +0200)]
daemon.h: avoid warning about possible noreturn function

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

11 years agosfdisk.c, fallocate.c: use a string literal as format
Jim Meyering [Mon, 17 Aug 2009 07:43:18 +0000 (09:43 +0200)]
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.

11 years agowc, blockdev: avoid warnings about discarding "const" qualifiers
Jim Meyering [Mon, 17 Aug 2009 07:40:16 +0000 (09:40 +0200)]
wc, blockdev: avoid warnings about discarding "const" qualifiers

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

11 years agodaemon/zero: don't ignore write and close errors
Jim Meyering [Mon, 17 Aug 2009 07:38:50 +0000 (09:38 +0200)]
daemon/zero: don't ignore write and close errors

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

11 years agoguestfsd.c: don't perform arithmetic on void pointers
Jim Meyering [Mon, 17 Aug 2009 07:24:47 +0000 (09:24 +0200)]
guestfsd.c: don't perform arithmetic on void pointers

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

11 years agocommand.c: avoid shadowing a global function
Jim Meyering [Mon, 17 Aug 2009 07:16:20 +0000 (09:16 +0200)]
command.c: avoid shadowing a global function

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