[rfc][icedtea-web] returning codecoverage

[Jiri Vanek]

On 10/10/2014 03:57 PM, Jiri Vanek wrote:
> As jacoco 0.7 with my bootclaspath patch is finaly reaching fedora in f21, it is time to abandon
> custom built of jacoco
>
> diff -r c6af2f50a95e Makefile.am
> --- a/Makefile.am    Thu Oct 09 18:42:42 2014 +0200
> +++ b/Makefile.am    Fri Oct 10 15:55:00 2014 +0200
> @@ -70,8 +70,8 @@
>   export JACOCO_BASE_EXCLUDE=org.junit.*:junit.*
>   export JACOCO_ADVANCED_EXCLUDE=:*jacoco*:java.lang.*:java.reflect.*:java.util.*:sun.reflect.*
>   export JACOCO_AGENT_SWITCH="$(JACOCO_AGENT_SWITCH_BODY)=excludes=$(JACOCO_BASE_EXCLUDE)"
> -export
> JACOCO_AGENT_JAVAWS_SWITCH=\"$(JACOCO_AGENT_SWITCH_BODY)=excludes=$(JACOCO_BASE_EXCLUDE)$(JACOCO_ADVANCED_EXCLUDE),xboot=true,destfile=$(JACOCO_JAVAWS_RESULTS)\"
>
> -export
> JACOCO_AGENT_PLUGIN_SWITCH=\"$(JACOCO_AGENT_SWITCH_BODY)=excludes=$(JACOCO_BASE_EXCLUDE)$(JACOCO_ADVANCED_EXCLUDE),xboot=true,destfile=$(JACOCO_PLUGIN_RESULTS)\"
>
> +export
> JACOCO_AGENT_JAVAWS_SWITCH=\"$(JACOCO_AGENT_SWITCH_BODY)=excludes=$(JACOCO_BASE_EXCLUDE)$(JACOCO_ADVANCED_EXCLUDE),inclbootstrapclasses=true,destfile=$(JACOCO_JAVAWS_RESULTS)\"
>
> +export
> JACOCO_AGENT_PLUGIN_SWITCH=\"$(JACOCO_AGENT_SWITCH_BODY)=excludes=$(JACOCO_BASE_EXCLUDE)$(JACOCO_ADVANCED_EXCLUDE),inclbootstrapclasses=true,destfile=$(JACOCO_PLUGIN_RESULTS)\"
>
>   export JACOCO_OPERATOR_EXEC=$(BOOT_DIR)/bin/java $(EMMA_JAVA_ARGS) -cp
> $(JACOCO_OPERATOR_DIR):$(JACOCO_CLASSPATH):. org.jacoco.operator.Main
>
>
>
>
> should do this, and make coverage alive in deily report again. Please not, there is bug in fedora
> jacoco ackages - https://bugzilla.redhat.com/show_bug.cgi?id=1151442 - As alexnader is maintainer..
> I will rather fix it asap :)
>
>
> However - th unites seems not to be covered properly. Investigation in progress, another changeset
> will come.. somewhen.
>
>
> J.

As i'm probably only user of this ever, and even maintinaing this patch (both upstream and in 
fedora) I will dare to push, as I wont to start weekend report on f21 today.


J.