mavlink / qgc-dev-guide

QGroundControl Developers Guide
https://dev.qgroundcontrol.com/en/
Other
42 stars 51 forks source link

Build fails due to curl is not installed on Ubuntu. Please add a hint in point "1" to indicate that #159

Closed alanmehio closed 1 year ago

alanmehio commented 1 year ago

The build when executing make after this step "Build using the "hammer" (or "play") icons:" it result with an error missing header file "#include <airmap/qt/client.h>" as per this post https://discuss.px4.io/t/missing-header-files/22635 The "curl" needs to be installed on the machine since this file "QGCExternalLibs.pri" has a curl call which gets to download the airmap inside the libs. It would be good to add the following hint in step 1 stating to check the existing of "curl" installed otherwise "curl" has to exist to avoid the error during the make.

hamishwillee commented 1 year ago

Airmap is only present on a few platforms right, and optional even on those? Good to mention any dependencies, but this should live next to instructions for installing Airmap.

DonLakeFlyer commented 1 year ago

There aren't any instructions for installing airmap. It is supposed to install itself during the build.

hamishwillee commented 1 year ago

Thanks @DonLakeFlyer - so should this be a Ununtu dependency as suggested? Or is this optional, or a bug or what?

hamishwillee commented 1 year ago

Thanks @alanmehio - sorry for the delay, fixed in #162.

alanmehio commented 1 year ago

Hamish, Great. All the best to you and your team. // Alan

On Tue, 22 Nov 2022 at 23:07, Hamish Willee @.***> wrote:

Thanks @alanmehio https://github.com/alanmehio - sorry for the delay, fixed in #162 https://github.com/mavlink/qgc-dev-guide/pull/162.

— Reply to this email directly, view it on GitHub https://github.com/mavlink/qgc-dev-guide/issues/159#issuecomment-1324296500, or unsubscribe https://github.com/notifications/unsubscribe-auth/AALPJ5KMLFDW4ZMI3LZHFATWJU7ZHANCNFSM6AAAAAARQNJGMY . You are receiving this because you were mentioned.Message ID: @.***>