-
Notifications
You must be signed in to change notification settings - Fork 70
Description
This is because:
a)
GradleRIO/src/main/java/edu/wpi/first/gradlerio/wpi/dependencies/tools/WPIToolsPlugin.java
Lines 54 to 61 in c7a425a
| project.getTasks().register("InstallAllTools", task -> { | |
| task.setGroup("GradleRIO"); | |
| task.setDescription("Install All Tools"); | |
| for (WPITool tool : tools) { | |
| task.dependsOn(tool.getToolInstallTask()); | |
| } | |
| }); |
b)
GradleRIO/src/main/java/edu/wpi/first/gradlerio/wpi/dependencies/tools/WPICppTool.java
Line 30 in c7a425a
| //toolInstallTask = project.tasks.register("${name}Install".toString(), CppToolInstallTask, name, config, dependency) |
and c) WPICppTool has no getToolInstallTask() method (not that it would be called without a being fixed.
Not sure whether it's intentional as the git history doesn't say much, but I believe I asked Peter (iirc) at Worlds and he said it wasn't intentional and was a bug (the specific question I asked was why installAllTools on Linux didn't install Glass or SysID, but it seems like it doesn't install it at all, just didn't notice as I use the installer on Windows).