From patchwork Mon Oct 29 18:14:14 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: autotools: Attempt to clean build directory when rerunning configure Date: Mon, 29 Oct 2012 18:14:14 -0000 From: Phil Blundell X-Patchwork-Id: 38675 Message-Id: <1351534455.13864.25.camel@phil-desktop> To: oe-core Existing logic in autotools_preconfigure() will clean the build directory before rerunning configure in the case where ${S} != ${B}. It is, however, desirable to make at least some attempt to clean the build artifacts for in-tree builds since many packages lack the appropriate dependency rules to force a recompile if (for example) CFLAGS has changed. Making distclean would be better but is not practical because many packages don't support it properly. Even "make clean" is not universally available, so we ignore any failure from that command and just carry on regardless. Signed-off-by: Phil Blundell --- meta/classes/autotools.bbclass | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/meta/classes/autotools.bbclass b/meta/classes/autotools.bbclass index b97d74b..6539df9 100644 --- a/meta/classes/autotools.bbclass +++ b/meta/classes/autotools.bbclass @@ -93,10 +93,15 @@ CONFIGURESTAMPFILE = "${WORKDIR}/configure.sstate" autotools_preconfigure() { if [ -n "${CONFIGURESTAMPFILE}" -a -e "${CONFIGURESTAMPFILE}" ]; then - if [ "`cat ${CONFIGURESTAMPFILE}`" != "${BB_TASKHASH}" -a "${S}" != "${B}" ]; then - echo "Previously configured separate build directory detected, cleaning ${B}" - rm -rf ${B} - mkdir ${B} + if [ "`cat ${CONFIGURESTAMPFILE}`" != "${BB_TASKHASH}" ]; then + if [ "${S}" != "${B}" ]; then + echo "Previously configured separate build directory detected, cleaning ${B}" + rm -rf ${B} + mkdir ${B} + else + echo "Previously configured source directory detected, cleaning ${S}" + ( oe_runmake clean ) || true + fi fi fi }