search

gmyoungblood-parc / docker-alpine-ardupilot

Docker container setup for ardupIlot on Alpine Linux
GNU Lesser General Public License v3.0
10 stars 7 forks source link

-A should have its own ENV option #4

Open pklapperich opened 5 years ago

pklapperich commented 5 years ago

sim_vehicle.py's -A option can have spaces in it (eg: -A '--uartD=uart:/dev/ttyUSB0 --uartE=uart:/dev/ttyUSB1' connects physical serial ports to telem2 and serial4 respectively; useful for testing real devices). I've tried all manner of escaping quotes and spaces and whenever I try to send a -A option that includes spaces, sim_vehicle.py complains with things like '--uartE is not an option'

I think this would be fixable by splitting out -A into its own ENV so that we'd have SPEEDUP, SIM_OPTIONS and SITL_ARGS

It does not appear that anything bad happens if -A is provided an empty string.

gmyoungblood-parc commented 5 years ago

I haven't been maintaining this repo, but I will take a look at this issue this Friday and perhaps update the ArduPilot version as well. Thanks!