Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 355562 - org.eclipse.help and org.eclipse.help.base should not depend on lucene jar...
Summary: org.eclipse.help and org.eclipse.help.base should not depend on lucene jar...
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: User Assistance (show other bugs)
Version: 3.8   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 4.3 M1   Edit
Assignee: John Arthorne CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 340563
Blocks:
  Show dependency tree
 
Reported: 2011-08-23 15:10 EDT by Sami Wagiaalla CLA
Modified: 2012-08-10 05:28 EDT (History)
3 users (show)

See Also:


Attachments
patch. (2.84 KB, patch)
2011-08-23 15:10 EDT, Sami Wagiaalla CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Sami Wagiaalla CLA 2011-08-23 15:10:36 EDT
Created attachment 202030 [details]
patch.

...since lucene no longer has a lucene.jar but has lucene-core.jar instead.

The lucene jar in orbit is an almost empty jar which simply requires lucene.core and exports org.apache.lucene.

The attached patch removes the dependency and replaces it with a dependency on lucene.core where needed.

I can generate two separate git patches against the individual projects if necessary.
Comment 1 Chris Goldthorpe CLA 2011-08-24 17:16:25 EDT
I'll take a look at this next time we move up to a new version of Lucene, which should happen some time in the Eclipse 3.8 timeframe.
Comment 2 Sami Wagiaalla CLA 2011-08-25 12:48:54 EDT
> I'll take a look at this next time we move up to a new version of Lucene, which
> should happen some time in the Eclipse 3.8 timeframe.

That would be great!.. Thanks
Comment 3 Chris Goldthorpe CLA 2011-10-18 14:05:42 EDT
Removing target milestone because upgrading to a new Lucene version is currently not in the Juno schedule.
Comment 4 John Arthorne CLA 2012-06-28 16:53:39 EDT
I made these changes already in bug 340563.
Comment 5 John Arthorne CLA 2012-07-24 13:47:17 EDT
Fixed as part of bug 340563. Pushed to master.