Fix the 'distclean' target.
authorRalph Giles <giles@mozilla.com>
Sat, 3 Sep 2011 23:34:51 +0000 (16:34 -0700)
committerRalph Giles <giles@mozilla.com>
Sat, 3 Sep 2011 23:34:51 +0000 (16:34 -0700)
With subdir-objects, both the top-level makefile, and
libcelt/Makefile, which runs celt-specific unit tests,
were sharing libcelt/.deps for generated dependencies.

Each thought it owned the directory, and so would remove
the files it created in that directory, followed by the
directory itself. The second makefile would then fail to
because the files it created weren't there to remove,
blocking a successful 'make distcheck' with in runs
'make distclean' as part of its verification tests.

We can work around the problem by generating the makefile
for the tests in the actual test directory, so it doesn't
share autotools scratch space with the top level makefile.

Makefile.am
configure.ac
libcelt/Makefile.am [deleted file]
libcelt/tests/Makefile.am [new file with mode: 0644]

index 7a28595..74f0688 100644 (file)
@@ -2,7 +2,7 @@ AUTOMAKE_OPTIONS = subdir-objects
 
 lib_LTLIBRARIES = libopus.la
 
-SUBDIRS = . libcelt
+SUBDIRS = . libcelt/tests
 
 INCLUDES = -I$(top_srcdir)/libcelt -I$(top_srcdir)/silk -I$(top_srcdir)/silk/float -I$(top_srcdir)/silk/fixed
 
index bb7a4d2..d299286 100644 (file)
@@ -208,7 +208,7 @@ fi
 AM_CONDITIONAL([FIXED_POINT], [test x$ac_enable_fixed = xyes])
 AM_CONDITIONAL([CUSTOM_MODES], [test x$ac_enable_custom_modes = xyes])
 
-AC_OUTPUT([Makefile libcelt/Makefile
+AC_OUTPUT([Makefile libcelt/tests/Makefile
            opus.pc opus-uninstalled.pc])
 
 AC_MSG_RESULT([
diff --git a/libcelt/Makefile.am b/libcelt/Makefile.am
deleted file mode 100644 (file)
index 92762d8..0000000
+++ /dev/null
@@ -1,15 +0,0 @@
-INCLUDES = -I$(top_srcdir)/libcelt
-METASOURCES = AUTO
-
-TESTS = type-test ectest cwrs32-test dft-test laplace-test mdct-test mathops-test rotation-test
-
-noinst_PROGRAMS = type-test ectest cwrs32-test dft-test laplace-test mdct-test mathops-test rotation-test
-
-type_test_SOURCES = tests/type-test.c
-ectest_SOURCES = tests/ectest.c
-cwrs32_test_SOURCES = tests/cwrs32-test.c
-dft_test_SOURCES = tests/dft-test.c
-laplace_test_SOURCES = tests/laplace-test.c
-mdct_test_SOURCES = tests/mdct-test.c
-mathops_test_SOURCES = tests/mathops-test.c
-rotation_test_SOURCES = tests/rotation-test.c
diff --git a/libcelt/tests/Makefile.am b/libcelt/tests/Makefile.am
new file mode 100644 (file)
index 0000000..55dcb95
--- /dev/null
@@ -0,0 +1,15 @@
+INCLUDES = -I$(top_srcdir)/libcelt
+METASOURCES = AUTO
+
+TESTS = type-test ectest cwrs32-test dft-test laplace-test mdct-test mathops-test rotation-test
+
+noinst_PROGRAMS = type-test ectest cwrs32-test dft-test laplace-test mdct-test mathops-test rotation-test
+
+type_test_SOURCES = type-test.c
+ectest_SOURCES = ectest.c
+cwrs32_test_SOURCES = cwrs32-test.c
+dft_test_SOURCES = dft-test.c
+laplace_test_SOURCES = laplace-test.c
+mdct_test_SOURCES = mdct-test.c
+mathops_test_SOURCES = mathops-test.c
+rotation_test_SOURCES = rotation-test.c