Description of problem: # mysql_config --cflags -I/usr/include/mysql -I/usr/include/mysql/.. ..... Version-Release number of selected component (if applicable): mariadb-5.5.29-4 Additional info: /usr/include/mysql/.. is the same as /usr/include. Having -I/usr/include in CFLAGS is not a good idea.
The reason is that we move mysql_config due to multilib issues and then guessing basedir fails. I'd propose adding the following sed that just uses correct %{prefix} macro. diff --git a/mariadb.spec b/mariadb.spec index 0bf2f1b..81cc548 100644 --- a/mariadb.spec +++ b/mariadb.spec @@ -307,6 +307,10 @@ cat %{SOURCE15} >> mysql-test/rh-skipped-tests.list echo "main.gis-precise : rhbz#906367" >> mysql-test/rh-skipped-tests.list %endif +# don't guess basedir in mysql_config; we place it under _libdir because +# of multilib conflicts +sed -ie "s;basedir=\`echo $me.*$;basedir=%{_prefix};" scripts/mysql_config.sh + # install mysql_plugin cp %{SOURCE16} man/
Fix sounds reasonable, but I'd recommend doing it as a patch; sed is a bit too fragile.
I should have mentioned why sed was preferred before patch file -- it was because of using %_prefix macro rather than fixed /usr path. However, there is a better way how to use configured prefix value -- simply use @prefix@. It's already fixed this way in Rawhide.
And when can we expext the build to happen?
(In reply to comment #4) > And when can we expext the build to happen? I'm doing also some other changes there but I believe I'll submit a build today or tomorrow.