Merge lp:~tplavcic/percona-server/bug1290087-5.5 into lp:percona-server/5.5

Proposed by Tomislav Plavcic
Status: Merged
Merged at revision: 694
Proposed branch: lp:~tplavcic/percona-server/bug1290087-5.5
Merge into: lp:percona-server/5.5
Diff against target: 94 lines (+36/-0)
2 files modified
build-ps/debian/percona-server-server-5.5.files (+1/-0)
build-ps/debian/rules (+35/-0)
To merge this branch: bzr merge lp:~tplavcic/percona-server/bug1290087-5.5
Reviewer Review Type Date Requested Status
Tomislav Plavcic Approve
Review via email: mp+232223@code.launchpad.net

This proposal supersedes a proposal from 2014-08-25.

Description of the change

Build for mysqld-debug and plugin/debug has been added to debian packages.
Test was done by building packages and inspecting content to see if files are in place.
Note that in the jenkins links first job for 5.6 has failed on ubuntu-precise but that is because
of the older version of cmake on that host (JEN-256 has been opened for this).
If param job needs to be runned for this change please notify me.

5.5 packages
http://jenkins.percona.com/view/TEST/job/percona-server-5.5-debian-binary-test/

5.6 packages
http://jenkins.percona.com/view/TEST/job/percona-server-5.6-debian-binary-test/
http://jenkins.percona.com/view/TEST/job/percona-server-5.6-debian-binary-32-test/

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

Can you please add -DWITH_DEBUG=ON in addition to -DCMAKE_BUILD_TYPE=Debug? This bit is not a regression to our previous packaging, but avoids http://bugs.mysql.com/bug.php?id=70647

review: Needs Fixing
Revision history for this message
Tomislav Plavcic (tplavcic) wrote : Posted in a previous version of this proposal

I guess that depends on which version of TokuDB we will include in this release because of this:
https://groups.google.com/a/percona.com/forum/#!topic/percona-tokutek/si8zKhdptyo
So until we start building from tokudb-7.1.8-rc1 or higher I don't think we can include it.
I think Hrvoje is waiting on input from PeterS on which tag will be included.

Revision history for this message
Tomislav Plavcic (tplavcic) wrote : Posted in a previous version of this proposal

As discussed over IRC opened https://bugs.launchpad.net/percona-server/+bug/1361658
for -DWITH_DEBUG=ON as this MP will be merged without it.

Revision history for this message
Tomislav Plavcic (tplavcic) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'build-ps/debian/percona-server-server-5.5.files'
--- build-ps/debian/percona-server-server-5.5.files 2014-03-18 16:05:28 +0000
+++ build-ps/debian/percona-server-server-5.5.files 2014-08-26 13:59:45 +0000
@@ -23,6 +23,7 @@
23usr/bin/resolve_stack_dump23usr/bin/resolve_stack_dump
24usr/bin/resolveip24usr/bin/resolveip
25usr/sbin/mysqld25usr/sbin/mysqld
26usr/sbin/mysqld-debug
26usr/share/man/man1/msql2mysql.127usr/share/man/man1/msql2mysql.1
27usr/share/man/man1/myisamchk.128usr/share/man/man1/myisamchk.1
28usr/share/man/man1/myisamlog.129usr/share/man/man1/myisamlog.1
2930
=== modified file 'build-ps/debian/rules'
--- build-ps/debian/rules 2014-08-22 10:09:55 +0000
+++ build-ps/debian/rules 2014-08-26 13:59:45 +0000
@@ -7,8 +7,10 @@
7PS_VERSION_EXTRA = '@@PERCONA_VERSION_EXTRA@@'7PS_VERSION_EXTRA = '@@PERCONA_VERSION_EXTRA@@'
8REVISION = '@@REVISION@@'8REVISION = '@@REVISION@@'
9COMPILATION_COMMENT_RELEASE = "Percona Server (GPL), Release $(PS_VERSION_EXTRA), Revision $(REVISION)"9COMPILATION_COMMENT_RELEASE = "Percona Server (GPL), Release $(PS_VERSION_EXTRA), Revision $(REVISION)"
10COMPILATION_COMMENT_DEBUG = "Percona Server - Debug (GPL), Release $(PS_VERSION_EXTRA), Revision $(REVISION)"
1011
11TMP=$(CURDIR)/debian/tmp/12TMP=$(CURDIR)/debian/tmp/
13TMPD=$(CURDIR)/debian/tmp-debug/
12prefix=/usr14prefix=/usr
1315
14ARCH = $(shell dpkg-architecture -qDEB_BUILD_ARCH)16ARCH = $(shell dpkg-architecture -qDEB_BUILD_ARCH)
@@ -49,6 +51,7 @@
49endif51endif
5052
51builddir = builddir53builddir = builddir
54builddebug = debug
5255
53configure: patch configure-stamp56configure: patch configure-stamp
5457
@@ -60,6 +63,32 @@
60 if [ ! -d /proc/self ]; then echo "/proc IS NEEDED" 1>&2; exit 1; fi 63 if [ ! -d /proc/self ]; then echo "/proc IS NEEDED" 1>&2; exit 1; fi
61endif64endif
6265
66ifeq ($(SKIP_DEBUG_BINARY),)
67 ( test -d $(builddebug) || mkdir $(builddebug) ) && cd $(builddebug) && \
68 sh -c 'PATH=$${MYSQL_BUILD_PATH:-"/bin:/usr/bin"} \
69 CC=$${MYSQL_BUILD_CC:-gcc} \
70 CFLAGS=$${MYSQL_BUILD_CFLAGS:-"-O2 -g -fno-strict-aliasing"} \
71 CXX=$${MYSQL_BUILD_CXX:-g++} \
72 CXXFLAGS=$${MYSQL_BUILD_CXXFLAGS:-"-O3 -g -felide-constructors -fno-exceptions -fno-rtti -fno-strict-aliasing"} \
73 cmake -DCMAKE_INSTALL_PREFIX=/usr \
74 \
75 -DMYSQL_UNIX_ADDR=/var/run/mysqld/mysqld.sock \
76 -DCMAKE_BUILD_TYPE=Debug \
77 -DWITH_LIBWRAP=ON \
78 -DWITH_SSL=system \
79 -DCOMPILATION_COMMENT=$(COMPILATION_COMMENT_DEBUG) \
80 -DSYSTEM_TYPE="debian-linux-gnu" \
81 -DINSTALL_LAYOUT=RPM \
82 -DINSTALL_LIBDIR=lib/$(DEB_HOST_MULTIARCH) \
83 -DINSTALL_PLUGINDIR=lib/mysql/plugin/debug \
84 -DWITH_EMBEDDED_SERVER=OFF \
85 -DWITH_ARCHIVE_STORAGE_ENGINE=ON \
86 -DWITH_BLACKHOLE_STORAGE_ENGINE=ON \
87 -DWITH_FEDERATED_STORAGE_ENGINE=ON \
88 -DWITH_PAM=ON \
89 -DWITH_EXTRA_CHARSETS=all ..'
90endif
91
63 ( test -d $(builddir) || mkdir $(builddir) ) && cd $(builddir) && \92 ( test -d $(builddir) || mkdir $(builddir) ) && cd $(builddir) && \
64 sh -c 'PATH=$${MYSQL_BUILD_PATH:-"/bin:/usr/bin"} \93 sh -c 'PATH=$${MYSQL_BUILD_PATH:-"/bin:/usr/bin"} \
65 CC=$${MYSQL_BUILD_CC:-gcc} \94 CC=$${MYSQL_BUILD_CC:-gcc} \
@@ -98,6 +127,10 @@
98 --exclude=debian . \127 --exclude=debian . \
99 --transform="s,^\./,percona-server-5.5/,"128 --transform="s,^\./,percona-server-5.5/,"
100129
130ifeq ($(SKIP_DEBUG_BINARY),)
131 cd $(builddebug) && $(MAKE) $(MAKE_JFLAG)
132endif
133
101 cd $(builddir) && $(MAKE) $(MAKE_JFLAG) VERBOSE=1134 cd $(builddir) && $(MAKE) $(MAKE_JFLAG) VERBOSE=1
102135
103ifeq ($(findstring nocheck,$(DEB_BUILD_OPTIONS)),)136ifeq ($(findstring nocheck,$(DEB_BUILD_OPTIONS)),)
@@ -129,6 +162,7 @@
129 [ ! -d mysql-test/var ] || rm -rf mysql-test/var162 [ ! -d mysql-test/var ] || rm -rf mysql-test/var
130 163
131 rm -rf $(builddir)164 rm -rf $(builddir)
165 rm -rf $(builddebug)
132166
133 debconf-updatepo167 debconf-updatepo
134 rm -f $(EXPORTED_SOURCE_TARBALL)168 rm -f $(EXPORTED_SOURCE_TARBALL)
@@ -143,6 +177,7 @@
143 dh_installdirs177 dh_installdirs
144178
145 # make install (trailing slash needed for innobase)179 # make install (trailing slash needed for innobase)
180 (cd $(builddebug) && $(MAKE) install DESTDIR=$(TMPD)/)
146 (cd $(builddir) && $(MAKE) install DESTDIR=$(TMP)/)181 (cd $(builddir) && $(MAKE) install DESTDIR=$(TMP)/)
147 182
148 # build HandlerSocket183 # build HandlerSocket

Subscribers

People subscribed via source and target branches