Merge lp:~tplavcic/percona-server/bug1314568-5.6 into lp:percona-server/5.6

Proposed by Tomislav Plavcic
Status: Superseded
Proposed branch: lp:~tplavcic/percona-server/bug1314568-5.6
Merge into: lp:percona-server/5.6
Diff against target: 43 lines (+13/-2)
2 files modified
build-ps/build-binary.sh (+12/-2)
configure.cmake (+1/-0)
To merge this branch: bzr merge lp:~tplavcic/percona-server/bug1314568-5.6
Reviewer Review Type Date Requested Status
Laurynas Biveinis (community) Needs Fixing
Review via email: mp+220916@code.launchpad.net

This proposal has been superseded by a proposal from 2014-05-27.

Description of the change

Initialised variable COMMON_FLAGS which was set only on centos so the script refused to run on Ubuntu
Added -DHAVE_purify in configure.cmake for valgrind build (agreed with Laurynas over mail)
Tested locally by running the script on Ubuntu

To post a comment you must log in.
Revision history for this message
Laurynas Biveinis (laurynas-biveinis) wrote :

The middle diff chunk is from another diff?

Will need a re-merge from fixed 5.5.

review: Needs Fixing

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'build-ps/build-binary.sh'
--- build-ps/build-binary.sh 2014-04-25 15:30:49 +0000
+++ build-ps/build-binary.sh 2014-05-27 06:43:06 +0000
@@ -23,7 +23,9 @@
23OPENSSL_LIBRARY=''23OPENSSL_LIBRARY=''
24CRYPTO_LIBRARY=''24CRYPTO_LIBRARY=''
25TAG=''25TAG=''
2626#
27COMMON_FLAGS=''
28#
27# Some programs that may be overriden29# Some programs that may be overriden
28TAR=${TAR:-tar}30TAR=${TAR:-tar}
2931
@@ -144,7 +146,15 @@
144PRODUCT="Percona-Server-$MYSQL_VERSION-$PERCONA_SERVER_VERSION"146PRODUCT="Percona-Server-$MYSQL_VERSION-$PERCONA_SERVER_VERSION"
145147
146# Build information148# Build information
147REVISION="$(cd "$SOURCEDIR"; grep '^revno: ' Docs/INFO_SRC |sed -e 's/revno: //')"149if test -e "$SOURCEDIR/Docs/INFO_SRC"
150then
151 REVISION="$(cd "$SOURCEDIR"; grep '^revno: ' Docs/INFO_SRC |sed -e 's/revno: //')"
152elif test -e "$SOURCEDIR/.bzr/branch/last-revision"
153then
154 REVISION="$(cd "$SOURCEDIR"; cat .bzr/branch/last-revision | awk -F ' ' '{print $1}')"
155else
156 REVISION=""
157fi
148PRODUCT_FULL="Percona-Server-$MYSQL_VERSION-$PERCONA_SERVER_VERSION"158PRODUCT_FULL="Percona-Server-$MYSQL_VERSION-$PERCONA_SERVER_VERSION"
149PRODUCT_FULL="$PRODUCT_FULL-$REVISION${BUILD_COMMENT:-}$TAG.$(uname -s).$TARGET"159PRODUCT_FULL="$PRODUCT_FULL-$REVISION${BUILD_COMMENT:-}$TAG.$(uname -s).$TARGET"
150COMMENT="Percona Server with XtraDB (GPL), Release $PERCONA_SERVER_VERSION"160COMMENT="Percona Server with XtraDB (GPL), Release $PERCONA_SERVER_VERSION"
151161
=== modified file 'configure.cmake'
--- configure.cmake 2014-03-03 17:51:33 +0000
+++ configure.cmake 2014-05-27 06:43:06 +0000
@@ -1097,6 +1097,7 @@
1097 CHECK_INCLUDE_FILES("${VALGRIND_HEADERS}" HAVE_VALGRIND_HEADERS)1097 CHECK_INCLUDE_FILES("${VALGRIND_HEADERS}" HAVE_VALGRIND_HEADERS)
1098 IF(HAVE_VALGRIND_HEADERS)1098 IF(HAVE_VALGRIND_HEADERS)
1099 SET(HAVE_VALGRIND 1)1099 SET(HAVE_VALGRIND 1)
1100 ADD_DEFINITIONS(-DHAVE_purify)
1100 ELSE()1101 ELSE()
1101 MESSAGE(FATAL_ERROR "Unable to find Valgrind header files ${VALGRIND_HEADERS}. Make sure you have them in your include path.")1102 MESSAGE(FATAL_ERROR "Unable to find Valgrind header files ${VALGRIND_HEADERS}. Make sure you have them in your include path.")
1102 ENDIF()1103 ENDIF()

Subscribers

People subscribed via source and target branches