'Review Request 110187: Don't communicate with mysql by env vars and autogenerated files'

[prev in list] [next in list] [prev in thread] [next in thread] 

List:       amarok-devel
Subject:    Review Request 110187: Don't communicate with mysql by env vars and autogenerated files
From:       "Patrick von Reth" <vonreth () kde ! org>
Date:       2013-04-25 14:36:29
Message-ID: 20130425143629.7022.44405 () vidsolbach ! de
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/110187/
-----------------------------------------------------------

Review request for Amarok.

Description
-------

Don't communicate with mysql by env vars and autogenerated files

Instead of generating the my.cnf every time amarok starts and to pass
the location of this file by setting an environment variable directly
pass the settings as arguments to mysql. As this is probably a better
approach and the only one working on windows.

This fixes an issue where amarok is writing the database to "C:\Program Files \
(x86)\Amarok\data\amarok". This issue prevents Amarok from running correctly, because \
regarding to the rights of the useraccount the directory can be not writeable.

The commands used are taken from the output of "mysqld --verbose --help". 

Diffs
-----

  src/core-impl/collections/db/sql/mysqlecollection/MySqlEmbeddedStorage.cpp \
0233498fdeb18ab51e709e9a78384fc37c47cb2a 

Diff: http://git.reviewboard.kde.org/r/110187/diff/

Testing
-------

Only on windows

Thanks,

Patrick von Reth

[Attachment #5 (text/html)]

<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/110187/">http://git.reviewboard.kde.org/r/110187/</a>
  </td>
    </tr>
   </table>
   <br />

<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" \
style="background-image: \
url('http://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); \
background-position: left top; background-repeat: repeat-x; border: 1px black \
solid;">  <tr>
  <td>

<div>Review request for Amarok.</div>
<div>By Patrick von Reth.</div>

<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;">Don&#39;t communicate with mysql by env vars and autogenerated files

Instead of generating the my.cnf every time amarok starts and to pass
the location of this file by setting an environment variable directly
pass the settings as arguments to mysql. As this is probably a better
approach and the only one working on windows.

This fixes an issue where amarok is writing the database to &quot;C:\Program Files \
(x86)\Amarok\data\amarok&quot;. This issue prevents Amarok from running correctly, \
because regarding to the rights of the useraccount the directory can be not \
writeable.

The commands used are taken from the output of &quot;mysqld --verbose --help&quot;. \
</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;">Only on windows</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">(0233498fdeb18ab51e709e9a78384fc37c47cb2a)</span></li>

</ul>

<p><a href="http://git.reviewboard.kde.org/r/110187/diff/" style="margin-left: \
3em;">View Diff</a></p>

  </td>
 </tr>
</table>

  </div>
 </body>
</html>

_______________________________________________
Amarok-devel mailing list
Amarok-devel@kde.org
https://mail.kde.org/mailman/listinfo/amarok-devel

[prev in list] [next in list] [prev in thread] [next in thread] 