Review Request 127523: Only link with MYSQL_EMBEDDED_LIBRARIES if WITH_MYSQL_EMBEDDED

[Matt Whitlock]

> On March 29, 2016, 6:48 p.m., Myriam Schweingruber wrote:
> > hm, I fail to see why you would not build amarok with MySQL embedded, as that is the default database. Could you please specify why this patch is needed?

Gentoo's philosophy is all about lean-and-mean, compiling support only for those options that one actually uses. There is not "one size fits all," and all users build all software from source, using the configure flags best suited to their needs. The Gentoo ebuild for Amarok has a USE flag "embedded" that enables or disables support for MySQL Embedded using `-DWITH_MYSQL_EMBEDDED` on the CMake command line. I have been running an Amarok built without MySQL Embedded for many years, as I run a full MySQL daemon on my system at all times anyway. The latest Amarok beta no longer builds successfully when MySQL Embedded is disabled. This patch is straightforward and logical and seems to be the obvious way to correct the build problem. (Indeed, I am listening to music in Amarok 2.8.90 right now that has been compiled without MySQL Embedded support by applying this patch to the build system.)


- Matt


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127523/#review94120
-----------------------------------------------------------


On March 29, 2016, 12:27 p.m., Matt Whitlock wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127523/
> -----------------------------------------------------------
> 
> (Updated March 29, 2016, 12:27 p.m.)
> 
> 
> Review request for Amarok.
> 
> 
> Repository: amarok
> 
> 
> Description
> -------
> 
> This patch corrects a build failure when compiling Amarok without MySQL Embedded.
> 
> 
> Diffs
> -----
> 
>   src/core-impl/collections/db/sql/mysqlcollection/CMakeLists.txt 244cde1 
> 
> Diff: https://git.reviewboard.kde.org/r/127523/diff/
> 
> 
> Testing
> -------
> 
> See [Gentoo bug 566980](https://bugs.gentoo.org/show_bug.cgi?id=566980).
> 
> 
> Thanks,
> 
> Matt Whitlock
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/amarok-devel/attachments/20160330/c6d34f61/attachment.html>