similar to: [PATCH 0/2] Clean up compilation from git a little bit

The embedded copy of ocaml-augeas does Val_int(-1), which in turns triggers warnings in newer GCC versions about "left shift of negative value". The issue actually lies in the OCaml headers (mlvalues.h in particular), and it was fixed in newer OCaml versions. Since the code is actually correct, disable -Wshift-negative-value with -Wno-shift-negative-value (checking whether the compiler

These patches first make src/ warning free, and then turn on the strict warning options. 75 0001-build-suppress-an-ignored-write-return-value-warning.patch 38 0002-build-suppress-an-ignored-dup-return-value-warning.patch 27 0003-generator.ml-suppress-signed-unsigned-compare-warnin.patch 48 0004-build-don-t-perform-arithmetic-on-void-pointers.patch 30

Hello! SSIA - when syslinux sees 'BOOT_IMAGE=' on the append line, something seems to go terribly wrong (didn't investigate in detail). You don't even get to see a boot-prompt, even if you try to force it. pxelinux shows the same problem, so I figure it's something generic in the config parser ... Best regards, Andreas P.S: and thanks for this software ;) -- "The

We routinely test the upstream code by running everything under valgrind, and in any case _FORTIFY_SOURCE is usually defined by downstream Linux distros and we can leave the optimization vs safety decision to them. See this bug: https://bugs.gentoo.org/640494 --- m4/guestfs-c.m4 | 6 ------ 1 file changed, 6 deletions(-) diff --git a/m4/guestfs-c.m4 b/m4/guestfs-c.m4 index 932b6de73..3e8642675

The warnings in daemon were aggravating and risky for development (too easy to miss new ones) so I spent some time last week and today working on removing them. The first patch gets us down to almost no warnings with the original -Wall setting. That was by far the hardest part. Once I'd done that, I enabled nearly all of gcc's warnings via gnulib's warnings and manywarnings modules

Nearly any program that writes to standard output can benefit from this sort of fix. Without it, running e.g., ./guestfish --version > /dev/full would exit successfully, even though it got ENOSPC when writing to the full device. That means regular output redirected to a file on a full partition may also fail to be written, and the error ignored. Before: $ guestfish --version >

Hi Rich, Prompted by your "New commands to list devices by UUID and label" patch, I've adjusted it to use gnulib. While just compiling hash.[ch] is probably ok, in general (given the strict requirements of libguestfs) using the C files of a module is usually not an option, since you don't get the benefit of the module's m4 tests or automake snippets. For other modules,

-- Richard Jones, Emerging Technologies, Red Hat http://et.redhat.com/~rjones virt-top is 'top' for virtual machines. Tiny program with many powerful monitoring features, net stats, disk stats, logging, etc. http://et.redhat.com/~rjones/virt-top -------------- next part -------------- >From e922f13d6b9b4da8040b721b20f858036f7b7e06 Mon Sep 17 00:00:00 2001 From: Richard Jones

Brought to you by the "I haven't rebuilt the libguestfs universe in a while" saga -- now with working test suite. Pino Toscano (5): build: remove extra gnulib submodule build: remove extra checks and submodules build: stop using gnulib in test-harness build: remove unused gnulib modules Remove extra entries from podfiles .gitmodules | 3 -- Makefile.am

OS X does not include pkg-config by default. This causes ./configure to fail when invoking PKG_CHECK_MODULES for libxml2. This change autodetects the path for aclocal, c/o RWMJ noting the real problem is a deficiency in aclocal on OS X. Signed-off-by: Alex Nelson <ajnelson at cs.ucsc.edu> --- bootstrap | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/bootstrap

>From cfab42b40752f6dc44971b3c48523b917b374e91 Mon Sep 17 00:00:00 2001 From: Jim Meyering <meyering at redhat.com> Date: Fri, 4 Sep 2009 18:00:23 +0200 Subject: [PATCH libguestfs] 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

This merely enforces (wrt gnulib) the existing convention that libguestfs is covered by LGPLv2+ . >From 99a8fab0fa0474b4ab3959a5bd5867779a1d08d6 Mon Sep 17 00:00:00 2001 From: Jim Meyering <meyering at redhat.com> Date: Mon, 26 Oct 2009 16:01:06 +0100 Subject: [PATCH libguestfs] build: tell gnulib-tool that this is an lgplv2+ library * bootstrap: Invoke gnulib-tool with --lgpl=2. ---

OS X does not include pkg-config by default. When installed, pkg.m4 tends to end up in some place that is not the autotool-expected /usr/share/aclocal/. Add to bootstrap a path check so pkg.m4 can be found, so ./configure does not fail invoking PKG_CHECK_MODULES. Signed-off-by: Alex Nelson <ajnelson at cs.ucsc.edu> --- bootstrap | 7 ++++++- 1 files changed, 6 insertions(+), 1

Brought to you by the "I haven't rebuilt the libguestfs universe in a while" saga. Pino Toscano (5): build: remove extra gnulib submodule build: remove extra checks and submodules build: stop using gnulib in test-harness build: remove unused gnulib modules Remove extra entries from podfiles .gitmodules | 3 -- Makefile.am | 2 - bootstrap

From: Jim Meyering <meyering at redhat.com> * lib/hivex.c (hivex_node_set_value): Remove unnecessary test-before-free. --- lib/hivex.c | 3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/lib/hivex.c b/lib/hivex.c index d042f4f..a72fa77 100644 --- a/lib/hivex.c +++ b/lib/hivex.c @@ -2748,8 +2748,7 @@ hivex_node_set_value (hive_h *h, hive_node_h node, leave_partial:

ocaml/.depend is automatically generated. This patch removes it from git. --- .gitignore | 1 + ocaml/.depend | 5 ----- 2 files changed, 1 insertions(+), 5 deletions(-) delete mode 100644 ocaml/.depend -------------- next part -------------- A non-text attachment was scrubbed... Name: 0001-Remove-ocaml-.depend-from-git.patch Type: text/x-patch Size: 495 bytes Desc: not available URL:

Although I haven't experienced a specific problem with bootstrapping an empty .depend file, this commit bootstraps it properly using ocamldep. This may or may not solve any real problem, but it's quite simple and more obviously correct. --- bootstrap | 2 +- ocaml/Makefile.am | 10 +++------- ocaml/Makefile.depend | 6 ++++++ 3 files changed, 10 insertions(+), 8

On Tue, Dec 27, 2016 at 4:13 PM, Teresa Johnson <tejohnson at google.com> wrote: > > On Tue, Dec 27, 2016 at 5:23 AM, Carsten Mattner <carstenmattner at gmail.com> wrote: >> >> After figuring out the fault in the configuration step and rebuilding, >> and then rebuilding again by forcing it with `ninja -k 16`, I managed to >> build everything but 12 ninja

On Tue, Dec 27, 2016 at 8:30 AM, Carsten Mattner <carstenmattner at gmail.com> wrote: > On Tue, Dec 27, 2016 at 4:13 PM, Teresa Johnson <tejohnson at google.com> > wrote: > > > > On Tue, Dec 27, 2016 at 5:23 AM, Carsten Mattner < > carstenmattner at gmail.com> wrote: > >> > >> After figuring out the fault in the configuration step and

In an effort to reduce the code, start to use few bits of GLib: - replace the gnulib c-type module - replace the gnulib getprogname module - use g_spawn_sync to launch curl, and drop a file reading function Pino Toscano (6): Include glib.h in p2v.h Use g_ascii_isspace instead of c_isspace from gnulib Use g_get_prgname instead of getprogname from gnulib build: remove no more used gnulib