[
https://issues.apache.org/jira/browse/JDO-639?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12769671#action_12769671
]
Michelle Caisse commented on JDO-639:
-------------------------------------
The variables jdo.tck.paclasses.sources and jdo.tck.pcclasses.sources are
always used together (as a union of the two sets). Same for *.files. Would it
be a good idea now to further simplify things by using only one variable
(pcclasses) and note in a comment that it also contains paclasses?
> tck enhancement should make use of feature to enhance an entire directory
> -------------------------------------------------------------------------
>
> Key: JDO-639
> URL: https://issues.apache.org/jira/browse/JDO-639
> Project: JDO
> Issue Type: Improvement
> Components: tck2
> Affects Versions: JDO 2 maintenance release 2
> Reporter: Michael Bouschen
> Assignee: Michelle Caisse
> Attachments: enhance.txt, jdo-639.txt
>
>
> The current enhancer call as part of the runtck goal takes a list of .jdo
> files as an argument (see property jdo.tck.jdometadata.files in
> project.properties and maven.xml). The new enhancer invocation API allows to
> enhance all files of a given directory. We should investigate to make use of
> this feature in order to get rid of listing all the .jdo in the property
> jdo.tck.jdometadata.files.
> There is a similar issue with the properties jdo.tck.pcclasses.sources and
> jdo.tck.pcclasses.files:
> - Property jdo.tck.pcclasses.sources is used when checking whether
> (re)enhancing is required. Maybe a pattern
> org/apache/jdo/tck/api/**/*.java, org/apache/jdo/tck/pc/**/*.java
> can be used instead of listing all the persistent capable classes explicitly.
> - Property jdo.tck.pcclasses.files is used when copying the class file into
> the identitytype specific subdirectories before running the enhancer. Maybe a
> similar pattern could replace the list of class files.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
