Bug 1470175 - mariadb library upgrade to 10.2 causes mydns FTBFS
Summary: mariadb library upgrade to 10.2 causes mydns FTBFS
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: mydns
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Itamar Reis Peixoto
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 1467297
TreeView+ depends on / blocked
 
Reported: 2017-07-12 13:37 UTC by Augusto Caringi
Modified: 2017-07-21 20:33 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-07-21 19:51:56 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
Patch to allow compilation of mydns with MariaDB 10.2 (352 bytes, patch)
2017-07-12 13:37 UTC, Augusto Caringi
no flags Details | Diff

Description Augusto Caringi 2017-07-12 13:37:07 UTC
Created attachment 1297014 [details]
Patch to allow compilation of mydns with MariaDB 10.2

Trying to build mydns with the new version of MariaDB 10.2 causes an error:

In file included from named.h:26:0,
                 from main.c:21:
main.c: In function 'cmdline':
../../lib/mydnsutil.h:180:36: error: expected ')' before 'MYSQL_SERVER_VERSION'
 # define  SQL_VERSION_STR "MySQL " MYSQL_SERVER_VERSION
                                    ^

In this new version of MariaDB they introduced a new mariadb_version.h header and mysql.h includes this header instead of mysql_version.h.
But the mysql_version.h is still present.

I'm attaching a patch proposal to fix the problem.

We are tracking all the possible problems regarding this MariaDB upgrade in the bug 1467297.

Copr build: https://copr.fedorainfracloud.org/coprs/g/db-sig/mariadb-10.2/package/mydns/

Comment 1 Adam Williamson 2017-07-21 19:51:56 UTC
Build done:

https://koji.fedoraproject.org/koji/taskinfo?taskID=20651297

Wasn't able to submit the patch upstream as registration for their bug tracker doesn't seem to be working (claims to be sending you a confirmation mail, never actually did).

Comment 2 Itamar Reis Peixoto 2017-07-21 20:27:33 UTC
thank you, I will be happy to accept your patch.

Comment 3 Adam Williamson 2017-07-21 20:33:48 UTC
Now managed to submit an issue upstream with a link to this bug and patch: http://bugs.mydns-ng.com/view.php?id=78


Note You need to log in before you can comment on or make changes to this bug.