----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/100345/#review3523 -----------------------------------------------------------
A summary of this discussion. Please check the correct answer. A: this patch is still fine and should be shipped B: this patch is working around a known bug and should be shipped C: this patch is working around a known bug, which is kind of useless and should be discarded D: I will rework this patch and update the diff so that it can be submitted - Ralf On Jan. 10, 2011, 11:44 a.m., Patrick Spendrin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/100345/ > ----------------------------------------------------------- > > (Updated Jan. 10, 2011, 11:44 a.m.) > > > Review request for Amarok. > > > Summary > ------- > > The mysql embedded library doesn't reliably find the config file on Windows, > instead it tries to write into the write-protected install location. This > patch uses the mysql API instead of setenv and should work on all platforms. > It shouldn't change the current behaviour, when you test please still backup > your .kde/share/apps/amarok/mysqle database directory before trying. > > > Diffs > ----- > > src/core-impl/collections/db/sql/mysqlecollection/MySqlEmbeddedStorage.cpp > 4e8088d > > Diff: http://git.reviewboard.kde.org/r/100345/diff > > > Testing > ------- > > on Microsoft Visual Studio 2010 EE (msvc2010) it works as expected. > > > Thanks, > > Patrick > >
_______________________________________________ Amarok-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/amarok-devel
