UBUNTU: [Config] Linearize module/abi checks
authorTim Gardner <tim.gardner@canonical.com>
Mon, 22 Aug 2011 16:00:45 +0000 (10:00 -0600)
committerLeann Ogasawara <leann.ogasawara@canonical.com>
Mon, 2 Apr 2012 20:10:38 +0000 (13:10 -0700)
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 27b3975..06b24a2 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.