<div dir="ltr">Hi Yuri,<div><br></div><div style>apparently the fix got a bit forgotten because of my typo,</div><div style>I created the below pull request to avoid further confusion</div><div style><a href="https://github.com/erlang/otp/pull/74">https://github.com/erlang/otp/pull/74</a><br>
</div><div style><br></div><div style>Fredrik, could you consider it for inclusion?</div><div style><br></div><div style>thanks</div><div style>Peter</div><div style><br></div><div style><br></div></div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Fri, Aug 30, 2013 at 7:50 AM, Yuri Lukyanov <span dir="ltr"><<a href="mailto:snaky@aboutecho.com" target="_blank">snaky@aboutecho.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Peti,<br>
<br>
Did you have a chance to push your fix to the main repo?<br>
<div class="HOEnZb"><div class="h5"><br>
On Wed, Jul 10, 2013 at 12:34 PM, Peti Gömöri <<a href="mailto:gomoripeti@gmail.com">gomoripeti@gmail.com</a>> wrote:<br>
> Hi Fredrik,<br>
><br>
> sorry, what about this one (maybe it was a wrong ö)<br>
><br>
> git fetch git://<a href="http://github.com/gomoripeti/otp.git" target="_blank">github.com/gomoripeti/otp.git</a> pgö-cover-export-all<br>
><br>
> On Wed, Jul 10, 2013 at 9:11 AM, Fredrik <<a href="mailto:fredrik@erlang.org">fredrik@erlang.org</a>> wrote:<br>
>><br>
>> On 07/10/2013 03:19 AM, Peti Gömöri wrote:<br>
>><br>
>> Hi OTP team,<br>
>><br>
>> here is a patch that addresses the problem Yuri described.<br>
>> (I haven't added any tests or docs though)<br>
>><br>
>> is it true that now you also accept pull requests?<br>
>><br>
>><br>
>><br>
>> git fetch git://<a href="http://github.com/gomoripeti/otp.git" target="_blank">github.com/gomoripeti/otp.git</a> pgö-cover-export-all<br>
>><br>
>><br>
>> <a href="https://github.com/gomoripeti/otp/compare/erlang:maint...pg%C3%B6-cover-export-all" target="_blank">https://github.com/gomoripeti/otp/compare/erlang:maint...pgö-cover-export-all</a><br>
>><br>
>> <a href="https://github.com/gomoripeti/otp/compare/erlang:maint...pg%C3%B6-cover-export-all.patch" target="_blank">https://github.com/gomoripeti/otp/compare/erlang:maint...pgö-cover-export-all.patch</a><br>
>><br>
>><br>
>> br<br>
>> Peter<br>
>><br>
>> On Wed, Jul 10, 2013 at 12:34 AM, Peti Gömöri <<a href="mailto:gomoripeti@gmail.com">gomoripeti@gmail.com</a>><br>
>> wrote:<br>
>>><br>
>>> Hi Yuri,<br>
>>><br>
>>> You are right, while cover compiling from source works (you can use this<br>
>>> as a workaround):<br>
>>> > cover:compile(cover_test, UserOptions = [export_all, debug_info]).<br>
>>> {ok,cover_test}<br>
>>> > cover_test:test().<br>
>>> ok<br>
>>><br>
>>> (because UserOptions from the arguments is taken when cover recompiles<br>
>>> the instrumented forms)<br>
>>> when cover compiling from beam UserOptions = [] is taken.<br>
>>> This could be fixed in cover by taking the compile options from the beam<br>
>>> file as you assumed how it works.<br>
>>><br>
>>> Actually the documentation still says that "Only options defining include<br>
>>> file directories and macros are passed to compile:file/2, everything else is<br>
>>> ignored." apparently a patch from Tobias Schlager added export_all to the<br>
>>> allowed options of cover:compile for the exact same use case as you had. But<br>
>>> this is missing from cover:compile_beam.<br>
>>><br>
>>> May be I try to come up with a patch<br>
>>><br>
>>> br<br>
>>> Peter<br>
>>><br>
>>><br>
>>><br>
>>> On Tue, Jul 9, 2013 at 11:56 AM, Yuri Lukyanov <<a href="mailto:y.snaky@gmail.com">y.snaky@gmail.com</a>> wrote:<br>
>>>><br>
>>>> It seems that there is no way to cover-compile modules with export_all.<br>
>>>> Here is a simple example:<br>
>>>><br>
>>>> cover_test.erl:<br>
>>>><br>
>>>> -module(cover_test).<br>
>>>> test() -> ok.<br>
>>>><br>
>>>><br>
>>>> $ erl<br>
>>>> Erlang R15B01 (erts-5.9.1) [source] [64-bit] [smp:4:4]<br>
>>>> [async-threads:0] [hipe] [kernel-poll:false]<br>
>>>><br>
>>>> Eshell V5.9.1  (abort with ^G)<br>
>>>> 1> c(cover_test, [debug_info,export_all]).<br>
>>>> {ok,cover_test}<br>
>>>> 2> cover_test:test().<br>
>>>> ok<br>
>>>> 3> cover:compile_beam(cover_test).<br>
>>>> {ok,cover_test}<br>
>>>> 4> cover_test:test().<br>
>>>> ** exception error: undefined function cover_test:test/0<br>
>>>> 5><br>
>>>><br>
>>>><br>
>>>> Could someone explain why it is like this? Is it done on purpose?<br>
>>>> Maybe it's a bug?<br>
>>>> The reason I want modules to be cover-compiled with +export_all is<br>
>>>> that it is sometimes convinient to have unit tests outside of a<br>
>>>> module. Before unit tests are run the modules get compiled with<br>
>>>> +export_all for tests to be able to access private functions. But the<br>
>>>> situation is that it is not possibe in this case to enable coverage<br>
>>>> analysys.<br>
>>>> _______________________________________________<br>
>>>> erlang-questions mailing list<br>
>>>> <a href="mailto:erlang-questions@erlang.org">erlang-questions@erlang.org</a><br>
>>>> <a href="http://erlang.org/mailman/listinfo/erlang-questions" target="_blank">http://erlang.org/mailman/listinfo/erlang-questions</a><br>
>>><br>
>>><br>
>><br>
>><br>
>><br>
>> _______________________________________________<br>
>> erlang-patches mailing list<br>
>> <a href="mailto:erlang-patches@erlang.org">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>
>> Hello Peti,<br>
>> I am getting, fatal: Couldn't find remote ref pgö-cover-export-all when I<br>
>> am trying to fetch.<br>
>> Yes pull requests are accepted.<br>
>><br>
>> --<br>
>><br>
>> BR Fredrik Gustafsson<br>
>> Erlang OTP Team<br>
><br>
><br>
><br>
> _______________________________________________<br>
> erlang-questions mailing list<br>
> <a href="mailto:erlang-questions@erlang.org">erlang-questions@erlang.org</a><br>
> <a href="http://erlang.org/mailman/listinfo/erlang-questions" target="_blank">http://erlang.org/mailman/listinfo/erlang-questions</a><br>
><br>
</div></div></blockquote></div><br></div>