Bug 286009 - Weaving properties should be persistence properties as well
Summary: Weaving properties should be persistence properties as well
Status: NEW
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Eclipselink (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-08-07 11:33 EDT by Michael Keith CLA
Modified: 2022-06-09 10:07 EDT (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Keith CLA 2009-08-07 11:33:06 EDT 
Right now you have to set the two weaving properties as system properties in order to get the weaver to respond. The two props in PersistenceWeaver:

public static final String WEAVING_OUTPUT_PATH 
                                    = "eclipselink.weaving.output.path";
public static final String WEAVING_SHOULD_OVERWRITE 
                                    = "eclipselink.weaving.overwrite.existing";

Weaving in JPA is going to require reading the persistence.xml file, so there is no reason why we should not be able to use persistence properties.
(Even though this is foundation code, the property should be abstracted out.)
Comment 1 Tom Ware CLA 2009-08-31 09:23:04 EDT 
Setting target and priority.  See the following page for the meanings of these fields:

http://wiki.eclipse.org/EclipseLink/Development/Bugs/Guidelines
Comment 2 Eclipse Webmaster CLA 2022-06-09 10:07:46 EDT 
The Eclipselink project has moved to Github: https://github.com/eclipse-ee4j/eclipselink