From e0ed62e3084b321a8c22ceaf5f677f8ed14fd719 Mon Sep 17 00:00:00 2001 From: Omer Katz Date: Wed, 21 May 2014 11:29:36 +0300 Subject: [PATCH 1/4] Running valgrind in a different build in order to detect memory leaks. --- .travis.yml | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/.travis.yml b/.travis.yml index 9cf85c8..0177dc4 100644 --- a/.travis.yml +++ b/.travis.yml @@ -6,20 +6,24 @@ compiler: matrix: include: - compiler: gcc - env: CONFIGURE_OPTION='--enable-debug' COVERALLS=yes + env: CONFIGURE_OPTION='--enable-debug' COVERALLS=yes VALGRIND=no + - compiler: gcc + env: CONFIGURE_OPTION='--enable-debug' COVERALLS=yes VALGRIND=yes install: - sudo apt-get update -qq - sudo apt-get install -qq automake pkg-config build-essential libtool automake autoconf m4 gnulib - sudo apt-get install -qq check libpcre3 libpcre3-dev libjemalloc-dev libjemalloc1 - sudo apt-get install -qq graphviz-dev graphviz - - if [ x$COVERALLS == xyes ]; then sudo pip install cpp-coveralls --use-mirror; fi + - if [ x$COVERALLS == xyes ]; then sudo pip install cpp-coveralls; fi + - if [x$VALGRIND == xyes ]; then sudo apt-get install valgrind; fi script: - ./autogen.sh - ./configure $CONFIGURE_OPTION - make - - make check + - if [x$VALGRIND == xyes ]; make check 2>&1 > /dev/null; else; make check; fi + - if [x$VALGRIND == xyes ]; then valgrind ./tests/.libs/* -v --trace-children=yes --show-leak-kinds=full --leak-check=full; fi after_success: - if [ x$COVERALLS == xyes ]; then coveralls ; fi From 1c8c63e0a56b2ea1c835c79dc73c759880264353 Mon Sep 17 00:00:00 2001 From: Omer Katz Date: Wed, 21 May 2014 11:44:24 +0300 Subject: [PATCH 2/4] Fixed a bash syntax error. --- .travis.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.travis.yml b/.travis.yml index 0177dc4..296f3eb 100644 --- a/.travis.yml +++ b/.travis.yml @@ -22,7 +22,7 @@ script: - ./autogen.sh - ./configure $CONFIGURE_OPTION - make - - if [x$VALGRIND == xyes ]; make check 2>&1 > /dev/null; else; make check; fi + - if [x$VALGRIND == xyes ]; make check 2>&1 > /dev/null; else make check; fi - if [x$VALGRIND == xyes ]; then valgrind ./tests/.libs/* -v --trace-children=yes --show-leak-kinds=full --leak-check=full; fi after_success: From 86cdf3800ed4efab6c7c71af39283763d7499e51 Mon Sep 17 00:00:00 2001 From: Omer Katz Date: Wed, 21 May 2014 12:00:18 +0300 Subject: [PATCH 3/4] Fixed bash syntax errors. --- .travis.yml | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/.travis.yml b/.travis.yml index 296f3eb..8a15872 100644 --- a/.travis.yml +++ b/.travis.yml @@ -15,15 +15,15 @@ install: - sudo apt-get install -qq automake pkg-config build-essential libtool automake autoconf m4 gnulib - sudo apt-get install -qq check libpcre3 libpcre3-dev libjemalloc-dev libjemalloc1 - sudo apt-get install -qq graphviz-dev graphviz - - if [ x$COVERALLS == xyes ]; then sudo pip install cpp-coveralls; fi - - if [x$VALGRIND == xyes ]; then sudo apt-get install valgrind; fi + - if [ "x$COVERALLS" == xyes ]; then sudo pip install cpp-coveralls; fi + - if[ "x$VALGRIND" == xyes ]; then sudo apt-get install valgrind; fi script: - ./autogen.sh - ./configure $CONFIGURE_OPTION - make - - if [x$VALGRIND == xyes ]; make check 2>&1 > /dev/null; else make check; fi - - if [x$VALGRIND == xyes ]; then valgrind ./tests/.libs/* -v --trace-children=yes --show-leak-kinds=full --leak-check=full; fi + - if [ "x$VALGRIND" == xyes ]; then make check > /dev/null 2>&1; else make check; fi + - if [ "x$VALGRIND" == xyes ]; then valgrind ./tests/.libs/* -v --trace-children=yes --show-leak-kinds=full --leak-check=full; fi after_success: - if [ x$COVERALLS == xyes ]; then coveralls ; fi From cd4d4647deb572b4c60d626296c356135db4dfd8 Mon Sep 17 00:00:00 2001 From: Omer Katz Date: Wed, 21 May 2014 12:02:52 +0300 Subject: [PATCH 4/4] Bash is being touchy. --- .travis.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.travis.yml b/.travis.yml index 8a15872..d27cbd1 100644 --- a/.travis.yml +++ b/.travis.yml @@ -16,7 +16,7 @@ install: - sudo apt-get install -qq check libpcre3 libpcre3-dev libjemalloc-dev libjemalloc1 - sudo apt-get install -qq graphviz-dev graphviz - if [ "x$COVERALLS" == xyes ]; then sudo pip install cpp-coveralls; fi - - if[ "x$VALGRIND" == xyes ]; then sudo apt-get install valgrind; fi + - if [ "x$VALGRIND" == xyes ]; then sudo apt-get install valgrind; fi script: - ./autogen.sh