Hi again, Joe!<div><br></div><div>Could you please explain a bit about the situation where you discovered this problem? I agree that the timeout and VM restart is not very good, and it makes sense to check if the supervisor is suspended. But I'm not really sure if it is correct to allow the upgrade to continue when this error occurs. Even if an error message is printed, I guess it could be quite easy to miss this fact... and the question is if that would be a problem or not? Why is the supervisor suspended in the first place?</div>
<div><br></div><div>Regards</div><div>/siri</div><div> <br><br><div class="gmail_quote">2011/8/25 Siri Hansen <span dir="ltr"><<a href="mailto:erlangsiri@gmail.com">erlangsiri@gmail.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Hi Joe - I've just started looking at this. Do you think it would be possible to add a test case for it?<div>Regards</div><div>/siri<div><div></div><div class="h5"><br><br><div class="gmail_quote">2011/8/24 Joe Williams <span dir="ltr"><<a href="mailto:joe@joetify.com" target="_blank">joe@joetify.com</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div>
Anything I can do regarding this patch? I have happily been running it in production since I submitted it to the list in June.
</div><div><div><br></div><div>-Joe</div><div><br></div>
<div><br>-- <br><div style="font-size:medium">Name: Joseph A. Williams</div><div style="font-size:medium">Email: <a href="mailto:joe@joetify.com" target="_blank">joe@joetify.com</a></div><div style="font-size:medium">
Blog: <a href="http://www.joeandmotorboat.com/" target="_blank">http://www.joeandmotorboat.com/</a></div><div style="font-size:medium">Twitter: <a href="http://twitter.com/williamsjoe" target="_blank">http://twitter.com/williamsjoe</a></div>
<br></div>
</div><div><div></div><div><p style="color:#A0A0A8">On Wednesday, July 6, 2011 at 3:43 PM, Joe Williams wrote:</p>
<blockquote type="cite" style="border-left-style:solid;border-width:1px;margin-left:0px;padding-left:10px">
<span><div><div><div><div>
<div>
<span>Anything I can do to help this patch graduate?</span></div><div><span><br></span></div><div><span>Thanks!</span></div><div><span><br></span></div><div><span>-Joe</span></div><div><span><br>
</span>
<span><br>-- <br><div style="font-size:medium">Name: Joseph A. Williams</div><div style="font-size:medium">Email: <a href="mailto:joe@joetify.com" target="_blank">joe@joetify.com</a></div><div style="font-size:medium">
Blog: <a href="http://www.joeandmotorboat.com" target="_blank">http://www.joeandmotorboat.com</a>/</div><div style="font-size:medium">Twitter: <a href="http://twitter.com/williamsjoe" target="_blank">http://twitter.com/williamsjoe</a></div>
<br></span>
<p style="color:#a0a0a0">On Tuesday, June 14, 2011 at 12:26 PM, Joe Williams wrote:</p><blockquote type="cite"><div>
<span><div><div><div>
<div>
<span>Updated this branch, please refetch.</span></div><div><br></div><div><span></span>git fetch git://<a href="http://github.com/joewilliams/otp.git" style="color:rgb(0, 58, 138)" target="_blank">github.com/joewilliams/otp.git</a> release_handler_1</div>
<div><br></div><div>
<span><br>-- <br><div style="font-size:medium">Name: Joseph A. Williams</div><div style="font-size:medium">Email: <a href="mailto:joe@joetify.com" target="_blank">joe@joetify.com</a></div><div style="font-size:medium">
Blog: <a href="http://www.joeandmotorboat.com" target="_blank">http://www.joeandmotorboat.com</a>/</div><div style="font-size:medium">Twitter: <a href="http://twitter.com/williamsjoe" target="_blank">http://twitter.com/williamsjoe</a></div>
<br></span>
<p style="color:#a0a0a0">On Friday, June 10, 2011 at 8:52 AM, Joe Williams wrote:</p><blockquote type="cite"><div>
<span><div><div><div>
<div>
<span>Great, thanks!</span></div><div><span><br>
</span>
<span><br>-- <br><div style="font-size:medium">Name: Joseph A. Williams</div><div style="font-size:medium">Email: <a href="mailto:joe@joetify.com" target="_blank">joe@joetify.com</a></div><div style="font-size:medium">
Blog: <a href="http://www.joeandmotorboat.com" target="_blank">http://www.joeandmotorboat.com</a>/</div><div style="font-size:medium">Twitter: <a href="http://twitter.com/williamsjoe" target="_blank">http://twitter.com/williamsjoe</a></div>
<br></span>
<p style="color:#a0a0a0">On Friday, June 10, 2011 at 8:51 AM, Raimo Niskanen wrote:</p><blockquote type="cite"><div>
<span><div><div>On Thu, Jun 09, 2011 at 08:20:51AM -0700, Joe Williams wrote:<br><blockquote type="cite"><div>Please fetch:<br><br>git fetch git://<a href="http://github.com/joewilliams/otp.git" target="_blank">github.com/joewilliams/otp.git</a> release_handler_1<br>
<br>This is a different branch with a better commit message and no white space changes.<br></div></blockquote><br>Excellent. I will include your patch in 'pu' after rewording the<br>summary line to imperative form.<br>
<br><blockquote type="cite"><div><br><br><br>-- <br>Name: Joseph A. Williams<br>Email: <a href="mailto:joe@joetify.com" target="_blank">joe@joetify.com</a><br>Blog: <a href="http://www.joeandmotorboat.com" target="_blank">http://www.joeandmotorboat.com</a>/<br>
Twitter: <a href="http://twitter.com/williamsjoe" target="_blank">http://twitter.com/williamsjoe</a><br><br><br>On Thursday, June 9, 2011 at 7:44 AM, Joe Williams wrote:<br><br><blockquote type="cite"><div>Nothing specific, just wondered if anyone had any thoughts on how I dealt with a couple of corner cases in installing releases.<br>
<br>I'll fix things up and get back shortly.<br><br>-- <br>Name: Joseph A. Williams<br>Email: <a href="mailto:joe@joetify.com" target="_blank">joe@joetify.com</a> (<a href="mailto:joe@joetify.com" target="_blank">mailto:joe@joetify.com</a>)<br>
Blog: <a href="http://www.joeandmotorboat.com" target="_blank">http://www.joeandmotorboat.com</a>/<br>Twitter: <a href="http://twitter.com/williamsjoe" target="_blank">http://twitter.com/williamsjoe</a><br><br><br>On Thursday, June 9, 2011 at 12:11 AM, Raimo Niskanen wrote:<br>
<br><blockquote type="cite"><div>On Wed, Jun 08, 2011 at 03:41:37PM -0700, Joe Williams wrote:<br><blockquote type="cite"><div>Any thoughts/feedback on this patch? I realize it doesn't follow the guidelines (<a href="https://github.com/erlang/otp/wiki/Submitting-patches" target="_blank">https://github.com/erlang/otp/wiki/Submitting-patches</a>) exactly and will clean it up soon.<br>
</div></blockquote><br>Anything in particular? I just got caught up in tideous merge work<br>yesterday and missed to include your patch in 'pu', I was about<br>to take it now.<br><br>But if you have a cleanup I can wait for it...<br>
<br><blockquote type="cite"><div><br><br>-- <br>Name: Joseph A. Williams<br>Email: <a href="mailto:joe@joetify.com" target="_blank">joe@joetify.com</a> (<a href="mailto:joe@joetify.com" target="_blank">mailto:joe@joetify.com</a>)<br>
Blog: <a href="http://www.joeandmotorboat.com" target="_blank">http://www.joeandmotorboat.com</a>/<br>Twitter: <a href="http://twitter.com/williamsjoe" target="_blank">http://twitter.com/williamsjoe</a><br><br><br>On Tuesday, June 7, 2011 at 2:33 PM, Joe Williams wrote:<br>
<br><blockquote type="cite"><div> git fetch git://<a href="http://github.com/joewilliams/otp.git" target="_blank">github.com/joewilliams/otp.git</a> (<a href="http://github.com/joewilliams/otp.git" target="_blank">http://github.com/joewilliams/otp.git</a>) (<a href="http://github.com/joewilliams/otp.git" target="_blank">http://github.com/joewilliams/otp.git</a>) release_handler<br>
</div></blockquote></div></blockquote><br><blockquote type="cite"><div>_______________________________________________<br>erlang-patches mailing list<br><a href="mailto:erlang-patches@erlang.org" target="_blank">erlang-patches@erlang.org</a> (<a href="mailto:erlang-patches@erlang.org" target="_blank">mailto:erlang-patches@erlang.org</a>)<br>
<a href="http://erlang.org/mailman/listinfo/erlang-patches" target="_blank">http://erlang.org/mailman/listinfo/erlang-patches</a><br></div></blockquote><br><br>-- <br><br>/ Raimo Niskanen, Erlang/OTP, Ericsson AB<br></div>
</blockquote><br>_______________________________________________<br>erlang-patches mailing list<br><a href="mailto:erlang-patches@erlang.org" target="_blank">erlang-patches@erlang.org</a> (<a href="mailto:erlang-patches@erlang.org" target="_blank">mailto:erlang-patches@erlang.org</a>)<br>
<a href="http://erlang.org/mailman/listinfo/erlang-patches" target="_blank">http://erlang.org/mailman/listinfo/erlang-patches</a><br></div></blockquote></div></blockquote><br>-- <br><br>/ Raimo Niskanen, Erlang/OTP, Ericsson AB<br>
</div></div></span>
</div></blockquote><div>
<br>
</div>
</div>
</div></div></div></span>
</div></blockquote><div>
<br>
</div>
</div>
</div></div></div></span>
</div></blockquote><div>
<br>
</div>
</div>
</div></div></div></div></span>
</blockquote>
<div>
<br>
</div>
</div></div><br>_______________________________________________<br>
erlang-patches mailing list<br>
<a href="mailto:erlang-patches@erlang.org" target="_blank">erlang-patches@erlang.org</a><br>
<a href="http://erlang.org/mailman/listinfo/erlang-patches" target="_blank">http://erlang.org/mailman/listinfo/erlang-patches</a><br>
<br></blockquote></div><br></div></div></div>
</blockquote></div><br></div>