UBUNTU: [Config] Linearize module/abi checks
authorTim Gardner <tim.gardner@canonical.com>
Mon, 22 Aug 2011 16:00:45 +0000 (10:00 -0600)
committerHerton Ronaldo Krzesinski <herton.krzesinski@canonical.com>
Tue, 27 Sep 2011 19:13:51 +0000 (16:13 -0300)
The module and ABI checks are not independent, so make sure
the modules check is dependent on the  ABI check having been
performed. Also drop some superfluous shortcuts along the way.

Signed-off-by: Tim Gardner <tim.gardner@canonical.com>

debian/rules.d/2-binary-arch.mk
debian/rules.d/4-checks.mk

index 62ec980..0c6a64b 100644 (file)
@@ -43,7 +43,7 @@ install-%: dbgpkgdir = $(CURDIR)/debian/$(bin_pkg_name)-$*-dbgsym
 install-%: basepkg = $(hdrs_pkg_name)
 install-%: hdrdir = $(CURDIR)/debian/$(basepkg)-$*/usr/src/$(basepkg)-$*
 install-%: target_flavour = $*
-install-%: $(stampdir)/stamp-build-% checks-%
+install-%: checks-%
        dh_testdir
        dh_testroot
        dh_clean -k -p$(bin_pkg_name)-$*
index d48c05d..d1c0ad9 100644 (file)
@@ -1,6 +1,4 @@
 # Check ABI for package against last release (if not same abinum)
-abi-%: $(abidir)/%
-       @# Empty for make to be happy
 $(abidir)/%: $(stampdir)/stamp-build-%
        install -d $(abidir)
        sed -e 's/^\(.\+\)[[:space:]]\+\(.\+\)[[:space:]]\(.\+\)$$/\3 \2 \1/'   \
@@ -11,9 +9,7 @@ abi-check-%: $(abidir)/%
                "$(prev_abidir)" "$(abidir)" "$(skipabi)"
 
 # Check the module list against the last release (always)
-module-%: $(abidir)/%.modules
-       @# Empty for make to be happy
-$(abidir)/%.modules: $(stampdir)/stamp-build-%
+$(abidir)/%.modules: abi-check-%
        install -d $(abidir)
        find $(builddir)/build-$*/ -name \*.ko | \
                sed -e 's/.*\/\([^\/]*\)\.ko/\1/' | sort > $@
@@ -22,7 +18,7 @@ module-check-%: $(abidir)/%.modules
        @perl -f $(DROOT)/scripts/module-check "$*" \
                "$(prev_abidir)" "$(abidir)" $(skipmodule)
 
-checks-%: abi-check-% module-check-%
+checks-%: module-check-%
        @# Will be calling more stuff later
 
 # Check the config against the known options list.