> On Jan. 11, 2011, 5 a.m., Ian Monroe wrote: > > The current way we do it is because if you have a MySQL that randomly > > doesn't support whatever argument you are trying to use, Bad Things Happen. > > If you look at the history of this code it is quite turbulent. > > > > Surely there is a place that is writable that we could use instead? > > > > Granted your solution only uses one argument so its probably fine. But in > > general it would be better to find a solution that doesn't alter the logic > > here. > > Ian Monroe wrote: > Well rereading your description I see this is because it "doesn't > reliably" find the config file. I guess that sounds like the mysql embedded > that I know. > > Still seems like a step backward to use the arguments again though... the > whole reason we use a config file in the first place is to avoid using > arguments. > > Jeff Mitchell wrote: > Indeed -- the reason we went to environment variables in the first place > was because --defaults-file didn't always work. I discussed this problem with > Monty at OSCON and worked with the Maria/MySQL guys and they couldn't figure > out a good solution, so they pointed me to the environment variable route. > > I know Windows supports environment variables, is there a way to make the > functionality work while keeping this method? > > Otherwise, maybe we can use defines to use one method or the other. > > > Ian Monroe wrote: > It looks like the environmental variable thing isn't working on mysql > 5.5.9. > > Is 5.5.x perhaps the version you were using Patrick? It might just be > that windows people noticed it first if they upgraded mysql first... (linux > distros are really conservative when it comes to upgrading mysql).
well nevermind, I tried a variation of your fix and I still have a problem with linux + 5.5.9. - Ian ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/100345/#review829 ----------------------------------------------------------- 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
