Skip to content

fix(mavlink): remove stale mavlink_tests add_subdirectory from CMakeLists#26740

Merged
mrpollo merged 1 commit intomainfrom
fix/mavlink-cmakelists-stale-tests
Mar 13, 2026
Merged

fix(mavlink): remove stale mavlink_tests add_subdirectory from CMakeLists#26740
mrpollo merged 1 commit intomainfrom
fix/mavlink-cmakelists-stale-tests

Conversation

@mrpollo
Copy link
Copy Markdown
Contributor

@mrpollo mrpollo commented Mar 13, 2026

Summary

  • Remove stale add_subdirectory(mavlink_tests) from src/modules/mavlink/CMakeLists.txt
  • The mavlink_tests/ directory was deleted in 1009268 but this reference was not removed, breaking all SITL builds

Test plan

  • Verify SITL build completes (make px4_sitl_default)
  • CI passes (Static Analysis, Failsafe Simulator, SITL Tests)

Ref: #26738

@mrpollo mrpollo force-pushed the fix/mavlink-cmakelists-stale-tests branch 5 times, most recently from a8baacc to ffaf1bb Compare March 13, 2026 17:02
The mavlink_tests module was deleted in 1009268 but several
references were left behind, breaking builds on all targets.

Removed:
- CMakeLists.txt: add_subdirectory(mavlink_tests)
- mavlink_ftp.cpp: #include of deleted mavlink_ftp_test.h
- mavlink_ftp.h: MavlinkFtpTest forward decl and friend class
- posix-configs/SITL/init/test/test_mavlink: dead init script
- sitl_tests.cmake: sitl-mavlink CTest target
- install-voxl.sh: px4-mavlink_tests symlink

Ref: #26738
Signed-off-by: Ramon Roche <mrpollo@gmail.com>
@mrpollo mrpollo force-pushed the fix/mavlink-cmakelists-stale-tests branch from ffaf1bb to 099998d Compare March 13, 2026 17:09
@mrpollo mrpollo merged commit 7388431 into main Mar 13, 2026
76 of 78 checks passed
@mrpollo mrpollo deleted the fix/mavlink-cmakelists-stale-tests branch March 13, 2026 17:49
@mrpollo
Copy link
Copy Markdown
Contributor Author

mrpollo commented Mar 13, 2026

Self-merging to fix main

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant