Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

com.sun.tools.xjc should run on JRE #1674

Closed
VirtualTim opened this issue Dec 16, 2022 · 1 comment · Fixed by #1690
Closed

com.sun.tools.xjc should run on JRE #1674

VirtualTim opened this issue Dec 16, 2022 · 1 comment · Fixed by #1690

Comments

@VirtualTim
Copy link

VirtualTim commented Dec 16, 2022

When running a project depending on

<dependency>
	<groupId>com.sun.xml.bind</groupId>
	<artifactId>jaxb-xjc</artifactId>
	<version>4.0.1</version>
</dependency>

using a JRE (not JDK) I get the following error on startup:

Error occurred during initialization of boot layer
java.lang.module.FindException: Module jdk.compiler not found, required by com.sun.tools.xjc

Is jaxb-xjc expected to be able to work with a JRE, or does it only support JDK's?

@lukasj lukasj changed the title com.sun.tools.xjc depends on jdk.compiler (ie: JRE not supported) com.sun.tools.xjc should run on JRE Jan 24, 2023
@lukasj
Copy link
Member

lukasj commented Jan 24, 2023

there is no reason for XJC to require JDK, JRE should be enough. It is JXC tool who requires JDK

lukasj added a commit to lukasj/jaxb-ri that referenced this issue Jan 24, 2023
Signed-off-by: Lukas Jungmann <lukas.jungmann@oracle.com>
lukasj added a commit that referenced this issue Jan 24, 2023
Signed-off-by: Lukas Jungmann <lukas.jungmann@oracle.com>
lukasj added a commit to lukasj/jaxb-ri that referenced this issue Feb 1, 2023
Signed-off-by: Lukas Jungmann <lukas.jungmann@oracle.com>
(cherry picked from commit 545af03)
lukasj added a commit that referenced this issue Feb 1, 2023
Signed-off-by: Lukas Jungmann <lukas.jungmann@oracle.com>
(cherry picked from commit 545af03)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants