Bug 365035 - include new bundle org.eclipse.jdt.annotation in build and packaging
Summary: include new bundle org.eclipse.jdt.annotation in build and packaging
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Releng (show other bugs)
Version: 3.8   Edit
Hardware: Other Linux
: P3 normal (vote)
Target Milestone: 3.8 M4   Edit
Assignee: Dani Megert CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 186342
  Show dependency tree
 
Reported: 2011-11-29 05:20 EST by Stephan Herrmann CLA
Modified: 2011-12-01 11:23 EST (History)
7 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Stephan Herrmann CLA 2011-11-29 05:20:47 EST
Hi Kim,

I just pushed a new tiny plugin project to JDT/Core's git:
http://git.eclipse.org/c/jdt/eclipse.jdt.core.git/tree/org.eclipse.jdt.annotation

This is supposed to be shipped together with the patch for bug 186342.
That bug is currently in its end game, and if nothing bad is detected
during the next few days we will want to include this like in Friday's
warm-up build towards M4.

Could you please coordinate with Satyam and Srikanth the necessary steps
to include this bundle in the build as part of the jdt feature?

Is it just adding entries to feature.xml and jdtcore.map?
How must the entry to the map file be initialized to participate in
automatic tagging?

TIA
Comment 1 John Arthorne CLA 2011-11-29 09:07:14 EST
Was there some discussion already about what namespace to use for these annotations? It would be interesting to be able to use these annotations in the Platform but being in the JDT feature would prevent that. I suspect other potential consumers would also not like the apparent JDT dependency in their runtime code (even though it is a single project that could be consumed independently from the JDT feature). I wonder if they should be placed in a core or equinox namespace and put into the RCP feature.
Comment 2 Dani Megert CLA 2011-11-29 09:17:41 EST
(In reply to comment #1)
> Was there some discussion already about what namespace to use for these
> annotations? It would be interesting to be able to use these annotations in the
> Platform but being in the JDT feature would prevent that. I suspect other
> potential consumers would also not like the apparent JDT dependency in their
> runtime code (even though it is a single project that could be consumed
> independently from the JDT feature). I wonder if they should be placed in a
> core or equinox namespace and put into the RCP feature.

Good point, though I'm not sure why people would prefer "equinox" over "jdt" or the other way around unless they don't get any additional stuff besides the annotations.
Comment 3 Dani Megert CLA 2011-11-29 09:19:38 EST
> Good point, though I'm not sure why people would prefer "equinox" over "jdt" or
> the other way around unless they don't get any additional stuff besides the
> annotations.
Also note that the annotations are Java specific i.e. they are not applicable for e.g. CDT developers. Hence a "jdt" or "java" should probably be in the name.
Comment 4 Stephan Herrmann CLA 2011-11-29 09:27:27 EST
(In reply to comment #1)
> Was there some discussion already about what namespace to use for these
> annotations? 

Discussion was held within JDT, not with eclipse at large.

> It would be interesting to be able to use these annotations in the
> Platform

glad to here that from you :)

> but being in the JDT feature would prevent that.

Why do you think so? The new bundle has no dependencies (except BREE-1.5)
so what exactly should prevent platform components from compiling against 
this bundle?

> I suspect other
> potential consumers would also not like the apparent JDT dependency in their
> runtime code

Well, it's not strictly a runtime dependency.
BTW: would this work?
  Import-Package: org.eclipse.jdt.annotation;resolution="optional"
This could help the compiler and still allow to run any bundle referencing
the annotations without having the annotations at runtime?

> (even though it is a single project that could be consumed
> independently from the JDT feature)

my point exactly.

> I wonder if they should be placed in a
> core or equinox namespace and put into the RCP feature.

Maybe from a build/packaging perspective there are good reasons for this
but I think process is be much easier if JDT keeps ownership of this bundle.
Comment 5 Dani Megert CLA 2011-11-29 10:58:01 EST
John, I'd like to go with the proposed bundle / name space. Note that those who don't like the name can simply use their own annotations and configure JDT accordingly.
Comment 6 John Arthorne CLA 2011-11-29 14:17:07 EST
Ok, I buy the argument that since this is a source annotation that won't be retained at runtime, we should be able to use these annotations in our source without adding a JDT dependency in our bundles. So, the current name is ok with me.
Comment 7 Dani Megert CLA 2011-11-30 04:36:27 EST
Kim, is there anything else to do than adding it to the feature and the map file?
Comment 8 Kim Moir CLA 2011-11-30 07:49:57 EST
That should do it.
Comment 9 Dani Megert CLA 2011-11-30 08:11:25 EST
(In reply to comment #7)
> Kim, is there anything else to do than adding it to the feature and the map
> file?

OK, then I can take care of this myself.
Comment 10 Dani Megert CLA 2011-11-30 08:26:40 EST
The feature and the map file have been updated.
Comment 11 Stephan Herrmann CLA 2011-11-30 10:28:02 EST
(In reply to comment #10)
> The feature and the map file have been updated.

Thanks!
Comment 12 Kim Moir CLA 2011-12-01 11:05:12 EST
The jdt.annotation.source bundle is missing the license files.  Please add them and update the build.properties.  From N20111130-2000

Plugin directory missing required files: /buildtest/N20111130-2000/eclipse-testing/test-eclipse/eclipse/plugins/org.eclipse.jdt.annotation.source_1.0.0.N20111130-2000.jar;
Comment 13 Markus Keller CLA 2011-12-01 11:23:15 EST
(In reply to comment #12)
> The jdt.annotation.source bundle is missing the license files.  Please add them
> and update the build.properties.  From N20111130-2000

Thanks, I've added the about.html to the source build.

commit 476fcf718e0e7f9c1d93eac18e530f17ac58b00f