[MDEV-6869] Error when compiling for Mac OS X 10.9 Created: 2014-10-14 Updated: 2018-01-02 Resolved: 2018-01-02 |
|
| Status: | Closed |
| Project: | MariaDB Server |
| Component/s: | Compiling, Storage Engine - TokuDB |
| Affects Version/s: | 10.0.14 |
| Fix Version/s: | 10.0.34, 10.2.12, 10.1.31 |
| Type: | Bug | Priority: | Minor |
| Reporter: | Jorge F. Hernandez | Assignee: | Daniel Black |
| Resolution: | Fixed | Votes: | 1 |
| Labels: | upstream | ||
| Environment: |
Mac OS X 10.9 |
||
| Issue Links: |
|
||||||||
| Description |
|
Hey guys, I'm getting this error:
When I try to compile Maria DB using instructions from https://mariadb.com/kb/en/mariadb/documentation/getting-started/compiling-mariadb-from-source/generic-build-instructions/ Any idea why? Thanks. |
| Comments |
| Comment by Elena Stepanova [ 2014-10-14 ] | |||||||||||||||||||||||||||||||
|
Were there any errors before? Can you provide the complete build output, or at least a considerable last part of it? | |||||||||||||||||||||||||||||||
| Comment by Jorge F. Hernandez [ 2014-10-14 ] | |||||||||||||||||||||||||||||||
|
Is this good enough or do yo need more:
| |||||||||||||||||||||||||||||||
| Comment by Elena Stepanova [ 2014-10-14 ] | |||||||||||||||||||||||||||||||
|
I think it should be enough, now we can see what the error was:
| |||||||||||||||||||||||||||||||
| Comment by Olivier Bertrand [ 2014-10-14 ] | |||||||||||||||||||||||||||||||
|
This was already fixed for | |||||||||||||||||||||||||||||||
| Comment by Jorge F. Hernandez [ 2014-10-14 ] | |||||||||||||||||||||||||||||||
|
is there a patch file to correct this or do I have to wait until 10.0.15 is released? | |||||||||||||||||||||||||||||||
| Comment by Olivier Bertrand [ 2014-10-14 ] | |||||||||||||||||||||||||||||||
|
No patch file but you can manually make the change in filamap.h; replace line 107 by:
| |||||||||||||||||||||||||||||||
| Comment by Jorge F. Hernandez [ 2014-10-14 ] | |||||||||||||||||||||||||||||||
|
I think there is still something wrong, I replaced line 107: virtual int GetNextPos(void) {return (int)Fpos + Nrec;}With the one you sent me and now I'm getting this: In file included from /Users/jorge/Downloads/mariadb-10.0.14/storage/connect/array.cpp:39: | |||||||||||||||||||||||||||||||
| Comment by Olivier Bertrand [ 2014-10-15 ] | |||||||||||||||||||||||||||||||
|
Indeed I forgot this one, sorry about that. It is a little more complicated to fix:
In filamtxt.cpp in the function AddListValue (line 280) the case for TYPE_VOID becomes:
Finally to remove your error, in array.cpp (132) the case becomes:
I hope this will do the job. Of course all this will be done in the new version. | |||||||||||||||||||||||||||||||
| Comment by Jorge F. Hernandez [ 2014-10-15 ] | |||||||||||||||||||||||||||||||
|
Almost there, that fixed that problem at 4X%, but now at 70% I'm getting this:
Thanks | |||||||||||||||||||||||||||||||
| Comment by Elena Stepanova [ 2014-10-15 ] | |||||||||||||||||||||||||||||||
|
Are you actually planning to use TokuDB? Maybe it's simpler to disable it entirely if you aren't? | |||||||||||||||||||||||||||||||
| Comment by Jorge F. Hernandez [ 2014-10-15 ] | |||||||||||||||||||||||||||||||
|
I'm not, but since I'm using the instructions from https://mariadb.com/kb/en/mariadb/documentation/getting-started/compiling-mariadb-from-source/generic-build-instructions/ it enables it by default, even though the first thing it says is that TokuDB is not compatible with the build platform. How can I disable it? I'm not familiar with cmake and I don't see a configure file that I could use to configure the build. Thanks | |||||||||||||||||||||||||||||||
| Comment by Jorge F. Hernandez [ 2014-10-15 ] | |||||||||||||||||||||||||||||||
|
Never mind, I found it in the same page, I'm trying now using configure with "--disable-tokudb" to see if it works. | |||||||||||||||||||||||||||||||
| Comment by Elena Stepanova [ 2014-10-15 ] | |||||||||||||||||||||||||||||||
|
It's a 5.3 approach, if it doesn't work, please try to add -DWITH_TOKUDB_STORAGE_ENGINE=OFF -DWITHOUT_TOKUDB_STORAGE_ENGINE=ON to whatever else cmake options you use. | |||||||||||||||||||||||||||||||
| Comment by Jorge F. Hernandez [ 2014-10-15 ] | |||||||||||||||||||||||||||||||
|
You are correct, it did not work, Thanks for the new args for the build, I'm trying them now. | |||||||||||||||||||||||||||||||
| Comment by Jorge F. Hernandez [ 2014-10-15 ] | |||||||||||||||||||||||||||||||
|
That worked, thanks a million. | |||||||||||||||||||||||||||||||
| Comment by Elena Stepanova [ 2014-10-15 ] | |||||||||||||||||||||||||||||||
|
You're welcome. I suppose OS X is still not supported (it wasn't some time ago), but I'd expect TokuDB to be automatically disabled then. Upd: According to Rich Prohaska, OS X is not supported indeed. | |||||||||||||||||||||||||||||||
| Comment by Daniel Black [ 2017-12-30 ] | |||||||||||||||||||||||||||||||
|
looking at travis in 10.2, both tokudb and connect storage engines are being compiled ok. Close? |