Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RMySQL fails build with mariadb >10.4 #214

Open
alecuba16 opened this issue Nov 3, 2017 · 12 comments
Open

RMySQL fails build with mariadb >10.4 #214

alecuba16 opened this issue Nov 3, 2017 · 12 comments

Comments

@alecuba16
Copy link

alecuba16 commented Nov 3, 2017

Because it looks for: MYSQL_SERVER_VERSION with libmariadb.so ???

Possible solution is given here: https://wiki.gentoo.org/wiki/MariaDB#MYSQL_SERVER_VERSION

#if defined MARIADB_CLIENT_VERSION_STR && !defined MYSQL_SERVER_VERSION
#define MYSQL_SERVER_VERSION MARIADB_CLIENT_VERSION_STR
#endif

@alecuba16
Copy link
Author

Fixed in this pull request:

#215

@lachlansimpson
Copy link

lachlansimpson commented Dec 19, 2017

Any chance this can be pushed? I'm still seeing this error in CentOS 7.4, R 3.4.2 and MariaDB 10.2.11.1

@alecuba16
Copy link
Author

alecuba16 commented Dec 19, 2017

@datakid Meanwhile you can use my fork with install GitHub from devtools.

@lachlansimpson
Copy link

Ah yeah. I might do. The RMariadb package installed - but the package I'm trying to install needs RMySQL and it's written poorly enough that I'd rather not fix it myself.

@alecuba16
Copy link
Author

alecuba16 commented Dec 19, 2017

@datakid My fork is from RMySQL master, to install it , you have to first install devtools from R package manager:

install.packages("devtools")

Then you will able to install RMySQL from my fork using the devtools (works on gentoo linux with latest mariadb beta too).

devtools::install_github("alecuba16/RMySQL")

PD: In windows you have to install RTOOLS before.

@lachlansimpson
Copy link

Perfect - thanks!

@mschubert
Copy link

This also affects mariadb=10.2.* (on Gentoo)

@vpbrendel
Copy link

I spent an hour or so trying to install RMySQL and re-discovered the patch suggested here. I can't see how this simple fix can break anything else, so I do hope this can be incorporated in the distribution and save a lot of others a bunch of time!

@krlmlr
Copy link
Member

krlmlr commented Mar 24, 2018

Does this now work with the CRAN version 0.10.14?

@mschubert
Copy link

@krlmlr works for me now

@dialvarezs
Copy link

@krlmlr not for me

utils.c: In function ‘rmysql_version’:
utils.c:269:40: error: ‘MYSQL_SERVER_VERSION’ undeclared (first use in this function); did you mean ‘MYSQL_PS_CONVERSION’?
   SET_STRING_ELT(output_nms, 0, mkChar(MYSQL_SERVER_VERSION));
                                        ^~~~~~~~~~~~~~~~~~~~
                                        MYSQL_PS_CONVERSION
utils.c:269:40: note: each undeclared identifier is reported only once for each function it appears in
make: *** [/usr/lib64/R/etc/Makeconf:159: utils.o] Error 1
ERROR: compilation failed for package ‘RMySQL’

OpenSUSE Tumbleweed, R 3.4.4, MariaDB 10.2.13

@odoublewen
Copy link

odoublewen commented Jan 10, 2019

I'm still seeing this error when building RMySQL (0.10.15) with MariaDB 10.2.8 to provide backend.

Edit: RMySQL 0.10.16 exhibits the same build failure.

utils.c: In function 'rmysql_version':
utils.c:269:40: error: 'MYSQL_SERVER_VERSION' undeclared (first use in this function); did you mean 'MYSQL_PS_CONVERSION'?
   SET_STRING_ELT(output_nms, 0, mkChar(MYSQL_SERVER_VERSION));
                                        ^~~~~~~~~~~~~~~~~~~~
                                        MYSQL_PS_CONVERSION

Seems like @jeroen implied that work done on Feb 25, 2018 (such as 6434882) was intended to fix this issue. That work should be in RMySQL (0.10.15).

Can't the patch suggested in #215 be merged?

CentOS Linux release 7.6.1810, R 3.5.1, MariaDB 10.2.8

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants