Merge ~ahasenack/ubuntu/+source/apr-util:eoan-dbd-mysql-module-load into ubuntu/+source/apr-util:ubuntu/devel

Proposed by Andreas Hasenack
Status: Superseded
Proposed branch: ~ahasenack/ubuntu/+source/apr-util:eoan-dbd-mysql-module-load
Merge into: ubuntu/+source/apr-util:ubuntu/devel
Diff against target: 66 lines (+36/-0) (has conflicts)
3 files modified
debian/changelog (+9/-0)
debian/patches/mysql8-my_init.patch (+23/-0)
debian/patches/series (+4/-0)
Conflict in debian/changelog
Conflict in debian/patches/series
Reviewer Review Type Date Requested Status
Canonical Server Core Reviewers Pending
Canonical Server Pending
Review via email: mp+380247@code.launchpad.net

This proposal has been superseded by a proposal from 2020-03-04.

Description of the change

Fix loading the mysql dbd module.

Testing instructions are in the linked bug.

I pinged Lars in #ubuntu-devel and he agreed my_init() shouldn't be needed at all.

PPA for testing: https://launchpad.net/~ahasenack/+archive/ubuntu/mysql8-my-init/

Note that eoan's mysql8 still export my_bool.

To post a comment you must log in.

Unmerged commits

312ea47... by Andreas Hasenack

update-metadata

08ee360... by Andreas Hasenack

changelog

3801058... by Andreas Hasenack

  * d/p/mysql8-my_init.patch: don't call my_init() since it's not
    exported anymore since MySQL 8.0.2 (LP: #1859773)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/debian/changelog b/debian/changelog
2index 75eba76..137ec01 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,4 @@
6+<<<<<<< debian/changelog
7 apr-util (1.6.1-4ubuntu1) focal; urgency=medium
8
9 * Adjust Build-Depends to unambiguously require Python 2.
10@@ -5,6 +6,14 @@ apr-util (1.6.1-4ubuntu1) focal; urgency=medium
11 (LP: #1863026)
12
13 -- Robie Basak <robie.basak@ubuntu.com> Tue, 18 Feb 2020 14:53:49 +0000
14+=======
15+apr-util (1.6.1-4ubuntu0.1) eoan; urgency=medium
16+
17+ * d/p/mysql8-my_init.patch: don't call my_init() since it's not
18+ exported anymore since MySQL 8.0.2 (LP: #1859773)
19+
20+ -- Andreas Hasenack <andreas@canonical.com> Wed, 04 Mar 2020 17:00:31 -0300
21+>>>>>>> debian/changelog
22
23 apr-util (1.6.1-4build1) eoan; urgency=medium
24
25diff --git a/debian/patches/mysql8-my_init.patch b/debian/patches/mysql8-my_init.patch
26new file mode 100644
27index 0000000..3be2139
28--- /dev/null
29+++ b/debian/patches/mysql8-my_init.patch
30@@ -0,0 +1,23 @@
31+Description: don't call my_init()
32+ Since MySQL 8.0.2, my_init() isn't exported anymore and shouldn't have to be
33+ called by clients directly. See
34+ https://dev.mysql.com/doc/relnotes/mysql/8.0/en/news-8-0-2.html
35+Author: Andreas Hasenack <andreas@canonical.com>
36+Bug-Ubuntu: https://bugs.launchpad.net/ubuntu/+source/apache2/+bug/1859773
37+Forwarded: no
38+X-Forwarded-Reason: Should have an explicit check for MySQL 8.0.2
39+Last-Update: 2020-03-04
40+---
41+This patch header follows DEP-3: http://dep.debian.net/deps/dep3/
42+--- a/dbd/apr_dbd_mysql.c
43++++ b/dbd/apr_dbd_mysql.c
44+@@ -1263,9 +1263,6 @@
45+
46+ static void dbd_mysql_init(apr_pool_t *pool)
47+ {
48+-#if MYSQL_VERSION_ID < 100000
49+- my_init();
50+-#endif
51+ mysql_thread_init();
52+
53+ /* FIXME: this is a guess; find out what it really does */
54diff --git a/debian/patches/series b/debian/patches/series
55index 3e71f6a..aafb7a0 100644
56--- a/debian/patches/series
57+++ b/debian/patches/series
58@@ -11,4 +11,8 @@ fix_doxygen_inputdir
59 support_mariadb.patch
60 doxygen_no_ful_path_names.patch
61 fix_gdbm_error_handling
62+<<<<<<< debian/patches/series
63 mysql8_my_bool.patch
64+=======
65+mysql8-my_init.patch
66+>>>>>>> debian/patches/series

Subscribers

People subscribed via source and target branches