Re: Release the reltool script
Siri Hansen <[email protected]>
| Newsgroups | gmane.comp.lang.erlang.patches |
|---|---|
| Message-ID | <CAGqERUGZdxgTOBjbZagVSOP1wyNx2T7bAUOxmxJU==ZD7SNfng@mail.gmail.com> |
Hi Håkan! I have looked at your patch and it looks good :) Only some minors: * 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? * In the start_gui/1 test you have commented out the actual start of the gui - is this intentional, and if so why? * Could you please undo the change of vsn.mk? This file will be updated at next release. * 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? Thanks for you contribution! Regards /siri 2013/6/26 Fredrik <[email protected]> > On 06/26/2013 04:04 PM, Håkan Mattsson wrote: > >> With this patch the reltool script is released among the the other top >> level executables, such as dialyzer, ct_run etc. >> I did also take the liberty of enhancing the script and renaming it >> script to "reltool". >> >> A new test suite and a new man(1) page is included. >> >> git fetch git://github.com/hawk/otp.git hawk/reltool_script >> >> or viewed here >> >> https://github.com/hawk/otp/**compare/hawk/reltool_script<https://github.com/hawk/otp/compare/hawk/reltool_script> >> https://github.com/hawk/otp/**compare/hawk/reltool_script.**patch<https://github.com/hawk/otp/compare/hawk/reltool_script.patch> >> >> /Håkan >> ______________________________**_________________ >> erlang-patches mailing list >> [email protected] >> http://erlang.org/mailman/**listinfo/erlang-patches<http://erlang.org/mailman/listinfo/erlang-patches> >> > Hello Håkan, > I've fetched your patch and it should soon be visible in the 'pu' branch > on github. > Thanks, > > -- > > BR Fredrik Gustafsson > Erlang OTP Team > > > ______________________________**_________________ > erlang-patches mailing list > [email protected] > http://erlang.org/mailman/**listinfo/erlang-patches<http://erlang.org/mailman/listinfo/erlang-patches> > _______________________________________________ erlang-patches mailing list [email protected] http://erlang.org/mailman/listinfo/erlang-patches