Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 337911 - Integrate Hermes JPQL Parser code into build
Summary: Integrate Hermes JPQL Parser code into build
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Eclipselink (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Nobody - feel free to take it CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 338144 338146 338248
Blocks:
  Show dependency tree
 
Reported: 2011-02-22 16:54 EST by Neil Hauge CLA
Modified: 2022-06-09 10:30 EDT (History)
4 users (show)

See Also:


Attachments
add hermes to nightly process (45.48 KB, patch)
2011-03-10 14:38 EST, Eric Gwin CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
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