Skip to content

Commit 81bc2e8

Browse files
authored
Warn if vendordep with duplicate uuid is found (#242)
1 parent dd5e4ca commit 81bc2e8

1 file changed

Lines changed: 14 additions & 0 deletions

File tree

src/main/java/edu/wpi/first/nativeutils/vendordeps/WPIVendorDepsExtension.java

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -190,6 +190,20 @@ private JsonDependency parse(File f) {
190190
private void load(JsonDependency dep) throws VendorParsingException {
191191
// Don"t double-add a dependency!
192192
if (dependencySet.findByName(dep.uuid) != null) {
193+
String requiredFrcYear = frcYear.getOrNull();
194+
if (requiredFrcYear != null) {
195+
if (!requiredFrcYear.equals(dep.frcYear)) {
196+
log.logError("Warning! Ignoring duplicate vendordep: " + dep.fileName
197+
+ " because it has the wrong year.");
198+
return;
199+
}
200+
}
201+
NamedJsonDependency duplicateDep = dependencySet.findByName(dep.uuid);
202+
log.logErrorHead(
203+
"Warning! Duplicate Vendordeps detected. " + dep.fileName + " and "
204+
+ duplicateDep.getDependency().fileName);
205+
log.logError("have the same UUID: " + dep.uuid);
206+
log.logError("Remove one of these vendordeps to avoid conflicts.");
193207
return;
194208
}
195209

0 commit comments

Comments
 (0)