[Buildroot] [PATCH 0/1] mariadb and mysql

[Buildroot] [PATCH 0/1] mariadb and mysql

[Christopher McCrory]

I ran into this again when testing perl-termreadkey.  I started with an
existing config that used perl-dbd-mysql. Thought I would send the
patch I generated a while ago. As an aside, used mariadb because the
oracle-mysql 5.1 version was EOL 5 ish years ago.

Hopefully my commit messages are getting better :)



Christopher McCrory (1):
  mariadb: fix mysql_config when cross compiling

 package/mariadb/mariadb.mk | 5 +++++
 1 file changed, 5 insertions(+)

-- 
2.14.5

[Buildroot] [PATCH 1/1] mariadb: fix mysql_config when cross compiling

[Christopher McCrory]

Mariadb_config in staging is not a shell script, this breaks
mysql_config when cross-compiling.  This in turn breaks package builds
needing the location of the include and library files. So remove it after
installation into $TARGET_DIR

Signed-off-by: Christopher McCrory <chrismcc@gmail.com>
---
 package/mariadb/mariadb.mk | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/package/mariadb/mariadb.mk b/package/mariadb/mariadb.mk
index 06d6365fab..52d47f98fc 100644
--- a/package/mariadb/mariadb.mk
+++ b/package/mariadb/mariadb.mk
@@ -125,10 +125,15 @@ endif
 
 # We don't need mysql_config on the target as it's only useful in staging
 # We also don't need the test suite on the target
+# Also mariadb_config in staging is not a shell script, this breaks
+# mysql_config when cross-compiling.  This in turn breaks package builds
+# needing the location of the include and library files. So remove it after
+# installation into $TARGET_DIR
 define MARIADB_POST_INSTALL
 	mkdir -p $(TARGET_DIR)/var/lib/mysql
 	$(RM) $(TARGET_DIR)/usr/bin/mysql_config
 	$(RM) -r $(TARGET_DIR)/usr/share/mysql/test
+	$(RM) $(STAGING_DIR)/usr/bin/mariadb_config
 endef
 
 MARIADB_POST_INSTALL_TARGET_HOOKS += MARIADB_POST_INSTALL
-- 
2.14.5

[Buildroot] [PATCH 1/1] mariadb: fix mysql_config when cross compiling

[Peter Korsgaard]

>>>>> "Christopher" == Christopher McCrory <chrismcc@gmail.com> writes:

 > Mariadb_config in staging is not a shell script, this breaks
 > mysql_config when cross-compiling.  This in turn breaks package builds
 > needing the location of the include and library files. So remove it after
 > installation into $TARGET_DIR

 > Signed-off-by: Christopher McCrory <chrismcc@gmail.com>
 > ---
 >  package/mariadb/mariadb.mk | 5 +++++
 >  1 file changed, 5 insertions(+)

 > diff --git a/package/mariadb/mariadb.mk b/package/mariadb/mariadb.mk
 > index 06d6365fab..52d47f98fc 100644
 > --- a/package/mariadb/mariadb.mk
 > +++ b/package/mariadb/mariadb.mk
 > @@ -125,10 +125,15 @@ endif
 
 >  # We don't need mysql_config on the target as it's only useful in staging
 >  # We also don't need the test suite on the target
 > +# Also mariadb_config in staging is not a shell script, this breaks
 > +# mysql_config when cross-compiling.  This in turn breaks package builds
 > +# needing the location of the include and library files. So remove it after
 > +# installation into $TARGET_DIR

Ehh, STAGING_DIR or TARGET_DIR?

What then happens when those packages try to run mariadb_config to get
the include/library locations? What if the build machine already has
mariadb_config in the path?

>  define MARIADB_POST_INSTALL
 >  	mkdir -p $(TARGET_DIR)/var/lib/mysql
 >  	$(RM) $(TARGET_DIR)/usr/bin/mysql_config
 >  	$(RM) -r $(TARGET_DIR)/usr/share/mysql/test
 > +	$(RM) $(STAGING_DIR)/usr/bin/mariadb_config

Fixups to STAGING_DIR should not be mixed up wiith the TARGET_DIR
installation logic, please add a POST_INSTALL_STAGING_HOOK instead.

>  endef
 
 >  MARIADB_POST_INSTALL_TARGET_HOOKS += MARIADB_POST_INSTALL
 > -- 
 > 2.14.5

 > _______________________________________________
 > buildroot mailing list
 > buildroot at busybox.net
 > http://lists.busybox.net/mailman/listinfo/buildroot

-- 
Bye, Peter Korsgaard

[Buildroot] [PATCH 0/1] mariadb and mysql

[Arnout Vandecappelle]

On 08/11/2018 18:09, Christopher McCrory wrote:
> I ran into this again when testing perl-termreadkey.  I started with an
> existing config that used perl-dbd-mysql. Thought I would send the
> patch I generated a while ago. As an aside, used mariadb because the
> oracle-mysql 5.1 version was EOL 5 ish years ago.

 Feel free to bump it :-)

 However, note that at the moment, there is a huge size difference between
oracle-mysql and mariadb: oracle-mysql server is just around 10MB, while mariadb
is over 100MB.


 Regards,
 Arnout

> 
> Hopefully my commit messages are getting better :)
> 
> 
> 
> Christopher McCrory (1):
>   mariadb: fix mysql_config when cross compiling
> 
>  package/mariadb/mariadb.mk | 5 +++++
>  1 file changed, 5 insertions(+)
>