Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[2.7] Ant build - MOXy tests fix #1907

Merged
merged 1 commit into from
Jul 4, 2023

Conversation

rfelcman
Copy link
Contributor

@rfelcman rfelcman commented Jul 4, 2023

Merge JVM arguments from internal store (antbuild.xml) and outside (passed with -Dadditional.jvmargs=.... command line parameter).
To correctly handle MOXy test execution command like
ant -f antbuild.xml -Dadditional.jvmargs='-javaagent:/usr/local/javaApplications/jacoco-0.8.10/lib/jacocoagent.jar=destfile=/0/0000/jacoco.out,append=true,includes=javax.persistence*:org.eclipse.persistence*,excludes=javax.resource*:org.eclipse.persistence.internal.libraries*,sessionid=lrg,output=file' test-moxy with needed internal -Ddummy2=dummy --add-opens java.base/java.math=ALL-UNNAMED JVM options.

Merge JVM arguments from internal store (antbuild.xml) and outside (passed with '-Dadditional.jvmargs=....' command line parameter).

Signed-off-by: Radek Felcman <radek.felcman@oracle.com>
@rfelcman rfelcman merged commit 350d193 into eclipse-ee4j:2.7 Jul 4, 2023
@rfelcman rfelcman deleted the moxy_jdk21_test_fix_2.7 branch July 4, 2023 11:53
rfelcman added a commit to rfelcman/eclipselink that referenced this pull request Jul 4, 2023
rfelcman added a commit that referenced this pull request Jul 4, 2023
* Revert "Ant build - MOXy tests fix (#1907)"

This reverts commit 350d193.

* Ant build - MOXy tests fix 2.0
Merge JVM arguments from internal store (antbuild.xml) and outside (passed with '-Dadditional.jvmargs=....' command line parameter).
There was issue with previous fix, that Ant will initialize undefined property automatically during calls like value="${additional.jvmargs}..." into value '${additional.jvmargs}'.

Signed-off-by: Radek Felcman <radek.felcman@oracle.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant