[hal] Fix potential gamedata out of bounds read (#3983)

The size was uninitialized.  If the size is smaller than the data,
NetComm just updates the size and does not initialize the data.

CMakeLists.txt

@@ -145,6 +145,8 @@ if (USE_VCPKG_EIGEN)
 set (EIGEN_VCPKG_REPLACE "find_package(Eigen3 CONFIG)")
 endif()
 
+find_package(LIBSSH 0.7.1)
+
 if (WITH_FLAT_INSTALL)
 set(WPIUTIL_DEP_REPLACE "include($\{SELF_DIR\}/wpiutil-config.cmake)")
 set(NTCORE_DEP_REPLACE "include($\{SELF_DIR\}/ntcore-config.cmake)")
@@ -245,7 +247,6 @@ if (WITH_TESTS)
     include(GoogleTest)
 endif()
 
-add_subdirectory(fieldImages)
 add_subdirectory(wpiutil)
 add_subdirectory(ntcore)
 
@@ -254,10 +255,14 @@ if (WITH_WPIMATH)
 endif()
 
 if (WITH_GUI)
+    add_subdirectory(fieldImages)
     add_subdirectory(imgui)
     add_subdirectory(wpigui)
     add_subdirectory(glass)
     add_subdirectory(outlineviewer)
+    if (LIBSSH_FOUND)
+        add_subdirectory(roborioteamnumbersetter)
+    endif()
 endif()
 
 if (WITH_WPILIB OR WITH_SIMULATION_MODULES)

build.gradle

@@ -19,9 +19,9 @@ plugins {
     id 'idea'
     id 'visual-studio'
     id 'net.ltgt.errorprone' version '2.0.2' apply false
-    id 'com.github.johnrengelman.shadow' version '7.1.1' apply false
-    id 'com.diffplug.spotless' version '6.0.5' apply false
-    id 'com.github.spotbugs' version '5.0.3' apply false
+    id 'com.github.johnrengelman.shadow' version '7.1.2' apply false
+    id 'com.diffplug.spotless' version '6.1.2' apply false
+    id 'com.github.spotbugs' version '5.0.4' apply false
 }
 
 wpilibVersioning.buildServerMode = project.hasProperty('buildServer')
@@ -147,5 +147,5 @@ ext.getCurrentArch = {
 }
 
 wrapper {
-    gradleVersion = '7.3.2'
+    gradleVersion = '7.3.3'
 }

cmake/modules/FindLIBSSH.cmake

@@ -0,0 +1,116 @@
+# - Try to find LibSSH
+# Once done this will define
+#
+#  LIBSSH_FOUND - system has LibSSH
+#  LIBSSH_INCLUDE_DIRS - the LibSSH include directory
+#  LIBSSH_LIBRARIES - link these to use LibSSH
+#  LIBSSH_VERSION -
+#
+#  Author Michal Vasko <mvasko@cesnet.cz>
+#  Copyright (c) 2020 CESNET, z.s.p.o.
+#
+#  Redistribution and use in source and binary forms, with or without
+#  modification, are permitted provided that the following conditions
+#  are met:
+#
+#  1. Redistributions of source code must retain the copyright
+#     notice, this list of conditions and the following disclaimer.
+#  2. Redistributions in binary form must reproduce the copyright
+#     notice, this list of conditions and the following disclaimer in the
+#     documentation and/or other materials provided with the distribution.
+#  3. The name of the author may not be used to endorse or promote products
+#     derived from this software without specific prior written permission.
+#
+#  THIS SOFTWARE IS PROVIDED BY THE AUTHOR ``AS IS'' AND ANY EXPRESS OR
+#  IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES
+#  OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED.
+#  IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR ANY DIRECT, INDIRECT,
+#  INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT
+#  NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+#  DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+#  THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+#  (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
+#  THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+#
+include(FindPackageHandleStandardArgs)
+
+if(LIBSSH_LIBRARIES AND LIBSSH_INCLUDE_DIRS)
+    # in cache already
+    set(LIBSSH_FOUND TRUE)
+else()
+    find_path(LIBSSH_INCLUDE_DIR
+        NAMES
+        libssh/libssh.h
+        PATHS
+        /usr/include
+        /usr/local/include
+        /opt/local/include
+        /sw/include
+        ${CMAKE_INCLUDE_PATH}
+        ${CMAKE_INSTALL_PREFIX}/include
+    )
+
+    find_library(LIBSSH_LIBRARY
+        NAMES
+        ssh.so
+        libssh.so
+        libssh.dylib
+        PATHS
+        /usr/lib
+        /usr/local/lib
+        /opt/local/lib
+        /sw/lib
+        ${CMAKE_LIBRARY_PATH}
+        ${CMAKE_INSTALL_PREFIX}/lib
+    )
+
+    if(LIBSSH_INCLUDE_DIR AND LIBSSH_LIBRARY)
+        # learn libssh version
+        if(EXISTS ${LIBSSH_INCLUDE_DIR}/libssh/libssh_version.h)
+            set(LIBSSH_HEADER_PATH ${LIBSSH_INCLUDE_DIR}/libssh/libssh_version.h)
+        else()
+            set(LIBSSH_HEADER_PATH ${LIBSSH_INCLUDE_DIR}/libssh/libssh.h)
+        endif()
+        file(STRINGS ${LIBSSH_HEADER_PATH} LIBSSH_VERSION_MAJOR
+            REGEX "#define[ ]+LIBSSH_VERSION_MAJOR[ ]+[0-9]+")
+        if(NOT LIBSSH_VERSION_MAJOR)
+            message(STATUS "LIBSSH_VERSION_MAJOR not found, assuming libssh is too old and cannot be used!")
+            set(LIBSSH_INCLUDE_DIR "LIBSSH_INCLUDE_DIR-NOTFOUND")
+            set(LIBSSH_LIBRARY "LIBSSH_LIBRARY-NOTFOUND")
+        else()
+            string(REGEX MATCH "[0-9]+" LIBSSH_VERSION_MAJOR ${LIBSSH_VERSION_MAJOR})
+            file(STRINGS ${LIBSSH_HEADER_PATH} LIBSSH_VERSION_MINOR
+                REGEX "#define[ ]+LIBSSH_VERSION_MINOR[ ]+[0-9]+")
+            string(REGEX MATCH "[0-9]+" LIBSSH_VERSION_MINOR ${LIBSSH_VERSION_MINOR})
+            file(STRINGS ${LIBSSH_HEADER_PATH} LIBSSH_VERSION_PATCH
+                REGEX "#define[ ]+LIBSSH_VERSION_MICRO[ ]+[0-9]+")
+            string(REGEX MATCH "[0-9]+" LIBSSH_VERSION_PATCH ${LIBSSH_VERSION_PATCH})
+
+            set(LIBSSH_VERSION ${LIBSSH_VERSION_MAJOR}.${LIBSSH_VERSION_MINOR}.${LIBSSH_VERSION_PATCH})
+
+            if(LIBSSH_VERSION VERSION_LESS 0.8.0)
+                # libssh_threads also needs to be linked for these versions
+                string(REPLACE "libssh.so" "libssh_threads.so"
+                    LIBSSH_THREADS_LIBRARY
+                    ${LIBSSH_LIBRARY}
+                )
+                string(REPLACE "libssh.dylib" "libssh_threads.dylib"
+                    LIBSSH_THREADS_LIBRARY
+                    ${LIBSSH_THREADS_LIBRARY}
+                )
+                string(REPLACE "ssh.so" "ssh_threads.so"
+                    LIBSSH_THREADS_LIBRARY
+                    ${LIBSSH_THREADS_LIBRARY}
+                )
+            endif()
+        endif()
+    endif()
+
+    set(LIBSSH_INCLUDE_DIRS ${LIBSSH_INCLUDE_DIR})
+    set(LIBSSH_LIBRARIES ${LIBSSH_LIBRARY} ${LIBSSH_THREADS_LIBRARY})
+    mark_as_advanced(LIBSSH_INCLUDE_DIRS LIBSSH_LIBRARIES)
+
+    find_package_handle_standard_args(LibSSH FOUND_VAR LIBSSH_FOUND
+        REQUIRED_VARS LIBSSH_INCLUDE_DIRS LIBSSH_LIBRARIES
+        VERSION_VAR LIBSSH_VERSION)
+endif()

fieldImages/build.gradle

@@ -15,10 +15,14 @@ if (!project.hasProperty('onlylinuxathena') && !project.hasProperty('onlylinuxra
 
     ext {
         nativeName = 'fieldImages'
+        baseId = nativeName
+        groupId = 'edu.wpi.first.fieldImages'
+        devMain = "edu.wpi.first.fieldImages.DevMain"
     }
 
     apply from: "${rootDir}/shared/resources.gradle"
     apply from: "${rootDir}/shared/config.gradle"
+    apply from: "${rootDir}/shared/java/javacommon.gradle"
 
     def generateTask = createGenerateResourcesTask('main', 'FIELDS', 'fields', project)
 

fieldImages/publish.gradle

@@ -1,14 +1,14 @@
 apply plugin: 'maven-publish'
 
-def baseArtifactId = 'fieldImages'
-def artifactGroupId = 'edu.wpi.first.fieldImages'
-def zipBaseName = '_GROUP_edu_wpi_first_field_images_ID_CLS'
+def baseArtifactId = project.nativeName
+def artifactGroupId = project.groupId
+def cppZipBaseName = "_GROUP_edu_wpi_first_fieldIimages_ID_${baseArtifactId}-cpp_CLS"
 
 def outputsFolder = file("$project.buildDir/outputs")
 
 task cppSourcesZip(type: Zip) {
     destinationDirectory = outputsFolder
-    archiveBaseName = zipBaseName
+    archiveBaseName = cppZipBaseName
     classifier = "sources"
 
     from(licenseFile) {
@@ -25,7 +25,7 @@ task cppSourcesZip(type: Zip) {
 
 task cppHeadersZip(type: Zip) {
     destinationDirectory = outputsFolder
-    archiveBaseName = zipBaseName
+    archiveBaseName = cppZipBaseName
     classifier = "headers"
 
     from(licenseFile) {
@@ -51,7 +51,7 @@ addTaskToCopyAllOutputs(cppSourcesZip)
 
 model {
     publishing {
-        def wpilibCTaskList = createComponentZipTasks($.components, ['fieldImages'], zipBaseName, Zip, project, includeStandardZipFormat)
+        def wpilibCTaskList = createComponentZipTasks($.components, ['fieldImages'], cppZipBaseName, Zip, project, includeStandardZipFormat)
 
         publications {
             cpp(MavenPublication) {
@@ -62,7 +62,7 @@ model {
                 artifact cppHeadersZip
                 artifact cppSourcesZip
 
-                artifactId = baseArtifactId
+                artifactId = "${baseArtifactId}-cpp"
                 groupId artifactGroupId
                 version wpilibVersioning.version.get()
             }

fieldImages/src/main/java/edu/wpi/fields/FieldImages.java

@@ -20,4 +20,6 @@ public class FieldImages {
   public static final String k2021GalacticSearchBFieldConfig =
       "/edu/wpi/first/fields/2021-galacticsearchb.json";
   public static final String k2021SlalomFieldConfig = "/edu/wpi/first/fields/2021-slalompath.json";
+  public static final String k2022RapidReactFieldConfig =
+      "/edu/wpi/first/fields/2022-rapidreact.json";
 }

fieldImages/src/main/native/include/fields/2022-rapidreact.h

@@ -0,0 +1,12 @@
+// Copyright (c) FIRST and other WPILib contributors.
+// Open Source Software; you can modify and/or share it under the terms of
+// the WPILib BSD license file in the root directory of this project.
+
+#pragma once
+
+#include <string_view>
+
+namespace fields {
+std::string_view GetResource_2022_rapidreact_json();
+std::string_view GetResource_2022_field_png();
+}  // namespace fields

fieldImages/src/main/native/resources/edu/wpi/first/fields/2022-rapidreact.json

@@ -0,0 +1,10 @@
+{
+    "game": "Rapid React",
+    "field-image": "2022-field.png",
+    "field-corners": {
+      "top-left": [74, 50],
+      "bottom-right": [1774, 900]
+    },
+    "field-size": [54, 27],
+    "field-unit": "foot"
+  }

glass/publish.gradle

@@ -130,6 +130,14 @@ model {
                             }
 
                             from(applicationPath)
+
+                            if (binary.targetPlatform.operatingSystem.isWindows()) {
+                                def exePath = binary.executable.file.absolutePath
+                                exePath = exePath.substring(0, exePath.length() - 4)
+                                def pdbPath = new File(exePath + '.pdb')
+                                from(pdbPath)
+                            }
+
                             into(nativeUtils.getPlatformPath(binary))
                         }
 

glass/src/libnt/native/include/glass/networktables/NTSpeedController.h

@@ -16,7 +16,7 @@
 namespace glass {
 class NTSpeedControllerModel : public SpeedControllerModel {
  public:
-  static constexpr const char* kType = "Speed Controller";
+  static constexpr const char* kType = "Motor Controller";
 
   explicit NTSpeedControllerModel(std::string_view path);
   NTSpeedControllerModel(NT_Inst instance, std::string_view path);

gradle/wrapper/gradle-wrapper.properties

@@ -1,5 +1,5 @@
 distributionBase=GRADLE_USER_HOME
 distributionPath=wrapper/dists
-distributionUrl=https\://services.gradle.org/distributions/gradle-7.3.2-bin.zip
+distributionUrl=https\://services.gradle.org/distributions/gradle-7.3.3-bin.zip
 zipStoreBase=GRADLE_USER_HOME
 zipStorePath=wrapper/dists

gradlew

@@ -1,7 +1,7 @@
 #!/bin/sh
 
 #
-# Copyright © 2015-2021 the original authors.
+# Copyright © 2015-2021 the original authors.
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
@@ -32,10 +32,10 @@
 #       Busybox and similar reduced shells will NOT work, because this script
 #       requires all of these POSIX shell features:
 #         * functions;
-#         * expansions «$var», «${var}», «${var:-default}», «${var+SET}»,
-#           «${var#prefix}», «${var%suffix}», and «$( cmd )»;
-#         * compound commands having a testable exit status, especially «case»;
-#         * various built-in commands including «command», «set», and «ulimit».
+#         * expansions «$var», «${var}», «${var:-default}», «${var+SET}»,
+#           «${var#prefix}», «${var%suffix}», and «$( cmd )»;
+#         * compound commands having a testable exit status, especially «case»;
+#         * various built-in commands including «command», «set», and «ulimit».
 #
 #   Important for patching:
 #

hal/src/main/native/athena/CTREPCM.cpp

@@ -335,8 +335,13 @@ HAL_Bool HAL_GetCTREPCMSolenoidVoltageFault(HAL_CTREPCMHandle handle,
 
 void HAL_ClearAllCTREPCMStickyFaults(HAL_CTREPCMHandle handle,
                                      int32_t* status) {
+  auto pcm = pcmHandles->Get(handle);
+  if (pcm == nullptr) {
+    *status = HAL_HANDLE_ERROR;
+    return;
+  }
   uint8_t controlData[] = {0, 0, 0, 0x80};
-  HAL_WriteCANPacket(handle, controlData, sizeof(controlData), Control2,
+  HAL_WriteCANPacket(pcm->canHandle, controlData, sizeof(controlData), Control2,
                      status);
 }
 
@@ -393,7 +398,7 @@ void HAL_SetCTREPCMOneShotDuration(HAL_CTREPCMHandle handle, int32_t index,
       (std::min)(static_cast<uint32_t>(durMs) / 10,
                  static_cast<uint32_t>(0xFF));
   HAL_WriteCANPacketRepeating(pcm->canHandle, pcm->oneShot.sol10MsPerUnit, 8,
-                              Control2, SendPeriod, status);
+                              Control3, SendPeriod, status);
 }
 
 }  // extern "C"

hal/src/main/native/athena/FRCDriverStation.cpp

@@ -110,10 +110,15 @@ static int32_t HAL_GetControlWordInternal(HAL_ControlWord* controlWord) {
 
 static int32_t HAL_GetMatchInfoInternal(HAL_MatchInfo* info) {
   MatchType_t matchType = MatchType_t::kMatchType_none;
+  info->gameSpecificMessageSize = sizeof(info->gameSpecificMessage);
   int status = FRC_NetworkCommunication_getMatchInfo(
       info->eventName, &matchType, &info->matchNumber, &info->replayNumber,
       info->gameSpecificMessage, &info->gameSpecificMessageSize);
 
+  if (info->gameSpecificMessageSize > sizeof(info->gameSpecificMessage)) {
+    info->gameSpecificMessageSize = 0;
+  }
+
   info->matchType = static_cast<HAL_MatchType>(matchType);
 
   *(std::end(info->eventName) - 1) = '\0';

myRobot/build.gradle

@@ -86,6 +86,9 @@ deploy {
 
                 myRobotCpp(NativeExecutableArtifact) {
                     libraryDirectory = '/usr/local/frc/third-party/lib'
+                    def excludes = getLibraryFilter().getExcludes()
+                    excludes.add('**/*.so.debug')
+                    excludes.add('**/*.so.*.debug')
                     postdeploy << { ctx ->
                         ctx.execute('chmod +x myRobotCpp')
                     }

outlineviewer/publish.gradle

@@ -76,6 +76,14 @@ model {
                             }
 
                             from(applicationPath)
+
+                            if (binary.targetPlatform.operatingSystem.isWindows()) {
+                                def exePath = binary.executable.file.absolutePath
+                                exePath = exePath.substring(0, exePath.length() - 4)
+                                def pdbPath = new File(exePath + '.pdb')
+                                from(pdbPath)
+                            }
+
                             into(nativeUtils.getPlatformPath(binary))
                         }
 

roborioteamnumbersetter/CMakeLists.txt

@@ -0,0 +1,29 @@
+project(roborioteamnumbersetter)
+
+include(CompileWarnings)
+include(GenResources)
+include(LinkMacOSGUI)
+
+configure_file(src/main/generate/WPILibVersion.cpp.in WPILibVersion.cpp)
+GENERATE_RESOURCES(src/main/native/resources generated/main/cpp RTNS rtns rtns_resources_src)
+
+file(GLOB rtns_src src/main/native/cpp/*.cpp ${CMAKE_CURRENT_BINARY_DIR}/WPILibVersion.cpp)
+
+if (WIN32)
+    set(rtns_rc src/main/native/win/roborioteamnumbersetter.rc)
+elseif(APPLE)
+    set(MACOSX_BUNDLE_ICON_FILE rtns.icns)
+    set(APP_ICON_MACOSX src/main/native/mac/rtns.icns)
+    set_source_files_properties(${APP_ICON_MACOSX} PROPERTIES MACOSX_PACKAGE_LOCATION "Resources")
+endif()
+
+add_executable(roborioteamnumbersetter ${rtns_src} ${rtns_resources_src} ${rtns_rc} ${APP_ICON_MACOSX})
+wpilib_link_macos_gui(roborioteamnumbersetter)
+target_link_libraries(roborioteamnumbersetter libglass ${LIBSSH_LIBRARIES})
+target_include_directories(roborioteamnumbersetter PRIVATE ${LIBSSH_INCLUDE_DIRS})
+
+if (WIN32)
+    set_target_properties(roborioteamnumbersetter PROPERTIES WIN32_EXECUTABLE YES)
+elseif(APPLE)
+    set_target_properties(roborioteamnumbersetter PROPERTIES MACOSX_BUNDLE YES OUTPUT_NAME "roborioTeamNumberSetter")
+endif()

roborioteamnumbersetter/publish.gradle

@@ -76,6 +76,14 @@ model {
                             }
 
                             from(applicationPath)
+
+                            if (binary.targetPlatform.operatingSystem.isWindows()) {
+                                def exePath = binary.executable.file.absolutePath
+                                exePath = exePath.substring(0, exePath.length() - 4)
+                                def pdbPath = new File(exePath + '.pdb')
+                                from(pdbPath)
+                            }
+
                             into(nativeUtils.getPlatformPath(binary))
                         }
 

roborioteamnumbersetter/src/main/native/cpp/App.cpp

@@ -31,6 +31,16 @@ namespace gui = wpi::gui;
 
 const char* GetWPILibVersion();
 
+namespace rtns {
+std::string_view GetResource_rtns_16_png();
+std::string_view GetResource_rtns_32_png();
+std::string_view GetResource_rtns_48_png();
+std::string_view GetResource_rtns_64_png();
+std::string_view GetResource_rtns_128_png();
+std::string_view GetResource_rtns_256_png();
+std::string_view GetResource_rtns_512_png();
+}  // namespace rtns
+
 #define GLFWAPI extern "C"
 GLFWAPI void glfwGetWindowSize(GLFWwindow* window, int* width, int* height);
 #define GLFW_DONT_CARE -1
@@ -240,6 +250,15 @@ void Application(std::string_view saveDir) {
   gui::CreateContext();
   glass::CreateContext();
 
+  // Add icons
+  gui::AddIcon(rtns::GetResource_rtns_16_png());
+  gui::AddIcon(rtns::GetResource_rtns_32_png());
+  gui::AddIcon(rtns::GetResource_rtns_48_png());
+  gui::AddIcon(rtns::GetResource_rtns_64_png());
+  gui::AddIcon(rtns::GetResource_rtns_128_png());
+  gui::AddIcon(rtns::GetResource_rtns_256_png());
+  gui::AddIcon(rtns::GetResource_rtns_512_png());
+
   glass::SetStorageName("roborioteamnumbersetter");
   glass::SetStorageDir(saveDir.empty() ? gui::GetPlatformSaveFileDir()
                                        : saveDir);

shared/config.gradle

@@ -10,7 +10,7 @@ nativeUtils {
     wpi {
         configureDependencies {
             wpiVersion = "-1"
-            niLibVersion = "2022.2.3"
+            niLibVersion = "2022.4.0"
             opencvVersion = "4.5.2-1"
             googleTestVersion = "1.9.0-5-437e100-1"
             imguiVersion = "1.86-1"
@@ -21,7 +21,10 @@ nativeUtils {
 
 nativeUtils.wpi.addWarnings()
 nativeUtils.wpi.addWarningsAsErrors()
-nativeUtils.wpi.addReleaseSymbolGeneration()
+
+if (project.name != 'wpilibcExamples') {
+    nativeUtils.wpi.addReleaseSymbolGeneration()
+}
 
 nativeUtils.setSinglePrintPerPlatform()
 nativeUtils.enableSourceLink()

upstream_utils/fmt-dont-throw-on-write-failure.patch

@@ -0,0 +1,42 @@
+diff --git a/wpiutil/src/main/native/fmtlib/include/fmt/format-inl.h b/wpiutil/src/main/native/fmtlib/include/fmt/format-inl.h
+index 2c51c50ae..cc89abdd3 100644
+--- a/wpiutil/src/main/native/fmtlib/include/fmt/format-inl.h
++++ b/wpiutil/src/main/native/fmtlib/include/fmt/format-inl.h
+@@ -92,8 +92,7 @@ FMT_FUNC void report_error(format_func func, int error_code,
+ // A wrapper around fwrite that throws on error.
+ inline void fwrite_fully(const void* ptr, size_t size, size_t count,
+                          FILE* stream) {
+-  size_t written = std::fwrite(ptr, size, count, stream);
+-  if (written < count) FMT_THROW(system_error(errno, "cannot write to file"));
++  std::fwrite(ptr, size, count, stream);
+ }
+ 
+ #ifndef FMT_STATIC_THOUSANDS_SEPARATOR
+diff --git a/wpiutil/src/main/native/fmtlib/include/fmt/xchar.h b/wpiutil/src/main/native/fmtlib/include/fmt/xchar.h
+index 55825077f..9acb893fa 100644
+--- a/wpiutil/src/main/native/fmtlib/include/fmt/xchar.h
++++ b/wpiutil/src/main/native/fmtlib/include/fmt/xchar.h
+@@ -207,8 +207,7 @@ inline void vprint(std::FILE* f, wstring_view fmt, wformat_args args) {
+   wmemory_buffer buffer;
+   detail::vformat_to(buffer, fmt, args);
+   buffer.push_back(L'\0');
+-  if (std::fputws(buffer.data(), f) == -1)
+-    FMT_THROW(system_error(errno, FMT_STRING("cannot write to file")));
++  std::fputws(buffer.data(), f);
+ }
+ 
+ inline void vprint(wstring_view fmt, wformat_args args) {
+diff --git a/wpiutil/src/main/native/fmtlib/src/os.cpp b/wpiutil/src/main/native/fmtlib/src/os.cpp
+index 04b4dc506..4eb3e1fdd 100644
+--- a/wpiutil/src/main/native/fmtlib/src/os.cpp
++++ b/wpiutil/src/main/native/fmtlib/src/os.cpp
+@@ -277,8 +277,7 @@ std::size_t file::read(void* buffer, std::size_t count) {
+ std::size_t file::write(const void* buffer, std::size_t count) {
+   rwresult result = 0;
+   FMT_RETRY(result, FMT_POSIX_CALL(write(fd_, buffer, convert_rwcount(count))));
+-  if (result < 0) FMT_THROW(system_error(errno, "cannot write to file"));
+-  return detail::to_unsigned(result);
++  return count;
+ }
+ 
+ file file::dup(int fd) {

upstream_utils/update_fmt.py

@@ -3,11 +3,11 @@
 import os
 import shutil
 
-from upstream_utils import setup_upstream_repo, comment_out_invalid_includes, walk_cwd_and_copy_if
+from upstream_utils import setup_upstream_repo, comment_out_invalid_includes, walk_cwd_and_copy_if, apply_patches
 
 
 def main():
-    root, repo = setup_upstream_repo("https://github.com/fmtlib/fmt", "8.0.1")
+    root, repo = setup_upstream_repo("https://github.com/fmtlib/fmt", "8.1.1")
     wpiutil = os.path.join(root, "wpiutil")
 
     # Delete old install
@@ -31,6 +31,9 @@ def main():
         comment_out_invalid_includes(
             f, [os.path.join(wpiutil, "src/main/native/fmtlib/include")])
 
+    apply_patches(root,
+                  ["upstream_utils/fmt-dont-throw-on-write-failure.patch"])
+
 
 if __name__ == "__main__":
     main()

wpilibNewCommands/WPILibNewCommands.json

@@ -1,7 +1,7 @@
 {
   "fileName": "WPILibNewCommands.json",
   "name": "WPILib-New-Commands",
-  "version": "2020.0.0",
+  "version": "1.0.0",
   "uuid": "111e20f7-815e-48f8-9dd6-e675ce75b266",
   "mavenUrls": [],
   "jsonUrl": "",

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/Command.java

@@ -14,6 +14,8 @@ import java.util.function.BooleanSupplier;
  *
  * <p>Commands are run synchronously from the main robot loop; no multithreading is used, unless
  * specified explicitly from the command implementation.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
  */
 public interface Command {
   /** The initial subroutine of a command. Called once when the command is initially scheduled. */

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/CommandBase.java

@@ -10,7 +10,11 @@ import edu.wpi.first.util.sendable.SendableRegistry;
 import java.util.HashSet;
 import java.util.Set;
 
-/** A {@link Sendable} base class for {@link Command}s. */
+/**
+ * A {@link Sendable} base class for {@link Command}s.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
+ */
 public abstract class CommandBase implements Sendable, Command {
   protected Set<Subsystem> m_requirements = new HashSet<>();
 

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/CommandGroupBase.java

@@ -13,6 +13,8 @@ import java.util.WeakHashMap;
  * A base for CommandGroups. Statically tracks commands that have been allocated to groups to ensure
  * those commands are not also used independently, which can result in inconsistent command state
  * and unpredictable execution.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
  */
 public abstract class CommandGroupBase extends CommandBase {
   private static final Set<Command> m_groupedCommands =

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/CommandScheduler.java

@@ -33,6 +33,8 @@ import java.util.function.Consumer;
  * synchronously from the main loop. Subsystems should be registered with the scheduler using {@link
  * CommandScheduler#registerSubsystem(Subsystem...)} in order for their {@link Subsystem#periodic()}
  * methods to be called and for their default commands to be scheduled.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
  */
 public final class CommandScheduler implements NTSendable, AutoCloseable {
   /** The Singleton Instance. */

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/CommandState.java

@@ -8,6 +8,8 @@ import edu.wpi.first.wpilibj.Timer;
 
 /**
  * Class that holds scheduling state for a command. Used internally by the {@link CommandScheduler}.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
  */
 class CommandState {
   // The time since this command was initialized.

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/ConditionalCommand.java

@@ -21,6 +21,8 @@ import java.util.function.BooleanSupplier;
  * scheduled individually.
  *
  * <p>As a rule, CommandGroups require the union of the requirements of their component commands.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
  */
 public class ConditionalCommand extends CommandBase {
   private final Command m_onTrue;

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/FunctionalCommand.java

@@ -14,6 +14,8 @@ import java.util.function.Consumer;
  * the constructor. Useful for inline definitions of complex commands - note, however, that if a
  * command is beyond a certain complexity it is usually better practice to write a proper class for
  * it than to inline it.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
  */
 public class FunctionalCommand extends CommandBase {
   protected final Runnable m_onInit;

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/InstantCommand.java

@@ -10,6 +10,8 @@ import static edu.wpi.first.wpilibj.util.ErrorMessages.requireNonNullParam;
  * A Command that runs instantly; it will initialize, execute once, and end on the same iteration of
  * the scheduler. Users can either pass in a Runnable and a set of requirements, or else subclass
  * this command if desired.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
  */
 public class InstantCommand extends CommandBase {
   private final Runnable m_toRun;

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/MecanumControllerCommand.java

@@ -35,6 +35,8 @@ import java.util.function.Supplier;
  *
  * <p>The robot angle controller does not follow the angle given by the trajectory but rather goes
  * to the angle given in the final state of the trajectory.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
  */
 @SuppressWarnings("MemberName")
 public class MecanumControllerCommand extends CommandBase {

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/NotifierCommand.java

@@ -14,6 +14,8 @@ import edu.wpi.first.wpilibj.Notifier;
  * <p>WARNING: Do not use this class unless you are confident in your ability to make the executed
  * code thread-safe. If you do not know what "thread-safe" means, that is a good sign that you
  * should not use this class.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
  */
 public class NotifierCommand extends CommandBase {
   protected final Notifier m_notifier;

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/PIDCommand.java

@@ -15,6 +15,8 @@ import java.util.function.DoubleSupplier;
  * A command that controls an output with a {@link PIDController}. Runs forever by default - to add
  * exit conditions and/or other behavior, subclass this class. The controller calculation and output
  * are performed synchronously in the command's execute() method.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
  */
 public class PIDCommand extends CommandBase {
   protected final PIDController m_controller;

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/PIDSubsystem.java

@@ -11,6 +11,8 @@ import edu.wpi.first.math.controller.PIDController;
 /**
  * A subsystem that uses a {@link PIDController} to control an output. The controller is run
  * synchronously from the subsystem's periodic() method.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
  */
 public abstract class PIDSubsystem extends SubsystemBase {
   protected final PIDController m_controller;

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/ParallelCommandGroup.java

@@ -12,6 +12,8 @@ import java.util.Map;
  * A CommandGroup that runs a set of commands in parallel, ending when the last command ends.
  *
  * <p>As a rule, CommandGroups require the union of the requirements of their component commands.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
  */
 public class ParallelCommandGroup extends CommandGroupBase {
   // maps commands in this group to whether they are still running

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/ParallelDeadlineGroup.java

@@ -13,6 +13,8 @@ import java.util.Map;
  * "deadline") ends, interrupting all other commands that are still running at that point.
  *
  * <p>As a rule, CommandGroups require the union of the requirements of their component commands.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
  */
 public class ParallelDeadlineGroup extends CommandGroupBase {
   // maps commands in this group to whether they are still running

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/ParallelRaceGroup.java

@@ -13,6 +13,8 @@ import java.util.Set;
  * and interrupting all the others.
  *
  * <p>As a rule, CommandGroups require the union of the requirements of their component commands.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
  */
 public class ParallelRaceGroup extends CommandGroupBase {
   private final Set<Command> m_commands = new HashSet<>();

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/PerpetualCommand.java

@@ -14,6 +14,8 @@ import static edu.wpi.first.wpilibj2.command.CommandGroupBase.requireUngrouped;
  * cannot be added to any other groups, or scheduled individually.
  *
  * <p>As a rule, CommandGroups require the union of the requirements of their component commands.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
  */
 public class PerpetualCommand extends CommandBase {
   protected final Command m_command;

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/PrintCommand.java

@@ -4,7 +4,11 @@
 
 package edu.wpi.first.wpilibj2.command;
 
-/** A command that prints a string when initialized. */
+/**
+ * A command that prints a string when initialized.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
+ */
 public class PrintCommand extends InstantCommand {
   /**
    * Creates a new a PrintCommand.

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/ProfiledPIDCommand.java

@@ -17,6 +17,8 @@ import java.util.function.Supplier;
  * A command that controls an output with a {@link ProfiledPIDController}. Runs forever by default -
  * to add exit conditions and/or other behavior, subclass this class. The controller calculation and
  * output are performed synchronously in the command's execute() method.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
  */
 public class ProfiledPIDCommand extends CommandBase {
   protected final ProfiledPIDController m_controller;

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/ProfiledPIDSubsystem.java

@@ -13,6 +13,8 @@ import edu.wpi.first.math.trajectory.TrapezoidProfile;
 /**
  * A subsystem that uses a {@link ProfiledPIDController} to control an output. The controller is run
  * synchronously from the subsystem's periodic() method.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
  */
 public abstract class ProfiledPIDSubsystem extends SubsystemBase {
   protected final ProfiledPIDController m_controller;

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/ProxyScheduleCommand.java

@@ -10,6 +10,8 @@ import java.util.Set;
  * Schedules the given commands when this command is initialized, and ends when all the commands are
  * no longer scheduled. Useful for forking off from CommandGroups. If this command is interrupted,
  * it will cancel all of the commands.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
  */
 public class ProxyScheduleCommand extends CommandBase {
   private final Set<Command> m_toSchedule;

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/RamseteCommand.java

@@ -29,6 +29,8 @@ import java.util.function.Supplier;
  * <p>Advanced teams seeking more flexibility (for example, those who wish to use the onboard PID
  * functionality of a "smart" motor controller) may use the secondary constructor that omits the PID
  * and feedforward functionality, returning only the raw wheel speeds from the RAMSETE controller.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
  */
 public class RamseteCommand extends CommandBase {
   private final Timer m_timer = new Timer();

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/RunCommand.java

@@ -12,6 +12,8 @@ import java.util.function.BooleanSupplier;
  * A command that runs a Runnable continuously. Has no end condition as-is; either subclass it or
  * use {@link Command#withTimeout(double)} or {@link Command#withInterrupt(BooleanSupplier)} to give
  * it one. If you only wish to execute a Runnable once, use {@link InstantCommand}.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
  */
 public class RunCommand extends CommandBase {
   protected final Runnable m_toRun;

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/ScheduleCommand.java

@@ -10,6 +10,8 @@ import java.util.Set;
  * Schedules the given commands when this command is initialized. Useful for forking off from
  * CommandGroups. Note that if run from a CommandGroup, the group will not know about the status of
  * the scheduled commands, and will treat this command as finishing instantly.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
  */
 public class ScheduleCommand extends CommandBase {
   private final Set<Command> m_toSchedule;

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/SelectCommand.java

@@ -23,6 +23,8 @@ import java.util.function.Supplier;
  * scheduled individually.
  *
  * <p>As a rule, CommandGroups require the union of the requirements of their component commands.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
  */
 public class SelectCommand extends CommandBase {
   private final Map<Object, Command> m_commands;

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/SequentialCommandGroup.java

@@ -11,6 +11,8 @@ import java.util.List;
  * A CommandGroups that runs a list of commands in sequence.
  *
  * <p>As a rule, CommandGroups require the union of the requirements of their component commands.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
  */
 public class SequentialCommandGroup extends CommandGroupBase {
   private final List<Command> m_commands = new ArrayList<>();

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/StartEndCommand.java

@@ -11,6 +11,8 @@ import static edu.wpi.first.wpilibj.util.ErrorMessages.requireNonNullParam;
  * Useful for running and then stopping a motor, or extending and then retracting a solenoid. Has no
  * end condition as-is; either subclass it or use {@link Command#withTimeout(double)} or {@link
  * Command#withInterrupt(java.util.function.BooleanSupplier)} to give it one.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
  */
 public class StartEndCommand extends CommandBase {
   protected final Runnable m_onInit;

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/Subsystem.java

@@ -18,6 +18,8 @@ package edu.wpi.first.wpilibj2.command;
  * Subsystem#periodic()} method to be called. It is recommended that this method be called from the
  * constructor of users' Subsystem implementations. The {@link SubsystemBase} class offers a simple
  * base for user implementations that handles this.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
  */
 public interface Subsystem {
   /**

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/SubsystemBase.java

@@ -11,6 +11,8 @@ import edu.wpi.first.util.sendable.SendableRegistry;
 /**
  * A base for subsystems that handles registration in the constructor, and provides a more intuitive
  * method for setting the default command.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
  */
 public abstract class SubsystemBase implements Subsystem, Sendable {
   /** Constructor. */

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/SwerveControllerCommand.java

@@ -28,6 +28,8 @@ import java.util.function.Supplier;
  *
  * <p>The robot angle controller does not follow the angle given by the trajectory but rather goes
  * to the angle given in the final state of the trajectory.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
  */
 @SuppressWarnings("MemberName")
 public class SwerveControllerCommand extends CommandBase {

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/TrapezoidProfileCommand.java

@@ -13,6 +13,8 @@ import java.util.function.Consumer;
 
 /**
  * A command that runs a {@link TrapezoidProfile}. Useful for smoothly controlling mechanism motion.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
  */
 public class TrapezoidProfileCommand extends CommandBase {
   private final TrapezoidProfile m_profile;

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/TrapezoidProfileSubsystem.java

@@ -11,6 +11,8 @@ import edu.wpi.first.math.trajectory.TrapezoidProfile;
 /**
  * A subsystem that generates and runs trapezoidal motion profiles automatically. The user specifies
  * how to use the current state of the motion profile by overriding the `useState` method.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
  */
 public abstract class TrapezoidProfileSubsystem extends SubsystemBase {
   private final double m_period;

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/WaitCommand.java

@@ -10,6 +10,8 @@ import edu.wpi.first.wpilibj.Timer;
 /**
  * A command that does nothing but takes a specified amount of time to finish. Useful for
  * CommandGroups. Can also be subclassed to make a command with an internal timer.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
  */
 public class WaitCommand extends CommandBase {
   protected Timer m_timer = new Timer();

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/WaitUntilCommand.java

@@ -12,6 +12,8 @@ import java.util.function.BooleanSupplier;
 /**
  * A command that does nothing but ends after a specified match time or condition. Useful for
  * CommandGroups.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
  */
 public class WaitUntilCommand extends CommandBase {
   private final BooleanSupplier m_condition;

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/button/Button.java

@@ -17,6 +17,8 @@ import java.util.function.BooleanSupplier;
  * <p>This class represents a subclass of Trigger that is specifically aimed at buttons on an
  * operator interface as a common use case of the more generalized Trigger objects. This is a simple
  * wrapper around Trigger with the method names renamed to fit the Button object use.
+ *
+ * <p>This class is provided by the OldCommands VendorDep
  */
 public class Button extends Trigger {
   /**

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/button/InternalButton.java

@@ -7,6 +7,8 @@ package edu.wpi.first.wpilibj2.command.button;
 /**
  * This class is intended to be used within a program. The programmer can manually set its value.
  * Also includes a setting for whether or not it should invert its value.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
  */
 public class InternalButton extends Button {
   private boolean m_pressed;

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/button/JoystickButton.java

@@ -8,7 +8,11 @@ import static edu.wpi.first.wpilibj.util.ErrorMessages.requireNonNullParam;
 
 import edu.wpi.first.wpilibj.GenericHID;
 
-/** A {@link Button} that gets its state from a {@link GenericHID}. */
+/**
+ * A {@link Button} that gets its state from a {@link GenericHID}.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
+ */
 public class JoystickButton extends Button {
   private final GenericHID m_joystick;
   private final int m_buttonNumber;

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/button/NetworkButton.java

@@ -8,7 +8,11 @@ import edu.wpi.first.networktables.NetworkTable;
 import edu.wpi.first.networktables.NetworkTableEntry;
 import edu.wpi.first.networktables.NetworkTableInstance;
 
-/** A {@link Button} that uses a {@link NetworkTable} boolean field. */
+/**
+ * A {@link Button} that uses a {@link NetworkTable} boolean field.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
+ */
 public class NetworkButton extends Button {
   private final NetworkTableEntry m_entry;
 

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/button/POVButton.java

@@ -8,7 +8,11 @@ import static edu.wpi.first.wpilibj.util.ErrorMessages.requireNonNullParam;
 
 import edu.wpi.first.wpilibj.GenericHID;
 
-/** A {@link Button} that gets its state from a POV on a {@link GenericHID}. */
+/**
+ * A {@link Button} that gets its state from a POV on a {@link GenericHID}.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
+ */
 public class POVButton extends Button {
   private final GenericHID m_joystick;
   private final int m_angle;

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/button/Trigger.java

@@ -23,6 +23,8 @@ import java.util.function.BooleanSupplier;
  * (for instance, if they want to react to the user holding a button while the robot is reading a
  * certain sensor input). For this, they only have to write the {@link Trigger#get()} method to get
  * the full functionality of the Trigger class.
+ *
+ * <p>This class is provided by the NewCommands VendorDep
  */
 public class Trigger implements BooleanSupplier {
   private final BooleanSupplier m_isActive;

wpilibNewCommands/src/main/native/include/frc2/command/Command.h

@@ -42,6 +42,8 @@ class ProxyScheduleCommand;
  * <p>Note: ALWAYS create a subclass by extending CommandHelper<Base, Subclass>,
  * or decorators will not function!
  *
+ * This class is provided by the NewCommands VendorDep
+ *
  * @see CommandScheduler
  * @see CommandHelper
  */

wpilibNewCommands/src/main/native/include/frc2/command/CommandBase.h

@@ -18,6 +18,8 @@
 namespace frc2 {
 /**
  * A Sendable base class for Commands.
+ *
+ * This class is provided by the NewCommands VendorDep
  */
 class CommandBase : public Command,
                     public wpi::Sendable,

wpilibNewCommands/src/main/native/include/frc2/command/CommandGroupBase.h

@@ -18,6 +18,8 @@ namespace frc2 {
  * A base for CommandGroups.  Statically tracks commands that have been
  * allocated to groups to ensure those commands are not also used independently,
  * which can result in inconsistent command state and unpredictable execution.
+ *
+ * This class is provided by the NewCommands VendorDep
  */
 class CommandGroupBase : public CommandBase {
  public:

wpilibNewCommands/src/main/native/include/frc2/command/CommandHelper.h

@@ -17,6 +17,8 @@ namespace frc2 {
  *
  * <p>Note: ALWAYS create a subclass by extending CommandHelper<Base, Subclass>,
  * or decorators will not function!
+ *
+ * This class is provided by the NewCommands VendorDep
  */
 template <typename Base, typename CRTP,
           typename = std::enable_if_t<std::is_base_of_v<Command, Base>>>

wpilibNewCommands/src/main/native/include/frc2/command/CommandScheduler.h

@@ -26,6 +26,8 @@ class Subsystem;
  * commands synchronously from the main loop.  Subsystems should be registered
  * with the scheduler using RegisterSubsystem() in order for their Periodic()
  * methods to be called and for their default commands to be scheduled.
+ *
+ * This class is provided by the NewCommands VendorDep
  */
 class CommandScheduler final : public nt::NTSendable,
                                public wpi::SendableHelper<CommandScheduler> {

wpilibNewCommands/src/main/native/include/frc2/command/CommandState.h

@@ -10,6 +10,8 @@ namespace frc2 {
 /**
  * Class that holds scheduling state for a command.  Used internally by the
  * CommandScheduler
+ *
+ * This class is provided by the NewCommands VendorDep
  */
 class CommandState final {
  public:

wpilibNewCommands/src/main/native/include/frc2/command/ConditionalCommand.h

@@ -29,6 +29,8 @@ namespace frc2 {
  * <p>As a rule, CommandGroups require the union of the requirements of their
  * component commands.
  *
+ * This class is provided by the NewCommands VendorDep
+ *
  * @see ScheduleCommand
  */
 class ConditionalCommand

wpilibNewCommands/src/main/native/include/frc2/command/FunctionalCommand.h

@@ -19,6 +19,8 @@ namespace frc2 {
  * complex commands - note, however, that if a command is beyond a certain
  * complexity it is usually better practice to write a proper class for it than
  * to inline it.
+ *
+ * This class is provided by the NewCommands VendorDep
  */
 class FunctionalCommand : public CommandHelper<CommandBase, FunctionalCommand> {
  public:

wpilibNewCommands/src/main/native/include/frc2/command/InstantCommand.h

@@ -17,6 +17,8 @@ namespace frc2 {
  * A Command that runs instantly; it will initialize, execute once, and end on
  * the same iteration of the scheduler.  Users can either pass in a Runnable and
  * a set of requirements, or else subclass this command if desired.
+ *
+ * This class is provided by the NewCommands VendorDep
  */
 class InstantCommand : public CommandHelper<CommandBase, InstantCommand> {
  public:

wpilibNewCommands/src/main/native/include/frc2/command/MecanumControllerCommand.h

@@ -47,6 +47,8 @@ namespace frc2 {
  * <p>The robot angle controller does not follow the angle given by
  * the trajectory but rather goes to the angle given in the final state of the
  * trajectory.
+ *
+ * This class is provided by the NewCommands VendorDep
  */
 class MecanumControllerCommand
     : public CommandHelper<CommandBase, MecanumControllerCommand> {

wpilibNewCommands/src/main/native/include/frc2/command/NotifierCommand.h

@@ -24,6 +24,8 @@ namespace frc2 {
  * <p>WARNING: Do not use this class unless you are confident in your ability to
  * make the executed code thread-safe.  If you do not know what "thread-safe"
  * means, that is a good sign that you should not use this class.
+ *
+ * This class is provided by the NewCommands VendorDep
  */
 class NotifierCommand : public CommandHelper<CommandBase, NotifierCommand> {
  public:

wpilibNewCommands/src/main/native/include/frc2/command/PIDCommand.h

@@ -20,6 +20,8 @@ namespace frc2 {
  * The controller calculation and output are performed synchronously in the
  * command's execute() method.
  *
+ * This class is provided by the NewCommands VendorDep
+ *
  * @see PIDController
  */
 class PIDCommand : public CommandHelper<CommandBase, PIDCommand> {

wpilibNewCommands/src/main/native/include/frc2/command/PIDSubsystem.h

@@ -13,6 +13,8 @@ namespace frc2 {
  * A subsystem that uses a PIDController to control an output.  The controller
  * is run synchronously from the subsystem's periodic() method.
  *
+ * This class is provided by the NewCommands VendorDep
+ *
  * @see PIDController
  */
 class PIDSubsystem : public SubsystemBase {

wpilibNewCommands/src/main/native/include/frc2/command/ParallelCommandGroup.h

@@ -23,6 +23,8 @@ namespace frc2 {
  *
  * <p>As a rule, CommandGroups require the union of the requirements of their
  * component commands.
+ *
+ * This class is provided by the NewCommands VendorDep
  */
 class ParallelCommandGroup
     : public CommandHelper<CommandGroupBase, ParallelCommandGroup> {

wpilibNewCommands/src/main/native/include/frc2/command/ParallelDeadlineGroup.h

@@ -24,6 +24,8 @@ namespace frc2 {
  *
  * <p>As a rule, CommandGroups require the union of the requirements of their
  * component commands.
+ *
+ * This class is provided by the NewCommands VendorDep
  */
 class ParallelDeadlineGroup
     : public CommandHelper<CommandGroupBase, ParallelDeadlineGroup> {

wpilibNewCommands/src/main/native/include/frc2/command/ParallelRaceGroup.h

@@ -23,6 +23,8 @@ namespace frc2 {
  *
  * <p>As a rule, CommandGroups require the union of the requirements of their
  * component commands.
+ *
+ * This class is provided by the NewCommands VendorDep
  */
 class ParallelRaceGroup
     : public CommandHelper<CommandGroupBase, ParallelRaceGroup> {

wpilibNewCommands/src/main/native/include/frc2/command/PerpetualCommand.h

@@ -26,6 +26,8 @@ namespace frc2 {
  *
  * <p>As a rule, CommandGroups require the union of the requirements of their
  * component commands.
+ *
+ * This class is provided by the NewCommands VendorDep
  */
 class PerpetualCommand : public CommandHelper<CommandBase, PerpetualCommand> {
  public:

wpilibNewCommands/src/main/native/include/frc2/command/PrintCommand.h

@@ -12,6 +12,8 @@
 namespace frc2 {
 /**
  * A command that prints a string when initialized.
+ *
+ * This class is provided by the NewCommands VendorDep
  */
 class PrintCommand : public CommandHelper<InstantCommand, PrintCommand> {
  public:

wpilibNewCommands/src/main/native/include/frc2/command/ProfiledPIDCommand.h

@@ -22,6 +22,8 @@ namespace frc2 {
  * class. The controller calculation and output are performed synchronously in
  * the command's execute() method.
  *
+ * This class is provided by the NewCommands VendorDep
+ *
  * @see ProfiledPIDController<Distance>
  */
 template <class Distance>
@@ -95,7 +97,7 @@ class ProfiledPIDCommand
                      std::initializer_list<Subsystem*> requirements)
       : ProfiledPIDCommand(
             controller, measurementSource,
-            [&goalSource]() {
+            [goalSource = std::move(goalSource)]() {
               return State{goalSource(), Velocity_t{0}};
             },
             useOutput, requirements) {}
@@ -117,7 +119,7 @@ class ProfiledPIDCommand
                      wpi::span<Subsystem* const> requirements = {})
       : ProfiledPIDCommand(
             controller, measurementSource,
-            [&goalSource]() {
+            [goalSource = std::move(goalSource)]() {
               return State{goalSource(), Velocity_t{0}};
             },
             useOutput, requirements) {}

wpilibNewCommands/src/main/native/include/frc2/command/ProfiledPIDSubsystem.h

@@ -14,6 +14,8 @@ namespace frc2 {
  * A subsystem that uses a ProfiledPIDController to control an output.  The
  * controller is run synchronously from the subsystem's periodic() method.
  *
+ * This class is provided by the NewCommands VendorDep
+ *
  * @see ProfiledPIDController
  */
 template <class Distance>

wpilibNewCommands/src/main/native/include/frc2/command/ProxyScheduleCommand.h

@@ -17,6 +17,8 @@ namespace frc2 {
  * all the commands are no longer scheduled.  Useful for forking off from
  * CommandGroups.  If this command is interrupted, it will cancel all of the
  * commands.
+ *
+ * This class is provided by the NewCommands VendorDep
  */
 class ProxyScheduleCommand
     : public CommandHelper<CommandBase, ProxyScheduleCommand> {

wpilibNewCommands/src/main/native/include/frc2/command/RamseteCommand.h

@@ -37,6 +37,8 @@ namespace frc2 {
  * secondary constructor that omits the PID and feedforward functionality,
  * returning only the raw wheel speeds from the RAMSETE controller.
  *
+ * This class is provided by the NewCommands VendorDep
+ *
  * @see RamseteController
  * @see Trajectory
  */

wpilibNewCommands/src/main/native/include/frc2/command/RunCommand.h

@@ -18,6 +18,8 @@ namespace frc2 {
  * either subclass it or use Command.WithTimeout() or
  * Command.WithInterrupt() to give it one.  If you only wish
  * to execute a Runnable once, use InstantCommand.
+ *
+ * This class is provided by the NewCommands VendorDep
  */
 class RunCommand : public CommandHelper<CommandBase, RunCommand> {
  public:

wpilibNewCommands/src/main/native/include/frc2/command/ScheduleCommand.h

@@ -17,6 +17,8 @@ namespace frc2 {
  * forking off from CommandGroups.  Note that if run from a CommandGroup, the
  * group will not know about the status of the scheduled commands, and will
  * treat this command as finishing instantly.
+ *
+ * This class is provided by the NewCommands VendorDep
  */
 class ScheduleCommand : public CommandHelper<CommandBase, ScheduleCommand> {
  public:

wpilibNewCommands/src/main/native/include/frc2/command/SelectCommand.h

@@ -35,6 +35,8 @@ namespace frc2 {
  *
  * <p>As a rule, CommandGroups require the union of the requirements of their
  * component commands.
+ *
+ * This class is provided by the NewCommands VendorDep
  */
 template <typename Key>
 class SelectCommand : public CommandHelper<CommandBase, SelectCommand<Key>> {

wpilibNewCommands/src/main/native/include/frc2/command/SequentialCommandGroup.h

@@ -29,6 +29,8 @@ const size_t invalid_index = std::numeric_limits<size_t>::max();
  *
  * <p>As a rule, CommandGroups require the union of the requirements of their
  * component commands.
+ *
+ * This class is provided by the NewCommands VendorDep
  */
 class SequentialCommandGroup
     : public CommandHelper<CommandGroupBase, SequentialCommandGroup> {

wpilibNewCommands/src/main/native/include/frc2/command/StartEndCommand.h

@@ -19,6 +19,8 @@ namespace frc2 {
  * extending and then retracting a solenoid. Has no end condition as-is; either
  * subclass it or use Command.WithTimeout() or Command.WithInterrupt() to give
  * it one.
+ *
+ * This class is provided by the NewCommands VendorDep
  */
 class StartEndCommand : public CommandHelper<CommandBase, StartEndCommand> {
  public:

wpilibNewCommands/src/main/native/include/frc2/command/Subsystem.h

@@ -29,6 +29,8 @@ class Command;
  * SubsystemBase class offers a simple base for user implementations
  * that handles this.
  *
+ * This class is provided by the NewCommands VendorDep
+ *
  * @see Command
  * @see CommandScheduler
  * @see SubsystemBase

wpilibNewCommands/src/main/native/include/frc2/command/SubsystemBase.h

@@ -16,6 +16,8 @@ namespace frc2 {
 /**
  * A base for subsystems that handles registration in the constructor, and
  * provides a more intuitive method for setting the default command.
+ *
+ * This class is provided by the NewCommands VendorDep
  */
 class SubsystemBase : public Subsystem,
                       public wpi::Sendable,

wpilibNewCommands/src/main/native/include/frc2/command/SwerveControllerCommand.h

@@ -46,6 +46,8 @@ namespace frc2 {
  * <p>The robot angle controller does not follow the angle given by
  * the trajectory but rather goes to the angle given in the final state of the
  * trajectory.
+ *
+ * This class is provided by the NewCommands VendorDep
  */
 template <size_t NumModules>
 class SwerveControllerCommand

wpilibNewCommands/src/main/native/include/frc2/command/TrapezoidProfileCommand.h

@@ -19,6 +19,8 @@ namespace frc2 {
  * A command that runs a TrapezoidProfile.  Useful for smoothly controlling
  * mechanism motion.
  *
+ * This class is provided by the NewCommands VendorDep
+ *
  * @see TrapezoidProfile
  */
 template <class Distance>

wpilibNewCommands/src/main/native/include/frc2/command/TrapezoidProfileSubsystem.h

@@ -14,6 +14,8 @@ namespace frc2 {
  * A subsystem that generates and runs trapezoidal motion profiles
  * automatically.  The user specifies how to use the current state of the motion
  * profile by overriding the `UseState` method.
+ *
+ * This class is provided by the NewCommands VendorDep
  */
 template <class Distance>
 class TrapezoidProfileSubsystem : public SubsystemBase {

wpilibNewCommands/src/main/native/include/frc2/command/WaitCommand.h

@@ -15,6 +15,8 @@ namespace frc2 {
  * A command that does nothing but takes a specified amount of time to finish.
  * Useful for CommandGroups.  Can also be subclassed to make a command with an
  * internal timer.
+ *
+ * This class is provided by the NewCommands VendorDep
  */
 class WaitCommand : public CommandHelper<CommandBase, WaitCommand> {
  public:

wpilibNewCommands/src/main/native/include/frc2/command/WaitUntilCommand.h

@@ -15,6 +15,8 @@ namespace frc2 {
 /**
  * A command that does nothing but ends after a specified match time or
  * condition.  Useful for CommandGroups.
+ *
+ * This class is provided by the NewCommands VendorDep
  */
 class WaitUntilCommand : public CommandHelper<CommandBase, WaitUntilCommand> {
  public:

wpilibNewCommands/src/main/native/include/frc2/command/button/Button.h

@@ -18,6 +18,8 @@ class Command;
  * A class used to bind command scheduling to button presses.  Can be composed
  * with other buttons with the operators in Trigger.
  *
+ * This class is provided by the NewCommands VendorDep
+ *
  * @see Trigger
  */
 class Button : public Trigger {

wpilibNewCommands/src/main/native/include/frc2/command/button/JoystickButton.h

@@ -12,6 +12,8 @@ namespace frc2 {
  * A class used to bind command scheduling to joystick button presses.  Can be
  * composed with other buttons with the operators in Trigger.
  *
+ * This class is provided by the NewCommands VendorDep
+ *
  * @see Trigger
  */
 class JoystickButton : public Button {

wpilibNewCommands/src/main/native/include/frc2/command/button/NetworkButton.h

@@ -15,6 +15,8 @@
 namespace frc2 {
 /**
  * A Button that uses a NetworkTable boolean field.
+ *
+ * This class is provided by the NewCommands VendorDep
  */
 class NetworkButton : public Button {
  public: