-
Notifications
You must be signed in to change notification settings - Fork 18.2k
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.
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
ROMFS: allow list over FTP #26225
ROMFS: allow list over FTP #26225
Conversation
needs checking with AP_Filesystem and MAVFtp |
Make sure folders can be listed by returning |
acbd2f3
to
cbd746b
Compare
This now allows full dir listing of ROMFS. It mostly works as any other file system. The mostly part is that we get a directory for each file. For example:
So we get 5 copy's of the To only show each directory once we would have to stash the last directory name somewhere and skip if there is no change. I'm not sure if its worth it.... |
we can do that in dir_list() by looking at the entry at ofs-1 |
@IamPete1 I've fixed it up here: |
cbd746b
to
3849595
Compare
Currently scripting will try and load any scripts in the
@ROMFS/scripts
directory or subdirectorys. This means it will also attempt to load any modules which will fail and cause a pre-arm. This is needed for #26157 to be useful.Tested in SITL with
ROMFS_custom
.