This Bugzilla instance is deprecated, and most Eclipse projects now use GitHub or Eclipse GitLab. Please see the deprecation plan for details.
Bug 373594 - remove JaCoCo from build
Summary: remove JaCoCo from build
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Releng (show other bugs)
Version: 3.8   Edit
Hardware: PC Windows 7
: P3 major (vote)
Target Milestone: 3.8 M6   Edit
Assignee: Kim Moir CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 378421 (view as bug list)
Depends on:
Blocks: 372867
  Show dependency tree
 
Reported: 2012-03-07 16:38 EST by Kim Moir CLA
Modified: 2012-05-09 09:35 EDT (History)
6 users (show)

See Also:


Attachments
patch (19.88 KB, patch)
2012-03-13 09:34 EDT, Kim Moir CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Kim Moir CLA 2012-03-07 16:38:26 EST

    
Comment 1 Remy Suen CLA 2012-03-08 07:30:07 EST
What is the rationale behind this?
Comment 2 John Arthorne CLA 2012-03-08 11:33:46 EST
There are a few reasons:

1) Jacoco has not actually been functional in our builds for many months. The coverage results for M4 and M5 show 0% coverage across all tests, and same with recent integration builds:

http://download.eclipse.org/eclipse/downloads/drops/S-3.8M4-201112091447/coverage.php

http://download.eclipse.org/eclipse/downloads/drops/S-3.8M5-201201251800/coverage.php

2) Jacoco has been found to be corrupting our class files. See for example bug 342460 and bug 372867. This doesn't seem to be a mature tool yet, which might explain why it is at version 0.5

3) We have no releng resources to focus on improving this in the near term. There are much bigger releng issues we need to sort out first. If things settle down after Juno and there is a volunteer to work on it (and Jacoco is more stable), we can always add it back in the future.
Comment 3 Dani Megert CLA 2012-03-13 03:11:56 EDT
Kim, please fix this asap. It causes random test failures and wastes our time tracking those down.
Comment 4 Dani Megert CLA 2012-03-13 03:58:11 EDT
Is only a change in /org.eclipse.test/library.xml needed?
Comment 6 Kim Moir CLA 2012-03-13 09:34:12 EDT
Created attachment 212548 [details]
patch
Comment 7 Kim Moir CLA 2012-03-13 09:34:35 EDT
Running test build to ensure this doesn't break anything.
Comment 8 Kim Moir CLA 2012-03-13 14:56:56 EDT
Test build successful.  Released for 3.8M6.
Comment 9 Dani Megert CLA 2012-03-14 09:24:19 EDT
Verified in I20120313-2000.
Comment 10 Kim Moir CLA 2012-03-16 10:44:27 EDT
I also released this change to the R4_2_primary branch of org.eclipse.releng.eclipsebuilder
Comment 11 Thomas Watson CLA 2012-05-09 09:35:23 EDT
*** Bug 378421 has been marked as a duplicate of this bug. ***