Merge lp://staging/~stewart/percona-server/pkg-5.6 into lp://staging/percona-server/5.6

Proposed by Stewart Smith
Status: Merged
Approved by: Laurynas Biveinis
Approved revision: no longer in the source branch.
Merged at revision: 531
Proposed branch: lp://staging/~stewart/percona-server/pkg-5.6
Merge into: lp://staging/percona-server/5.6
Diff against target: 9433 lines (+7292/-745)
94 files modified
.bzrignore (+6/-6)
CMakeLists.txt (+1/-1)
Makefile-ps (+4/-20)
README (+11/-0)
UDF/Makefile.am (+2/-0)
UDF/configure.ac (+2/-3)
VERSION (+1/-1)
build-ps/build-binary.sh (+8/-9)
build-ps/build-rpm.sh (+2/-2)
build-ps/debian/control (+30/-34)
build-ps/debian/copyright (+787/-86)
build-ps/debian/libperconaserverclient18.1.install (+1/-1)
build-ps/debian/libperconaserverclient18.1.postinst (+2/-0)
build-ps/debian/libperconaserverclient18.lintian-overrides (+6/-0)
build-ps/debian/percona-server-client-5.6.files (+0/-1)
build-ps/debian/percona-server-client-5.6.lintian-overrides (+18/-0)
build-ps/debian/percona-server-server-5.6.docs (+2/-3)
build-ps/debian/percona-server-server-5.6.files (+3/-4)
build-ps/debian/percona-server-server-5.6.lintian-overrides (+10/-0)
build-ps/debian/percona-server-server-5.6.postinst (+19/-15)
build-ps/debian/percona-server-server-5.6.postrm (+1/-1)
build-ps/debian/percona-server-server-5.6.preinst (+12/-17)
build-ps/debian/percona-server-source-5.6.files (+1/-0)
build-ps/debian/percona-server-test-5.6.files (+0/-1)
build-ps/debian/po/POTFILES.in (+1/-0)
build-ps/debian/po/ar.po (+252/-0)
build-ps/debian/po/ca.po (+227/-0)
build-ps/debian/po/cs.po (+346/-0)
build-ps/debian/po/da.po (+382/-0)
build-ps/debian/po/de.po (+239/-0)
build-ps/debian/po/es.po (+390/-0)
build-ps/debian/po/eu.po (+236/-0)
build-ps/debian/po/fr.po (+251/-0)
build-ps/debian/po/gl.po (+249/-0)
build-ps/debian/po/it.po (+223/-0)
build-ps/debian/po/ja.po (+229/-0)
build-ps/debian/po/nb.po (+297/-0)
build-ps/debian/po/nl.po (+290/-0)
build-ps/debian/po/pt.po (+307/-0)
build-ps/debian/po/pt_BR.po (+443/-0)
build-ps/debian/po/ro.po (+319/-0)
build-ps/debian/po/ru.po (+229/-0)
build-ps/debian/po/sk.po (+219/-0)
build-ps/debian/po/sv.po (+229/-0)
build-ps/debian/po/templates.pot (+187/-0)
build-ps/debian/po/tr.po (+342/-0)
build-ps/debian/rules (+290/-75)
build-ps/debian/source/format (+1/-0)
build-ps/percona-server.spec (+18/-57)
build/build-dpkg.sh (+0/-138)
build/debian/libmysqlclient-dev.README.Maintainer (+0/-4)
build/debian/libmysqlclient-dev.install (+0/-6)
build/debian/libmysqlclient-dev.links (+0/-2)
build/debian/libmysqlclient18.1.links (+0/-1)
build/debian/percona-server-client-5.6.menu (+0/-3)
build/debian/source/format (+0/-1)
build/rpm/mysql-dubious-exports.patch (+0/-103)
client/CMakeLists.txt (+12/-12)
cmake/info_macros.cmake.in (+1/-1)
cmake/libutils.cmake (+3/-3)
cmake/make_dist.cmake.in (+10/-0)
cmake/mysql_version.cmake (+4/-4)
doc/source/installation.rst (+78/-70)
doc/source/management/pam_plugin.rst (+1/-1)
doc/source/upgrading_guide_55_56.rst (+1/-1)
libmysql/CMakeLists.txt (+14/-14)
libmysql/libmysql.map (+4/-4)
libmysql/libmysql.ver.in (+1/-1)
man/mysql_config.1 (+3/-3)
mysql-test/CMakeLists.txt (+0/-1)
packaging/rpm-uln/mysql.spec.sh (+5/-5)
plugin/percona-pam-for-mysql/CMakeLists.txt (+1/-1)
scripts/CMakeLists.txt (+2/-2)
scripts/mysql_config.pl.in (+3/-3)
scripts/mysql_config.sh (+2/-2)
sql/password.c (+1/-1)
storage/ndb/ndbapi-examples/mgmapi_logevent/Makefile (+1/-1)
storage/ndb/ndbapi-examples/mgmapi_logevent2/Makefile (+1/-1)
storage/ndb/ndbapi-examples/ndbapi_async/Makefile (+1/-1)
storage/ndb/ndbapi-examples/ndbapi_async1/Makefile (+1/-1)
storage/ndb/ndbapi-examples/ndbapi_blob/Makefile (+1/-1)
storage/ndb/ndbapi-examples/ndbapi_blob_ndbrecord/Makefile (+1/-1)
storage/ndb/ndbapi-examples/ndbapi_event/Makefile (+1/-1)
storage/ndb/ndbapi-examples/ndbapi_recattr_vs_record/Makefile (+1/-1)
storage/ndb/ndbapi-examples/ndbapi_retries/Makefile (+1/-1)
storage/ndb/ndbapi-examples/ndbapi_s_i_ndbrecord/Makefile (+1/-1)
storage/ndb/ndbapi-examples/ndbapi_scan/Makefile (+1/-1)
storage/ndb/ndbapi-examples/ndbapi_simple/Makefile (+1/-1)
storage/ndb/ndbapi-examples/ndbapi_simple_dual/Makefile (+1/-1)
storage/ndb/ndbapi-examples/ndbapi_simple_index/Makefile (+1/-1)
storage/ndb/test/run-test/CMakeLists.txt (+1/-1)
support-files/MySQL-shared-compat.spec.sh (+1/-1)
support-files/mysql.spec.sh (+3/-3)
tests/CMakeLists.txt (+2/-2)
To merge this branch: bzr merge lp://staging/~stewart/percona-server/pkg-5.6
Reviewer Review Type Date Requested Status
Laurynas Biveinis (community) Needs Fixing
Alexey Bychko (community) Approve
Review via email: mp+199744@code.staging.launchpad.net

This proposal supersedes a proposal from 2013-10-11.

Description of the change

This is near identical to the 5.5 merge: https://code.launchpad.net/~stewart/percona-server/pkg-5.5/+merge/199742

We don't have HandlerSocket to worry about in 5.6, so that's one less thing than 5.5.

'make dist' and build-binary.sh both work and will produce a binary tarball from a source tarball.

I've also tested deb and RPMs... although I have sometimes seen some strange things from mock and buliding RPMs on Fedora 19 that may need some attention.

To post a comment you must log in.
Revision history for this message
Alexey Bychko (abychko) wrote :

`--> grep 5.5 build-ps/debian/rules
# Also #659565
        mkdir -p $(TMP)/usr/share/doc/percona-server-5.5/examples
        mv $(TMP)/usr/share/mysql/*cnf $(TMP)/usr/share/doc/percona-server-5.5/examples/
        nm -n $(builddir)/sql/mysqld |gzip -9 > $(TMP)/usr/share/doc/percona-server-5.5/mysqld.sym.gz
        # FIXME install -D -m 644 debian/mysql-server-5.5.py $(TMP)/usr/share/apport/package-hooks/source_mysql-5.5.py
          dh_apparmor -pmysql-server-5.5 --profile-name=usr.sbin.mysqld; \
        dh_makeshlibs -plibmysqlclient18 -V'libmysqlclient18 (>= 5.5.13-1)'
        dh_strip --dbg-package=percona-server-5.5-dbg

it should be 5.6, no?

Revision history for this message
Alexey Bychko (abychko) wrote :

`--> grep -rn 5.5 build-ps | wc -l
676

Revision history for this message
Alexey Bychko (abychko) wrote :

Makefile-ps contains GXX=gcc line, which caused linker error

Revision history for this message
Alexey Bychko (abychko) wrote :

for debian build i's unable to use build-dpkg.sh script, please point it to Makefile-ps instead of Makefile

Revision history for this message
Alexey Bychko (abychko) wrote :

cmake options from Makefile-ps are used for building dpkg packages, so options from debian/rules file are completely ignored. for example, I saw an error, related to embedded library

Revision history for this message
Stewart Smith (stewart) wrote :

Alexey Bychko <email address hidden> writes:
> `--> grep 5.5 build-ps/debian/rules
> # Also #659565
> mkdir -p $(TMP)/usr/share/doc/percona-server-5.5/examples
> mv $(TMP)/usr/share/mysql/*cnf $(TMP)/usr/share/doc/percona-server-5.5/examples/
> nm -n $(builddir)/sql/mysqld |gzip -9 > $(TMP)/usr/share/doc/percona-server-5.5/mysqld.sym.gz
> # FIXME install -D -m 644 debian/mysql-server-5.5.py $(TMP)/usr/share/apport/package-hooks/source_mysql-5.5.py
> dh_apparmor -pmysql-server-5.5 --profile-name=usr.sbin.mysqld; \
> dh_makeshlibs -plibmysqlclient18 -V'libmysqlclient18 (>= 5.5.13-1)'
> dh_strip --dbg-package=percona-server-5.5-dbg
>
>
> it should be 5.6, no?

Quite likely, yes. I'll need to fix that.

--
Stewart Smith

Revision history for this message
Stewart Smith (stewart) wrote :

Alexey Bychko <email address hidden> writes:
> Makefile-ps contains GXX=gcc line, which caused linker error

I'm pretty sure Makefile-ps should be retired... I'm wondering what it
brings to the table that the build-ps/*sh scripts and the SPEC and
debian packaging don't.

--
Stewart Smith

Revision history for this message
Stewart Smith (stewart) wrote :

Alexey Bychko <email address hidden> writes:
> for debian build i's unable to use build-dpkg.sh script, please point
> it to Makefile-ps instead of Makefile

We should probably just remove the build-dpkg.sh script, as with these
branches we're just left with the normal debian packaging process:
- extract source tarball,
- cp -r build-ps/debian .
- dpkg-buildpackage -S -k12345
- sbuild (or dput)

--
Stewart Smith

Revision history for this message
Stewart Smith (stewart) wrote :

Alexey Bychko <email address hidden> writes:
> cmake options from Makefile-ps are used for building dpkg packages, so
> options from debian/rules file are completely ignored. for example, I
> saw an error, related to embedded library

Hrm... it really shouldn't be... I thought I had all of that taken away,
or is this from the build-dpkg.sh run?

--
Stewart Smith

Revision history for this message
Stewart Smith (stewart) wrote :

Alexey Bychko <email address hidden> writes:
> `--> grep -rn 5.5 build-ps | wc -l
> 676

Now fixed (except for changelog and similar)

--
Stewart Smith

Revision history for this message
Alexey Bychko (abychko) wrote :

> Alexey Bychko <email address hidden> writes:
> > cmake options from Makefile-ps are used for building dpkg packages, so
> > options from debian/rules file are completely ignored. for example, I
> > saw an error, related to embedded library
>
> Hrm... it really shouldn't be... I thought I had all of that taken away,
> or is this from the build-dpkg.sh run?
>
> --
> Stewart Smith
build-dpkg.sh comtains something like "make clean all" for preparing compilation

Revision history for this message
Alexey Bychko (abychko) wrote :

In file included from murmur_udf.cc:43:
/root/rpmbuild/BUILD/percona-server-5.6.15-62.0/include/my_global.h:68:23: error: my_config.h: No such file or directory
In file included from fnv1a_udf.cc:111:
/root/rpmbuild/BUILD/percona-server-5.6.15-62.0/include/my_global.h:68:23: error: my_config.h: No such file or directory
In file included from fnv_udf.cc:116:
/root/rpmbuild/BUILD/percona-server-5.6.15-62.0/include/my_global.h:68:23: error: my_config.h: No such file or directory
/root/rpmbuild/BUILD/percona-server-5.6.15-62.0/include/my_global.h:719:2: error: #error "please add -DSTACK_DIRECTION=1 or -1 to your CPPFLAGS"
/root/rpmbuild/BUILD/percona-server-5.6.15-62.0/include/my_global.h:789:1: warning: "isnan" redefined
In file included from /root/rpmbuild/BUILD/percona-server-5.6.15-62.0/include/my_global.h:319,

Revision history for this message
Roel Van de Paar (roel11) wrote :

Stewart, any updates here?

Revision history for this message
Laurynas Biveinis (laurynas-biveinis) wrote :

Same comments as for 5.5 +

- Why is percona_innodb_version test adjusted in an exact opposite of
  the 5.5 change, i.e. one "-x.x" removed from "5.6.x-x.x-x.x"?

review: Needs Fixing
Revision history for this message
Alexey Bychko (abychko) wrote :

debs are ok, so approved for merging

review: Approve
Revision history for this message
Laurynas Biveinis (laurynas-biveinis) wrote :

Consistent file_contents failure, e.g. http://jenkins.percona.com/job/percona-server-5.6-param/481/BUILD_TYPE=debug,Host=ubuntu-saucy-64bit/testReport/junit/%28root%29/main/file_contents/:

_StringException: Text attachment: traceback
------------
Comment:

Logfile:
CURRENT_TEST: main.file_contents
--- /mnt/workspace/percona-server-5.6-param/BUILD_TYPE/debug/Host/ubuntu-saucy-64bit/mysql-test/r/file_contents.result 2013-12-27 19:29:58.000000000 +0300
+++ /mnt/workspace/percona-server-5.6-param/BUILD_TYPE/debug/Host/ubuntu-saucy-64bit/build/mysql-test/var/3/log/file_contents.reject 2014-01-13 17:30:16.000000000 +0300
@@ -1,6 +1,6 @@

 Checking 'INFO_SRC' and 'INFO_BIN'
-INFO_SRC: Found MySQL version number / Found BZR revision id
+INFO_SRC: Found MySQL version number / No line 'revision-id: .....'
 INFO_BIN: Found 'Compiler ... used' line / Found 'Feature flags' line

 End of tests

mysqltest: Result length mismatch
------------

review: Needs Fixing
Revision history for this message
Stewart Smith (stewart) wrote :

Laurynas Biveinis <email address hidden> writes:
> CURRENT_TEST: main.file_contents

fixed for 5.5 and 5.6, needed to revert our workaround.

--
Stewart Smith

Revision history for this message
Laurynas Biveinis (laurynas-biveinis) wrote :

The top-level README refers to 5.5.

review: Needs Fixing

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches