<div dir="ltr">Hi Håkan!<div><br></div><div>I have looked at your patch and it looks good :) Only some minors:</div><div><br></div><div style>* You have copied a lot of code from reltool_server_SUITE into reltool_escript_SUITE. Could you please move these functions into e.g. reltool_test_lib.erl instead?</div>
<div style><br></div><div style>* In the start_gui/1 test you have commented out the actual start of the gui - is this intentional, and if so why?</div><div style><br></div><div style>* Could you please undo the change of <a href="http://vsn.mk">vsn.mk</a>? This file will be updated at next release.</div>
<div style><br></div><div style>* And finally, when you renamed the escript it looks like you created a new file and removed the old instead of doing 'git mv'? Is this correct? If so, could you please fix this so the full history of the file is still available?</div>
<div style><br></div><div style>Thanks for you contribution!</div><div style>Regards</div><div style>/siri</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/6/26 Fredrik <span dir="ltr"><<a href="mailto:fredrik@erlang.org" target="_blank">fredrik@erlang.org</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On 06/26/2013 04:04 PM, Håkan Mattsson wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
With this patch the reltool script is released among the the other top<br>
level executables, such as dialyzer, ct_run etc.<br>
I did also take the liberty of enhancing the script and renaming it<br>
script to "reltool".<br>
<br>
A new test suite and a new man(1) page is included.<br>
<br>
   git fetch git://<a href="http://github.com/hawk/otp.git" target="_blank">github.com/hawk/otp.git</a> hawk/reltool_script<br>
<br>
or viewed here<br>
<br>
   <a href="https://github.com/hawk/otp/compare/hawk/reltool_script" target="_blank">https://github.com/hawk/otp/<u></u>compare/hawk/reltool_script</a><br>
   <a href="https://github.com/hawk/otp/compare/hawk/reltool_script.patch" target="_blank">https://github.com/hawk/otp/<u></u>compare/hawk/reltool_script.<u></u>patch</a><br>
<br>
/Håkan<br>
______________________________<u></u>_________________<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/<u></u>listinfo/erlang-patches</a><br>
</blockquote></div></div>
Hello Håkan,<br>
I've fetched your patch and it should soon be visible in the 'pu' branch on github.<br>
Thanks,<span class="HOEnZb"><font color="#888888"><br>
<br>
-- <br>
<br>
BR Fredrik Gustafsson<br>
Erlang OTP Team</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
______________________________<u></u>_________________<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/<u></u>listinfo/erlang-patches</a><br>
</div></div></blockquote></div><br></div>