Skip to content

Conversation

iftahnaf
Copy link
Contributor

Update MAVLink.md - Remove Unused CMake Variable

Description

Remove CPM_mavlink_SOURCE variable from mavlink.md documentation as it is not used anywhere.

Test Steps

Checklist:

Related Issue

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@HTRamsey
Copy link
Collaborator

This is actually read by CPM, you can use the same variable format to override any of the dependencies
image

@iftahnaf
Copy link
Contributor Author

iftahnaf commented Jun 29, 2025

@HTRamsey Thanks for the quick response.
Apologies for missing it.
I've added a usage example, which may be helpful.

@HTRamsey HTRamsey changed the title docs(mavlink.md): remove CPM_mavlink_SOURCE variable docs(mavlink.md): add CPM_mavlink_SOURCE variable example Jun 30, 2025
@HTRamsey HTRamsey merged commit cac91bd into mavlink:master Jun 30, 2025
3 checks passed
@dakejahl
Copy link
Contributor

dakejahl commented Aug 4, 2025

This isn't working for me, am I missing something?

In file included from /home/jake/code/jake/qgroundcontrol/src/AutoPilotPlugins/APM/APMCameraComponent.cc:11:
/home/jake/code/jake/qgroundcontrol/src/Vehicle/Vehicle.h:639:37: error: ‘MAV_CMD’ has not been declared
# MAVLink
# set(QGC_MAVLINK_GIT_REPO "https://github.com/mavlink/c_library_v2.git" CACHE STRING "URL to MAVLink Git Repo")
# set(QGC_MAVLINK_GIT_TAG "19f9955598af9a9181064619bd2e3c04bd2d848a" CACHE STRING "Tag of MAVLink Git Repo")
set(CPM_mavlink_SOURCE "/home/jake/code/jake/PX4-Autopilot/build/ark_fpv_default/mavlink")
image

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.

3 participants