Re: [PATCH] build: distribute commandtest files

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 12/10/2010 08:10 AM, Eric Blake wrote:
> * tests/Makefile.am (SUBDIRS): Add commanddata.
> * tests/commanddata/Makefile.am: New file.
> * configure.ac (AC_OUTPUT): Build new makefile.
> Reported by Dominik Klein.
> ---
> 
> This should do the trick.  I'll push it under the build-breaker
> rule if it passes 'make distcheck' (still in progress as I type this).

Nope.  Failed because commanddata.{log,pid} were left around.  I'm
squashing this in (no need to unlink() the .log file before starting
testing, since each test re-opens it with O_CREAT|O_TRUNC):

diff --git a/tests/commandtest.c b/tests/commandtest.c
index e956205..a1bcf68 100644
--- a/tests/commandtest.c
+++ b/tests/commandtest.c
@@ -86,7 +86,7 @@ static int checkoutput(const char *testname)
     ret = 0;

 cleanup:
-    unlink(actuallog);
+    unlink(actualname);
     VIR_FREE(actuallog);
     VIR_FREE(actualname);
     VIR_FREE(expectlog);
@@ -248,6 +248,7 @@ static int test4(const void *unused ATTRIBUTE_UNUSED)

 cleanup:
     virCommandFree(cmd);
+    unlink(pidfile);
     VIR_FREE(pidfile);
     return ret;
 }
@@ -707,12 +708,6 @@ mymain(int argc, char **argv)
                     1, NAME, NULL) < 0)                               \
         ret = -1

-    char *actualname;
-    if (virAsprintf(&actualname, "%s/commandhelper.log", abs_builddir) < 0)
-        return EXIT_FAILURE;
-    unlink(actualname);
-    VIR_FREE(actualname);
-
     DO_TEST(test0);
     DO_TEST(test1);
     DO_TEST(test2);


I'm also posting a followup patch to clean up a memory leak in
virCommand as detected by valgrind.

-- 
Eric Blake   eblake@xxxxxxxxxx    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]