Skip to content

Commit 4823999

Browse files
committed
HBX-3146: Remove the use of Ant BuildException from the Maven mojos
Signed-off-by: Koen Aers <[email protected]>
1 parent 7026d1f commit 4823999

File tree

1 file changed

+5
-7
lines changed

1 file changed

+5
-7
lines changed

maven/src/main/java/org/hibernate/tool/maven/AbstractGenerationMojo.java

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -25,15 +25,13 @@
2525
import java.net.URL;
2626
import java.net.URLClassLoader;
2727
import java.util.ArrayList;
28-
import java.util.Arrays;
29-
import java.util.HashSet;
3028
import java.util.Properties;
3129

3230
import org.apache.maven.artifact.DependencyResolutionRequiredException;
3331
import org.apache.maven.plugin.AbstractMojo;
32+
import org.apache.maven.plugin.MojoFailureException;
3433
import org.apache.maven.plugins.annotations.Parameter;
3534
import org.apache.maven.project.MavenProject;
36-
import org.apache.tools.ant.BuildException;
3735
import org.hibernate.tool.api.metadata.MetadataDescriptor;
3836
import org.hibernate.tool.api.metadata.MetadataDescriptorFactory;
3937
import org.hibernate.tool.api.metadata.MetadataConstants;
@@ -92,7 +90,7 @@ public abstract class AbstractGenerationMojo extends AbstractMojo {
9290
@Parameter(defaultValue = "${project}", readonly = true, required = true)
9391
private MavenProject project;
9492

95-
public void execute() {
93+
public void execute() throws MojoFailureException {
9694
ClassLoader original = Thread.currentThread().getContextClassLoader();
9795
try {
9896
Thread.currentThread().setContextClassLoader(createExporterClassLoader(original));
@@ -130,15 +128,15 @@ private RevengStrategy setupReverseEngineeringStrategy() {
130128
return strategy;
131129
}
132130

133-
private Properties loadPropertiesFile() {
131+
private Properties loadPropertiesFile() throws MojoFailureException {
134132
try (FileInputStream is = new FileInputStream(propertyFile)) {
135133
Properties result = new Properties();
136134
result.load(is);
137135
return result;
138136
} catch (FileNotFoundException e) {
139-
throw new BuildException(propertyFile + " not found.", e);
137+
throw new MojoFailureException(propertyFile + " not found.", e);
140138
} catch (IOException e) {
141-
throw new BuildException("Problem while loading " + propertyFile, e);
139+
throw new MojoFailureException("Problem while loading " + propertyFile, e);
142140
}
143141
}
144142

0 commit comments

Comments
 (0)