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

Bug 337911

Summary: Integrate Hermes JPQL Parser code into build
Product: z_Archived Reporter: Neil Hauge <neil.hauge>
Component: EclipselinkAssignee: Nobody - feel free to take it <nobody>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: eric.gwin, pascal.filion, shaun.smith, tom.ware
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Windows 7   
Whiteboard:
Bug Depends on: 338144, 338146, 338248    
Bug Blocks:    
Attachments:
Description Flags
add hermes to nightly process none

Description Neil Hauge CLA 2011-02-22 16:54:53 EST 
The JPQL Parser utility needs to be built, packaged, and published so that Dali can make use of the parser.  This new contribution is detailed in bug 331817.  This bug is being opened for the discussion and tracking of this request.
Comment 1 Neil Hauge CLA 2011-02-22 17:08:44 EST 
The output of this build should be an OSGi bundle, as this needs to be consumed by Dali.  Future requirements may dictate additional packaging needs, but for now, I think the OSGi packaging is all that is required.
Comment 2 Eric Gwin CLA 2011-02-23 09:16:54 EST 
I've added some build files to the project. From the commandline the project seems to compile, but it bnd won't run. I think because it is missing the necessary OSGi classes. Anyway, the files added are simply a start.
Comment 3 Eric Gwin CLA 2011-02-23 09:48:37 EST 
fixed the problem. was typo in BND file. also updated manifest to add exports and other misc properties to static manifest.
Comment 4 Eric Gwin CLA 2011-02-23 14:22:07 EST 
added a "build-hermes" target to the main build, and allowed hermes to build as 1.0.0, yet still depend upon 2.3.0 core.
Comment 5 Eric Gwin CLA 2011-02-23 14:23:26 EST 
also reverted manifest changes. instead I attached proposed changes to bug 337928
Comment 6 Eric Gwin CLA 2011-03-10 14:38:08 EST 
Created attachment 190908 [details]
add hermes to nightly process

This patch modifies the build to:
- add hermes to the core build process
- add hermes to the jpa feature
- add hermes (o.e.p.jpa.jpql*.jar) to the bundle.zip

Note:
-Hermes is excluded from the creation of eclipselink.jar.
-It is not included in the installer.zip
-This patch will not be applied until after the Indigo M6 contibution is published, because testing is not addressed.
-Due to package reorganization, hermes itself no longer has any compile or runtime dependencies on EclipseLink. That code is being submitted as a separate patch to core.
Comment 7 Eric Gwin CLA 2011-03-14 12:03:45 EDT 
merged patch
Comment 8 Eric Gwin CLA 2011-05-05 10:18:59 EDT 
looks like I didn't close this after I merged. I think this bug is resolved, and other issues related to Hermes are dealt with in other bugs.
Comment 9 Eclipse Webmaster CLA 2022-06-09 10:14:46 EDT 
The Eclipselink project has moved to Github: https://github.com/eclipse-ee4j/eclipselink
Comment 10 Eclipse Webmaster CLA 2022-06-09 10:30:04 EDT 
The Eclipselink project has moved to Github: https://github.com/eclipse-ee4j/eclipselink