search

ArduPilot / companion

Companion computer startup scripts and examples
GNU General Public License v3.0
104 stars 164 forks source link

Common/Ubuntu: fix dxx.service typo #57

Closed tajisoft closed 4 years ago

tajisoft commented 4 years ago

@rmackay9 I think this is a typo. I confirmed yesterday that the script will not start when the typo of t265.service in the image of 2020/7/3 is modified. It's a little strange.

rmackay9 commented 4 years ago

@tajisoft, ok thanks for the fix.. but.. actually the similar PR that I merged regarding the t265.service has actually cost me days of effort. The problem appears to be fixing this line causes the rc.local file to fail perhaps because it's asking which user we want to start the service using (Ubuntu or apsync).

So although this may be a proper fix we need to test first the impact.

Sorry, it's just that I've lost days because I've been unable to create a new APSync image and it's the previous change that stopped things from working.

rmackay9 commented 4 years ago

I've included this fix in my new PR which resolves the underlying startup issue. txs again

https://github.com/ArduPilot/companion/pull/58