Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

MavlinkFTP: FYI #2216

Closed
olliw42 opened this issue Sep 4, 2019 · 3 comments
Closed

MavlinkFTP: FYI #2216

olliw42 opened this issue Sep 4, 2019 · 3 comments

Comments

@olliw42
Copy link

olliw42 commented Sep 4, 2019

FYI, in case you haven't seen, MAVLink FTP specifies now a NACK error for when a file/directory is not found: https://mavlink.io/en/services/ftp.html#error_codes

you may want to implement this, instead of the previous "workaround".

thx btw for the MAVLinkFTP feature, I'm lovin' it :)
@meee1
Copy link
Contributor

meee1 commented Sep 4, 2019

@bugobliterator looks like the error code enum has changed?

@olliw42
Copy link
Author

olliw42 commented Sep 5, 2019

only extended by one entry, not changed

@meee1
Copy link
Contributor

meee1 commented Sep 5, 2019

Ardupilot already has that extra as Busy, not 404

@olliw42 olliw42 closed this as completed Nov 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@meee1 @olliw42