[libvirt] [PATCH] build: silence gcc warning

Eric Blake eblake at redhat.com
Fri Aug 12 22:15:12 UTC 2011


On 08/12/2011 03:55 PM, Eric Blake wrote:
> util/virpidfile.c: In function 'virPidFileAcquirePath':
> util/virpidfile.c:308:66: error: nested extern declaration of '_gl_verify_function2' [-Wnested-externs]
>
> * src/util/virpidfile.c (virPidFileAcquirePath): Move verify to
> top level.
> ---
>
> Pushing under the build-breaker rule.

Plus this, too:

diff --git c/tests/commandtest.c w/tests/commandtest.c
index 9ba53b8..4a798fc 100644
--- c/tests/commandtest.c
+++ w/tests/commandtest.c
@@ -35,6 +35,7 @@
  #include "memory.h"
  #include "command.h"
  #include "virfile.h"
+#include "virpidfile.h"

  #ifdef WIN32

@@ -214,7 +215,7 @@ cleanup:
  static int test4(const void *unused ATTRIBUTE_UNUSED)
  {
      virCommandPtr cmd = virCommandNew(abs_builddir "/commandhelper");
-    char *pidfile = virFilePid(abs_builddir, "commandhelper");
+    char *pidfile = virPidFileBuildPath(abs_builddir, "commandhelper");
      pid_t pid;
      int ret = -1;

@@ -230,7 +231,7 @@ static int test4(const void *unused ATTRIBUTE_UNUSED)
          goto cleanup;
      }

-    if (virFileReadPid(abs_builddir, "commandhelper", &pid) < 0) {
+    if (virPidFileRead(abs_builddir, "commandhelper", &pid) < 0) {
          printf("cannot read pidfile\n");
          goto cleanup;
      }
@@ -668,7 +669,7 @@ cleanup:
  static int test18(const void *unused ATTRIBUTE_UNUSED)
  {
      virCommandPtr cmd = virCommandNewArgList("sleep", "100", NULL);
-    char *pidfile = virFilePid(abs_builddir, "commandhelper");
+    char *pidfile = virPidFileBuildPath(abs_builddir, "commandhelper");
      pid_t pid;
      int ret = -1;

@@ -686,7 +687,7 @@ static int test18(const void *unused ATTRIBUTE_UNUSED)
      }
      alarm(0);

-    if (virFileReadPid(abs_builddir, "commandhelper", &pid) < 0) {
+    if (virPidFileRead(abs_builddir, "commandhelper", &pid) < 0) {
          printf("cannot read pidfile\n");
          goto cleanup;
      }


-- 
Eric Blake   eblake at redhat.com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org




More information about the libvir-list mailing list