Opened 8 years ago

Closed 7 years ago

Last modified 6 years ago

#13304 closed Bug Report - General (fixed)

FTBFS with MySQL 8.0

Reported by: ltangvald Owned by: David Hampton
Priority: minor Milestone: 31.0
Component: Plugin - MythZoneminder Version: Unspecified
Severity: medium Keywords:
Cc: Ticket locked: no

Description

Build fails because the my_bool type has been removed in MySQL, to be replaced with either int or bool (ref: https://dev.mysql.com/doc/refman/8.0/en/c-api-data-structures.html)

To reproduce: build mythtv source with libmysqlclient-dev 8.0+

Change History (8)

comment:1 by Gary Buhrmaster, 8 years ago

Hopefully useful information for the MythZoneminder wrangler....

Related to #13093 (where the use of the my_bool was added as that approach was the solution being done in other projects (now they get to change again too)).

It appears that mysql and mariadb are starting to diverge at the client API level.

comment:2 by ltangvald, 8 years ago

Yeah, they've been steadily diverging (generally we had more issues of this type for the transition from 5.6 to 5.7, but it varies what software is affected by each change). There's a reasonably simple way to support both by adding a precompile directive:

#if !defined(MARIADB_BASE_VERSION) && MYSQL_VERSION_ID >= 80000
typedef int reconnect_t
#else
typedef my_bool reconnect_t
#endif
reconnect_t reconnect = 1;
Last edited 8 years ago by ltangvald (previous) (diff)

comment:3 by David Hampton, 7 years ago

Milestone: needs_triage31.0
Owner: set to David Hampton
Status: newaccepted

comment:4 by Lars Tangvald <lars.tangvald@…>, 7 years ago

Resolution: fixed
Status: acceptedclosed

In e87829674/mythtv:

Error: Processor CommitTicketReference failed
GIT backend not available

comment:5 by Lars Tangvald <lars.tangvald@…>, 7 years ago

In f21b8e56b/mythtv:

Error: Processor CommitTicketReference failed
GIT backend not available

comment:6 by Lars Tangvald <lars.tangvald@…>, 7 years ago

In f21b8e56b/mythtv:

Error: Processor CommitTicketReference failed
GIT backend not available

comment:7 by Gary Buhrmaster, 7 years ago

Should this fix be backported to fixes/30 to support customers who choose to run a newer OS with MySQL 8.0 but continue with the released fixes branch?

comment:8 by Lars Tangvald <lars.tangvald@…>, 6 years ago

In 5a3e9d6a0/mythtv:

Error: Processor CommitTicketReference failed
GIT backend not available
Note: See TracTickets for help on using tickets.