Skip to content

Glass and SysId not installed via installAllTools #711

@person4268

Description

@person4268

This is because:
a)

project.getTasks().register("InstallAllTools", task -> {
task.setGroup("GradleRIO");
task.setDescription("Install All Tools");
for (WPITool tool : tools) {
task.dependsOn(tool.getToolInstallTask());
}
});
doesn't iterate over the cppTools array

b)

//toolInstallTask = project.tasks.register("${name}Install".toString(), CppToolInstallTask, name, config, dependency)
is commented out

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).

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions