<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/100345/">http://git.reviewboard.kde.org/r/100345/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On January 11th, 2011, 5 a.m., <b>Ian Monroe</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">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.</pre>
</blockquote>
<p>On January 11th, 2011, 5:07 a.m., <b>Ian Monroe</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">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.</pre>
</blockquote>
<p>On January 11th, 2011, 5:07 p.m., <b>Jeff Mitchell</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">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.
</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">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).</pre>
<br />
<p>- Ian</p>
<br />
<p>On January 10th, 2011, 11:44 a.m., Patrick Spendrin wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.orgrb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Amarok.</div>
<div>By Patrick Spendrin.</div>
<p style="color: grey;"><i>Updated Jan. 10, 2011, 11:44 a.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">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.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">on Microsoft Visual Studio 2010 EE (msvc2010) it works as expected.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>src/core-impl/collections/db/sql/mysqlecollection/MySqlEmbeddedStorage.cpp <span style="color: grey">(4e8088d)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/100345/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>