[erlang-patches] Bug fixes for cover

José Valim jose.valim@REDACTED
Wed Jan 30 19:07:47 CET 2013


Hello OTP team, congratulations on the release!

I have noticed that the cover bug fixes were not included in R16A.

Is there something pending that I could do?

Thank you,


*José Valim*
www.plataformatec.com.br
Skype: jv.ptec
Founder and Lead Developer


On Thu, Jan 10, 2013 at 3:08 AM, Fredrik <fredrik@REDACTED> wrote:

>  Great, I'll re-fetch this.
> Thanks for your contribution!
>
>
> BR Fredrik Gustafsson
> Erlang OTP Team
> On 01/10/2013 11:04 AM, José Valim wrote:
>
> Anyway, here are the patches without ?line.
>
>  git fetch git://github.com/josevalim/otp.git cover-patches-no-line
>
>  https://github.com/josevalim/otp/compare/cover-patches-no-line
> https://github.com/josevalim/otp/compare/cover-patches-no-line.patch
>
>
>  *José Valim*
>  www.plataformatec.com.br
> Skype: jv.ptec
> Founder and Lead Developer
>
>
> On Thu, Jan 10, 2013 at 10:53 AM, José Valim <
> jose.valim@REDACTED> wrote:
>
>> I have added ?line to keep consistent with the remaining of the test
>> file. Do you want me to go ahead and remove ?line from the whole *
>> lib/tools/test/cover_SUITE.erl* file or this is something you have
>> already done internally and I should remove just the ones in my patch?
>>
>>
>>
>>  *José Valim*
>>  www.plataformatec.com.br
>> Skype: jv.ptec
>> Founder and Lead Developer
>>
>>
>>   On Thu, Jan 10, 2013 at 10:45 AM, Fredrik <fredrik@REDACTED> wrote:
>>
>>>  Hello,
>>> Your patch has been into review and the feedback you got back was that
>>> everything looks good but you have to remove the '?line' macros, because
>>> they are not used and thus has no effect anymore.
>>> Please fix this and give me a notice when it is done
>>>
>>> BR Fredrik Gustafsson
>>> Erlang OTP Team
>>>
>>> On 11/28/2012 02:11 PM, José Valim wrote:
>>>
>>>  Hello,
>>>
>>>  I am sending a couple bug fixes for cover.
>>>
>>>  I have broken those fixes into three granular commits.
>>> The commit messages contains the rationale behind them.
>>>
>>>  One of the commits changes cover to get the source from
>>> "Module:module_info(compile)" if the current heuristic that
>>> traverses directories fails. In my opinion, we could rely
>>> solely on the source information and remove the heuristic
>>> completely but I have kept the current heuristic as the first
>>> mechanism in order to minimize the impact of the changes.
>>>
>>>  As such, I would appreciate if those changes could be merged
>>> into maint. :)
>>>
>>>  git fetch git://github.com/josevalim/otp.git cover-patches
>>>
>>>  https://github.com/josevalim/otp/compare/cover-patches
>>> https://github.com/josevalim/otp/compare/cover-patches.patch
>>>
>>>  Thank you,
>>>
>>>
>>>  *José Valim*
>>>  www.plataformatec.com.br
>>> Skype: jv.ptec
>>> Founder and Lead Developer
>>>
>>>
>>>
>>>  _______________________________________________
>>> erlang-patches mailing listerlang-patches@REDACTED://erlang.org/mailman/listinfo/erlang-patches
>>>
>>>
>>>
>>> _______________________________________________
>>> erlang-patches mailing list
>>> erlang-patches@REDACTED
>>> http://erlang.org/mailman/listinfo/erlang-patches
>>>
>>>
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://erlang.org/pipermail/erlang-patches/attachments/20130130/8d5a84f6/attachment.htm>


More information about the erlang-patches mailing list