Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 338615

Summary:	com.goole.gwtjsonrpc, com.google.gwtorm, com.google.gerrit.* should have execution environment set to 1.6
Product:	[Tools] Orbit	Reporter:	Steffen Pingel <steffen.pingel>
Component:	bundles	Assignee:	Chris Aniszczyk <caniszczyk>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	P3	CC:	caniszczyk, david_williams, sop
Version:	unspecified
Target Milestone:	Indigo M6
Hardware:	PC
OS:	Linux
Whiteboard:
Bug Depends on:
Bug Blocks:	338616

Description Steffen Pingel

2011-03-01 22:22:46 EST

It appears that classes in these bundles where compiled against 1.6 and hence the execution environment should be updated:

 com.goole.gwtjsonrpc
 com.google.gwtorm
 com.google.gerrit.common
 com.google.gerrit.prettify
 com.google.gerrit.reviewdb

Not sure if 1.6 is strictly necessary but the original libraries specify that in their poms and we would need to recompile classes to make the bundles work on 1.5.

Comment 1 Shawn Pearce

2011-03-02 11:27:17 EST

(In reply to comment #0)
> Not sure if 1.6 is strictly necessary but the original libraries specify that
> in their poms and we would need to recompile classes to make the bundles work
> on 1.5.

We might be able to rebuild these using Java 5. There were some APIs we needed in Gerrit that forced us to require Java 6... but I think those are all on the server. The only problem with the client side might be using String.isEmpty() rather than String.length() == 0. Those can be easily fixed by hand and submitted back upstream. But keeping them out might be harder. :-)

Comment 2 David Williams

2011-03-02 11:34:37 EST

> We might be able to rebuild these using Java 5. 

In general, this wouldn't be required unless someone (some client, or adopter) has that specific need. As a whole, Indigo release will required Java 6, so I don't think we need to do it just to do it. 

But ... EE should definitely be accurate.

Comment 3 David Williams

2011-03-02 12:45:06 EST

Chris, I think these are "yours"? Seems M6 would be nice if you can fix (and agree with the fix) in next day or two. 

Thanks,

Comment 4 Chris Aniszczyk

2011-03-02 18:36:37 EST

Done, tagged as v201103020530.