PX4-Autopilot icon indicating copy to clipboard operation
PX4-Autopilot copied to clipboard

Decouple filepaths from rcS/MTD using Kconfig rootpath

Open PetervdPerk-NXP opened this issue 1 year ago • 3 comments

Removes hardcoded filepaths allows for use of filesystem and boardspecific filelocations.

Replaces #21891

PetervdPerk-NXP avatar Apr 10 '24 12:04 PetervdPerk-NXP

@PetervdPerk-NXP - Rebased on man and added the rest of the paths.

davids5 avatar Apr 11 '24 10:04 davids5

@PetervdPerk-NXP Shal we un-draft this?

davids5 avatar Apr 12 '24 06:04 davids5

I just tested the filepath changes on the CANHUBK3 FMU and the params are there, metadata seems to work better (don't need mavlink over ethernet if telem/uart connection is good).

Had to manually change BOARD_ROOT_PATH to '/fs/microsd' get logging to work, though

dirksavage88 avatar Apr 17 '24 21:04 dirksavage88

This pull request has been mentioned on Discussion Forum for PX4, Pixhawk, QGroundControl, MAVSDK, MAVLink. There might be relevant details there:

https://discuss.px4.io/t/px4-sync-q-a-july-24-2024/39852/1

DronecodeBot avatar Jul 24 '24 15:07 DronecodeBot

If I'm not mistaken then this PR broke some CI tests: image

sfuhrer avatar Jul 26 '24 15:07 sfuhrer

The module documentation CI also fails after this. Here's my suggestion to not trip the parser anymore: https://github.com/PX4/PX4-Autopilot/pull/23499

MaEtUgR avatar Aug 06 '24 08:08 MaEtUgR

Can we revert this until we fix the path issues with CI?

Jaeyoung-Lim avatar Aug 07 '24 13:08 Jaeyoung-Lim

@Jaeyoung-Lim can you rebase on https://github.com/PX4/PX4-Autopilot/pull/23457 that should've fixed it

PetervdPerk-NXP avatar Aug 07 '24 13:08 PetervdPerk-NXP