<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
</head>
<body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; color: rgb(0, 0, 0); font-size: 14px; font-family: Calibri, sans-serif; ">
<div>
<div>
<div><br>
</div>
<div>Yes, for any change to be in a release, it has to go through the CI system into the 4.8 branch.</div>
<div><br>
</div>
<div>As has been discussed before, we are making the 4.8 patch release directly from 4.8 branch. So we do not cherry pick anything, getting a new fix in means we take all that are in by that point in time.</div>
<div><br>
</div>
<div>So for this particular item first thing is to get it into the 4.8 branch, after which we can evaluate if we should switch to different sha1 for the release being created (otherwise it will be in the next patch release).</div>
<div><br>
</div>
<div>Yours,</div>
<div>
<div><br>
</div>
<div>
<p class="MsoNormal" style="margin-top: 0cm; margin-right: 0cm; margin-left: 0cm; margin-bottom: 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif; ">
--<o:p></o:p></p>
<p class="MsoNormal" style="margin-top: 0cm; margin-right: 0cm; margin-left: 0cm; margin-bottom: 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif; ">
Tuukka Turunen<o:p></o:p></p>
<p class="MsoNormal" style="margin-top: 0cm; margin-right: 0cm; margin-left: 0cm; margin-bottom: 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif; ">
Director, Qt Commercial R&D<o:p></o:p></p>
<p class="MsoNormal" style="margin-top: 0cm; margin-right: 0cm; margin-left: 0cm; margin-bottom: 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif; ">
Digia Plc<o:p></o:p></p>
<p class="MsoNormal" style="margin-top: 0cm; margin-right: 0cm; margin-left: 0cm; margin-bottom: 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif; ">
Piippukatu 11, 40100 Jyväskylä, Finland<o:p></o:p></p>
<p class="MsoNormal" style="margin-top: 0cm; margin-right: 0cm; margin-left: 0cm; margin-bottom: 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif; ">
<span style="font-size: 10pt; font-family: Verdana, sans-serif; "><o:p> </o:p></span></p>
<p class="MsoNormal" style="margin-top: 0cm; margin-right: 0cm; margin-left: 0cm; margin-bottom: 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif; ">
<span style="font-size: 10pt; ">Visit us at: <a href="applewebdata://7FB7E1CD-55A7-4FAD-B1A5-7721230C2E29/www.digia.com" style="color: blue; text-decoration: underline; "><span style="color: blue; ">www.digia.com</span></a> or <a href="http://qt.digia.com/" style="color: blue; text-decoration: underline; "><span style="color: blue; ">qt.digia.com</span></a><o:p></o:p></span></p>
<p class="MsoNormal" style="margin-top: 0cm; margin-right: 0cm; margin-left: 0cm; margin-bottom: 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif; ">
<o:p> <br>
</o:p></p>
</div>
</div>
</div>
</div>
<div><br>
</div>
<span id="OLK_SRC_BODY_SECTION">
<div style="font-family:Calibri; font-size:11pt; text-align:left; color:black; BORDER-BOTTOM: medium none; BORDER-LEFT: medium none; PADDING-BOTTOM: 0in; PADDING-LEFT: 0in; PADDING-RIGHT: 0in; BORDER-TOP: #b5c4df 1pt solid; BORDER-RIGHT: medium none; PADDING-TOP: 3pt">
<span style="font-weight:bold">From: </span>Nils Jeisecke <<a href="mailto:njeisecke@saltation.de">njeisecke@saltation.de</a>><br>
<span style="font-weight:bold">Date: </span>Wed, 25 Apr 2012 10:56:14 +0200<br>
<span style="font-weight:bold">To: </span>Robin Burchell <<a href="mailto:robin+qt@viroteck.net">robin+qt@viroteck.net</a>><br>
<span style="font-weight:bold">Cc: </span><<a href="mailto:releasing@qt-project.org">releasing@qt-project.org</a>><br>
<span style="font-weight:bold">Subject: </span>Re: [Releasing] Change I39f1b1fd in 4.8.2<br>
</div>
<div><br>
</div>
<div class="gmail_extra">Hi Robin,</div>
<div class="gmail_extra"><br>
</div>
<div class="gmail_extra">thanks for the feedback.</div>
<div class="gmail_extra"><br>
</div>
<div class="gmail_extra">I'm not sure how to implement a really meaningful unit test for this Mac OS specific problem.</div>
<div class="gmail_extra">Maybe you could have a look at patch set 1 which tries to solve this on the cocoa level.</div>
<div class="gmail_extra"><br>
</div>
<div class="gmail_extra">But I guess this is the wrong list for further discussion anyway.</div>
<div class="gmail_extra"><br>
</div>
<div class="gmail_extra">Nils</div>
<div class="gmail_extra"><br>
<div class="gmail_quote">On Wed, Apr 25, 2012 at 10:28 AM, Robin Burchell <span dir="ltr">
<<a href="mailto:robin+qt@viroteck.net" target="_blank">robin+qt@viroteck.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi,<br>
<br>
2012/4/25 Nils Jeisecke <<a href="mailto:njeisecke@saltation.de">njeisecke@saltation.de</a>>:<br>
<div class="im">> I case it's not too late, would some consider<br>
> reviewing <a href="https://codereview.qt-project.org/#change,24075" target="_blank">https://codereview.qt-project.org/#change,24075</a> for 4.8.2?<br>
<br>
</div>
As noted on the review, you're missing unit tests, which would<br>
personally make me wary of slipping this in without some pretty<br>
intensive testing at this stage. It may be too late for that.<br>
<br>
Your bug report notes that this also affects 4.7.4 and all 4.8<br>
releases so far, so I'd be inclined to let it go into 4.8.3 - which,<br>
given the past two releases, won't be far away.<br>
<br>
BR,<br>
<br>
Robin<br>
</blockquote>
</div>
<br>
<br clear="all">
<div><br>
</div>
</div>
_______________________________________________ Releasing mailing list <a href="mailto:Releasing@qt-project.org">
Releasing@qt-project.org</a> <a href="http://lists.qt-project.org/mailman/listinfo/releasing">
http://lists.qt-project.org/mailman/listinfo/releasing</a> </span>
</body>
</html>