Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 337069 - MVN: object teams parent pom is missing required dependency BCEL
Summary: MVN: object teams parent pom is missing required dependency BCEL
Status: VERIFIED FIXED
Alias: None
Product: Objectteams
Classification: Tools
Component: Releng (show other bugs)
Version: 0.8   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 0.8 M6   Edit
Assignee: Olaf Otto CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-02-13 11:34 EST by Olaf Otto CLA
Modified: 2013-06-23 12:42 EDT (History)
1 user (show)

See Also:

Attachments
mylyn/context/zip (917 bytes, application/octet-stream)
2011-03-02 16:20 EST, Olaf Otto CLA 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Olaf Otto CLA 2011-02-13 11:34:04 EST 
In rev. 1318 of svn://dev.eclipse.org/svnroot/tools/org.eclipse.objectteams/trunk/maven/infrastructure/parent-pom, the pom specifies usage of BCEL as part of the instrumentation agent's classpath, however BCEL is not declared as a dependency of the POM. This leads to coclassdeffound errors in sub-projects, specifically when running tests.

Recommended solution: Added BCEL as dependency with scope=test.
Comment 1 Olaf Otto CLA 2011-03-02 16:20:22 EST 
Correction: scope=provided appears more suitable. BCEL is (at first) most likely to be used during test runs, however any program launch outside the eclipse IDE does require the presence of this agent, thus "provided" is the better choice, as it signals that the agent dependency shall be provided for OT runs.
Comment 2 Olaf Otto CLA 2011-03-02 16:20:49 EST 
Created attachment 190204 [details]
mylyn/context/zip
Comment 3 Olaf Otto CLA 2011-03-02 16:22:00 EST 
Fixed. Added BCEL with scope=provided to 0.8.0 SNAPSHOT.
Comment 4 Stephan Herrmann CLA 2011-03-03 08:43:14 EST 
Closing as released for 0.8M6.
Comment 5 Stephan Herrmann CLA 2013-06-23 12:42:39 EDT 
Works nicely with OT/J 2.2.0