Fix bugs in tests/Makefile.am.

* tests/Makefile.am (TESTS_ENVIRONMENT): Correct invalid
settings of abs_top_builddir and abs_top_srcdir.
Also prepend src/ to PATH, so we test the just-built virsh,
not whatever happens to be in the original $PATH.

* src/xend_internal.c: applied patch from Saori Fukuta adding the
  scheduler API for the Xen daemon driver
* po/POTFILES.in: add the two new linux container modules
Daniel

Initial Linux containers work
* configure.in include/libvirt/virterror.h src/Makefile.am
  src/driver.h src/lxc_conf.[ch] src/lxc_driver.[ch] src/virterror.c:
  Applied 3 patches from Dave Leskovec for intial support of
  Linux containers, configured off by default, work in progress.
* src/libvirt.c: improve virDomainCreateLinux xmlDesc description
Daniel

* src/util.c src/util.h src/xml.c: applied patch from Hiroyuki Kaguchi
  fixing Ethernet MAC addresses parsingand separating as an util
  function, with added comments from Jim Meyering
Daniel

* src/qemu_conf.c src/qemu_conf.h src/qemu_driver.c: applied patch
  from Cole Robinson implementing memory operations for qemu driver
Daniel

Avoid "make syntax-check" failure.

* src/xen_unified.c: Remove trailing blank.

Use explicit logic rules for opening Xen sub-drivers

Treat ENOTSUP like ENODATA, after failed fgetfilecon.

* src/storage_backend.c (virStorageBackendUpdateVolInfoFD): Treat a
failed fgetfilecon with errno == ENOTSUP the same as for ENODATA.

fixed DefinedDomains function comments c.f. 437216
* libvirt.c: fixed DefinedDomains function comments c.f. 437216
* docs/libvirt-*.xml docs/*/libvirt-libvirt.html: regenerated
Daniel

Fix creation of unmanaged pools

Cope with missing selinux attribute

* src/libvirt.c docs/libvirt-*.xml docs/html/libvirt-libvirt.html:
  fixed a few missing argument descriptions and regenerated.
Daniel

* NEWS virsh.1 docs//* include/libvirt/libvirt.h[.in] qemud/mdns.h
  src/libvirt.c src/qemu.conf src/remote_internal.c src/xend_internal.c
  tests/confdata/libvirtd.conf tests/confdata/libvirtd.out: patch from
  Atsushi SAKAI fixing a ot more typo
Daniel

Don't set MAC on TAP devs. Pass TAP device name to QEMU

* po/*: updated translations
Daniel

typo
* src/bridge.c: fix a typo in a comment
Daniel

String fixes #436620
* qemud/qemud.c src/virsh.c: fixing some user facing strings
  problems pointed out by Francesco Tombolini should fix #436620
Daniel

Small cleanup, Daniel

Fix typo "informations" -> "information" (Atsushi SAKAI and
Saori FUKUTA).

* src/libvirt.c: virDomainSetSchedulerParameters need to be
  blocked on read-only connections, patch from Saori Fukuta
Daniel

* src/bridge.c src/virsh.c: typo fixes from Atsushi SAKAI
daniel

* src/qemu_conf.c src/qemu_driver.c: patch from Cole Robinson
  fixing CD Rom change on live QEmu/KVM domains.
Daniel

Remove unused variable

Remove use of polkit-grant. Keep stdio open when running polkit-auth

Don't force Xen connections to readonly when non-root

Don't run cleannup code for parent process in daemon mode

* docs/libvir.html docs/storage.html: apply documentation fixes
  and typos cleanup from Atsushi Sakai
Daniel

* src/xend_internal.c: applied patch from Cole Robinson to not
  loose the boot tag when defining a fully virtualized xen domain
Daniel

Make iSCSI login actually work
* src/storage_backend_iscsi.c: Make iSCSI login actually work
  (Chris Lalancette).

Report error when vcpupin is given bad vCPU number.
* src/virsh.c: Report error when vcpupin is given bad vCPU number
  (Shigeki Sakamoto).

fix typo in comment

Also ignore leading zeros when comparing MAC addresses.

* src/util.c: Include <ctype.h>.
(TOLOWER): Define.
(__virMacAddrCompare): Rewrite to also ignore leading zeros.

* src/capabilities.c: Cole Robinson pointed out a well formedness
  problem when generating the capabilities in the test driver.
Daniel

Don't use first byte of string as a pointer.

* src/qemu_conf.c (qemudReportError): Use the pointer, errorMessage,
not its first byte, errorMessage[0].

Avoid segfault upon early libvirtd failure.

* qemud/qemud.c (main): Don't call qemudCleanup on an
uninitialized pointer.

Release of 0.4.1
* NEWS configure.in libvirt.spec.in docs/* include/libvirt/libvirt.h
  po/*: preparing release of libvirt-0.4.1
Daniel

Plug saslUsernameWhitelist leak.

* qemud/qemud.c (qemudCleanup): Free the array as well as
the strings in it.

Plug skipped-qemudCleanup leak.

* qemud/qemud.c (main): Call qemudCleanup also upon failure.
Otherwise, an error return would skip it and induce leaks.

Plug remoteReadConfigFile leak.

* qemud/qemud.c (remoteReadConfigFile): Be careful to free
"conf" after a successful virConfReadFile.

Plug two leaks: host.migrateTrans and arch.name

* src/capabilities.c (virCapabilitiesFree): Free host.migrateTrans.
(virCapabilitiesFreeGuest): Free arch.name member.

Plug virCapabilities*GuestDomain leak.

* src/capabilities.c (virCapabilitiesFreeGuestDomain): Free dom->type.
(virCapabilitiesAddGuestDomain): Remove duplicate strdup stmt.

missing comments for _virDomainBlockStats fields
* include/libvirt/libvirt.h include/libvirt/libvirt.h.in:
  missing comments for _virDomainBlockStats fields
* docs/*: rebuilt docs
Daniel

fix a broken line split
* qemud/Makefile.am: fix a broken line split
Daniel

Many typos fixed (Atsushi SAKAI).

Rewrite test-coverage rules to accommodate multiple .o files per .c.

* src/Makefile.am (cov): Rewrite rule to merge gcov results corresponding
to two .o files: the libvirt_la- one, and the libvirt_test_la- one.
(tst): Remove unused rule.
* configure.in (LV_LIBTOOL_OBJDIR): Define and AC_SUBST.

Avoid link errors with "configure --disable-shared".

* src/Makefile.am: Create a convenience library, libvirt_test.la,
and don't restrict access to *its* symbols.
* tests/Makefile.am (LDADDS): Add ../src/libvirt_test.la, so that
"configure --disable-shared" no longer provokes link errors.
(LIBVIRT): Remove definition.
(LDADDS): Remove use.
($(LIBVIRT)): Remove rule.
(LDADDS): Use the new convenience library instead.
(CLEANFILES): Define.
* docs/examples/index.py (dump_Makefile): Append $(COVERAGE_LDFLAGS),
to the LDADDS definition, to avoid link error with the combination of
--enable-test-coverage and --disable-shared.
* docs/examples/Makefile.am: Regenerate.
* docs/examples/index.html: Likewise.
* qemud/Makefile.am (libvirtd_LDFLAGS): Append $(COVERAGE_LDFLAGS).
* src/libvirt_sym.version: Remove some SP-before-TAB.

Fix qemudReportError.
* src/qemu_conf.c: applied patch from Cole Robinson to use virErrorMsg
  in qemudReportError.
Daniel

Typo.

Set MAC address on TUN device for Xenner compatability

Make qemud/'s rpcgen rules work.

* qemud/Makefile.am (SUFFIXES): Define to .x before implicit rule.

virMacAddrCompare for comparing MAC addresses
* src/util.c, src/util.h, src/libvirt_sym.version: Added
  virMacAddrCompare utility function for comparing MAC
  addresses.
* src/virsh.c, src/xm_internal.c: Use virMacAddrCompare
  to compare addresses.  (Shigeki Sakamoto and Richard Jones).

Avoid "make distcheck" failure.

* src/Makefile.am (virsh_SOURCES): Add util-lib.h.

Fix a "make syntax-check" failure.

* src/capabilities.c: Include <config.h> from the .c file,
* src/capabilities.h: ... not from the .h file.

Fix typos (Atsushi SAKAI).

Add formal internal capabilities API and update drivers to use it

Implement virDomainBlockStats for QEMU/KVM.
* src/qemu_driver.c: Implement virDomainBlockStats for QEMU/KVM
  using QEMU/KVM 'info blockstats' command (Richard Jones).
* src/qemu_driver.c, src/qemu_conf.h: const-correctness
  fixes (Jim Meyering).

No address attribute in Xen domain XML.
* src/xm_internal.c: handle the case where <mac/> clause
  in Xen domain interface has no address attribute
  (Shigeki Sakamoto).

Oops trailing blank, Daniel

* src/driver.h src/libvirt.c src/openvz_driver.c src/qemu_driver.c
  src/remote_internal.c src/test.c src/util.c src/util.h
  src/xen_unified.c: added probe interface at the driver level,
  use that to improve default hypervisor selection in the absence
  of a given URI when connecting
Daniel

fix compilation against static libraries
* src/Makefile.am src/util-lib.h: fix done with Jim to allow
  compilation of virsh with the static libraries.
Daniel

Avoid new "make syntax-check" failures.

* .x-sc_avoid_write: Exempt src/util-lib.c, too.
* Makefile.maint (sc_unmarked_diagnostics): Filter out false positives.
* src/storage_backend_fs.c (virStorageBackendFileSystemVolCreate):
Use safewrite, not write.
* src/storage_backend_logical.c (virStorageBackendLogicalBuildPool): Likewise.

avoid format string warnings

Avoid warnings like this:
file:nnn:format not a string literal and no format arguments

* qemud/qemud.c: Insert a "%s" format argument.
* qemud/remote.c: Likewise.
* src/iptables.c: Likewise.
* src/qemu_driver.c: Likewise.
* src/storage_backend.c: Likewise.
* src/storage_backend_fs.c: Likewise.
* src/storage_backend_iscsi.c Likewise.
* src/storage_backend_logical.c: Likewise.
* src/storage_conf.c: Likewise.
* src/storage_driver.c: Likewise.

Use safewrite in place of write, in many cases.

Also add "make syntax-check" rules to ensure no new uses sneak in.

There are many uses of write like this:

    if (write (fd, xml, towrite) != towrite)
        return -1;

The problem is that the syscall can succeed, yet write less than
the requested number of bytes, so the caller should retry
rather than simply failing.

This patch changes most of them to use util.c's safewrite wrapper,
which encapsulates the process.  Also, there were a few cases in
which the retry loop was open-coded, and I replaced those, too.

* Makefile.maint (sc_avoid_write): New rule, to avoid recurrence.
* .x-sc_avoid_write: New file.  Record two legitimate exemptions.
* qemud/qemud.c (sig_handler, qemudClientWriteBuf): Use safewrite, not write.
* src/conf.c (__virConfWriteFile): Likewise.
* src/qemu_conf.c (qemudSaveConfig, qemudSaveNetworkConfig): Likewise.
* src/qemu_driver.c (qemudWaitForMonitor, qemudStartVMDaemon)
(qemudVMData, PROC_IP_FORWARD): Likewise.
* proxy/libvirt_proxy.c: Include "util.h".
(proxyWriteClientSocket): Use safewrite.
* src/test.c (testDomainSave, testDomainCoreDump): Likewise.
* src/proxy_internal.c (virProxyWriteClientSocket): Likewise.
* src/virsh.c: Include "util-lib.h".
(vshOutputLogFile): Use safewrite.
* src/console.c: Include "util-lib.h".
(vshRunConsole): Use safewrite.

Move safewrite and saferead to a separate file.

We currently use safewrite from inside libvirt and don't want to publish
any such function name.  However, we do want to use it in applications
like virsh, libvirtd and libvirt_proxy that link with libvirt.  To that
end, this change moves that function definition (along with the nearly
identical saferead) into a new file, util-lib.c.  To avoid maintaining
separate copies of even such small functions, we simply include that new
file from util.c.  Then, the separate applications that need to use
safewrite simply compile and link with util-lib.c.

Of course, this does mean that each of those applications will
containing two copies of these functions.  However, the functions
are so small that it's not worth worrying about that.

* src/util.c (saferead, safewrite): Move function definitions to
util-lib.c and include that .c file.
* src/util-lib.c (saferead, safewrite): New file.  Functions from src/util.c
with slight change (s/int r =/ssize_t r =/) to reflect read/write return type.
* src/util-lib.h: Declare the two moved functions.
* src/util.h: Remove declarations.  Include src/util-lib.h.
* proxy/Makefile.am (libvirt_proxy_SOURCES): Add src/util-lib.c.
* qemud/Makefile.am (libvirtd_SOURCES): Likewise.
* src/Makefile.am (virsh_SOURCES): Add util-lib.c.  Remove some SP-before-TAB.

With --enable-iptables-lokkit=no, avoid warning about unused parameter.

* src/iptables.c (iptRulesSave) [!ENABLE_IPTABLES_LOKKIT]:
Mark parameter as used.

With --without-xen, avoid warning about unused function.

* tests/statstest.c (testQuietError) [!WITH_XEN]: Don't define.

Rewrite openvzSetUUID.

* src/openvz_conf.c (openvzSetUUID): Rewrite to avoid unchecked
lseek, write, and close as well as a potential file descriptor leak.

Handle failed openvzLocateConfDir.

* src/openvz_conf.c (openvzLocateConfDir, openvzGetVPSUUID):
(openvzSetUUID): Don't dereference NULL upon failure.

Fix double-free in OpenVZ driver.
        * src/openvz_conf.c: Double-free fix (Anton Protopopov).

Added missing symbol export

Avoid "make syntax-check" failure.

* docs/examples/index.py: Don't emit trailing blanks.
* docs/examples/Makefile.am: Regenerate.

Update to work with new policy kit APIs

Update po files with latest translatable strings

Ignore libvirt_parthelper binary

Added docs on storage management capabilities

Ignore doc files for syntax check

Added no-op stub of storage APIs to test driver

Added all example storage XML files

Added disk partition storage pool backend

Added iSCSI storage pool backend

Set close-on-exec flag for signal pipe

Added directory/filesystem/netfs based storage pool backend

Added logical volume storage pool driver

Added directory/filesystem/netfs based storage pool backend

Added storage backend helper APIs

Added main internal storage driver impl

Added convenience virsh command for creating pools/volumes without XML

Added virsh commands for storage management

Added python binding for storage APIs

Add storage APIs to remote driver

Add remote daemon support for storage APIs

Regenerate docs to include storage APIs

Internal driver API for storage management

Add public API for storage management

Pull useless-if-before-free from gnulib, and update.

* bootstrap (gnulib_tool): Add useless-if-before-free to
the list of modules.
* build-aux/useless-if-before-free: Update from gnulib.
* gnulib/lib/vasnprintf.c: Work around a bug in HPUX 10.20.
* gnulib/m4/vasnprintf.m4: Likewise.

Ignore ChangeLog/NEWS file for whitespace checks

Network doc example fixes
* docs/libvir.html docs/format.html: fixes the examples on the
  section about ethernet bridging, as raised by Maxwell Bottiger,
  fixes bug #430790
Daniel

Fix gcc-4.3.0 "inlining failed" warning.

* src/internal.h: move xstrol() variants from here ...

* src/util.[ch]: ... to here and rename to virStrToLong()

* src/libvirt_sym.version: export __virStrToLong_i() for
virsh and qemud.

* src/nodeinfo.c, src/stats_linux.c, src/virsh.c,
  src/xend_internal.c, qemud/qemud.c: replace xstrtol()
calls with virStrToLong()

* src/nodeinfo.h: don't include internal.h, which was only
needed for xstrtol(), but instead include libvirt.h which
is suffificient for the declarations in the header.

Enable another syntax-check rule.

* Makefile.cfg (local-checks-to-skip): Remove sc_no_have_config_h.
* qemud/mdns.c: Remove "#ifdef HAVE_CONFIG_H".
* Makefile.maint (sc_no_have_config_h): Tighten up regexp,
so that the above mention of "HAVE_CONFIG_H" doesn't match.
* .x-sc_no_have_config_h: New file, to exempt gnulib/ straggler.

autobuild.sh: Run "make syntax-check", so the autobuilder does this.