<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#FFFFFF">
On 27/07/2011 21:24, Michael T. Dean wrote:
<blockquote cite="mid:4E3073EF.6000907@thirdcontact.com" type="cite">On
07/27/2011 03:09 PM, Brian J. Murrell wrote:
<br>
<blockquote type="cite">On 11-07-27 03:01 PM, Michael T. Dean
wrote:
<br>
<blockquote type="cite">A better test would be to apply
<br>
<a class="moz-txt-link-freetext" href="https://github.com/MythTV/mythtv/commit/dd2d7a14d">https://github.com/MythTV/mythtv/commit/dd2d7a14d</a> to your
MythTV (or, if
<br>
you're running master, just pull an update). If you're on
0.24-fixes,
<br>
it will be much easier to backport this change than it was to
revert the
<br>
commit I mentioned. Just make sure you're using Qt 4.6 or
higher if
<br>
you're on 0.24-fixes.
<br>
</blockquote>
Is all of this to indicate that this won't land on 0.24-fixes?
<br>
<br>
I've been following this thread with great interest since I
think it's
<br>
the cause of #9704 also, which has been plaguing me for much too
long.
<br>
</blockquote>
<br>
Since 0.24-fixes requires Qt 4.5+, we can't use this on it.<br>
</blockquote>
<br>
Can it not be wrapped in:<br>
<br>
#if (QT_VERSION >= QT_VERSION_CHECK(4, 6, 0))<br>
<br>
...<br>
<br>
#endif<br>
<br>
??<br>
<br>
<blockquote cite="mid:4E3073EF.6000907@thirdcontact.com" type="cite">
<br>
OK, actually, we kind of could, but the logs would be littered
with
<br>
<br>
QMYSQLDriver::open: Illegal connect option value
'MYSQL_OPT_RECONNECT=1'
<br>
<br>
or something like that for anyone using Qt4.5. In theory, it
won't cause any problems for them, but until we have some
reasonable proof that it's actually fixing the issue, we'll just
test the code in unstable. If we do find evidence that it
actually works to prevent these deadlocks, I'll see about adding
it to 0.24-fixes--so at least people using Qt 4.6+ will benefit
from it.
<br>
<br>
Input from those of you experiencing the deadlocks would make it
much easier. So, if you can, please try building your MythTV with
the patch and using it.
<br>
<br>
I've attached a version of the patch for 0.24-fixes.
<br>
<br>
Thanks,
<br>
Mike
<br>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
<pre wrap="">_______________________________________________
mythtv-users mailing list
<a class="moz-txt-link-abbreviated" href="mailto:mythtv-users@mythtv.org">mythtv-users@mythtv.org</a>
<a class="moz-txt-link-freetext" href="http://www.mythtv.org/mailman/listinfo/mythtv-users">http://www.mythtv.org/mailman/listinfo/mythtv-users</a>
</pre>
</blockquote>
<br>
</body>
</html>