mfs limitation ftp remote #4774
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Limitation consistent with rclone about remote: and mentioned in the rclone forum https://forum.rclone.org/t/rclone-union-mfs-most-free-space-not-working-for-ftp/20346
Modify to take account of @ncw comments 16/11/20 - more broadly consider impact of not supporting about, not just mfs in union
Remove some 'some' repetition and explain fuller consequences of lacking about capability
|
Thanks, that makes more sense. Modified ftp.md to make the more general points re 'about' capability. Had been foxed by the OP on https://forum.rclone.org/t/rclone-union-mfs-most-free-space-not-working-for-ftp/20346 suggesting rclone about worked for him (def doesn't on my vsftpd). Also reflected same changes in overview.md and removed repeated 'some'. |
|
That looks great now - will merge - thank you :-) |
3 tasks
edwardxml
added a commit
to edwardxml/rclone
that referenced
this pull request
Nov 25, 2020
Rclone about docs Purpose: Create a full loop of documentation for rclone about, backends overview and individual backend pages. Discussion: rclone#4774 relates Previously requested, in part, under ref rclone#4801 Notes: Introduce a tentative draft see-link format the end of sections to try rather than lots of in-para links. Update about.go incl link to list of backends not supporting about. In list of backends not supporting about, include link to about command reference. I appreciate there may be decisions to make going forward about whether command links should be code formatted, and using proper pretty url links, but I have fudged that for now. Update backend pages that do not support about with wording used previously for ftp - it is in passive voice but I can live with it. (my own wording and fault). The note is applied to a limitations section. If one does not already exist it is created (even if there are other limitations with their own sections)
edwardxml
added a commit
to edwardxml/rclone
that referenced
this pull request
Nov 26, 2020
Purpose: Create a full loop of documentation for rclone about, backends overview and individual backend pages. Discussion: rclone#4774 relates Previously pull was requested, in part, under ref rclone#4801 Notes: Introduce a tentative draft see-link format the end of sections to try rather than lots of in-para links. Update about.go incl link to list of backends not supporting about. In list of backends not supporting about, include link to about command reference. I appreciate there may be decisions to make going forward about whether command links should be code formatted, and using proper pretty url links, but I have fudged that for now. Update backend pages that do not support about with wording used previously for ftp - it is in passive voice but I can live with it. (my own wording and fault). The note is applied to a limitations section. If one does not already exist it is created (even if there are other limitations with their own sections)
ncw
pushed a commit
that referenced
this pull request
Nov 27, 2020
Create a full loop of documentation for rclone about, backends overview and individual backend pages. Discussion: #4774 relates Previously pull was requested, in part, under ref #4801 Notes: Introduce a tentative draft see-link format the end of sections to try rather than lots of in-para links. Update about.go incl link to list of backends not supporting about. In list of backends not supporting about, include link to about command reference. I appreciate there may be decisions to make going forward about whether command links should be code formatted, and using proper pretty url links, but I have fudged that for now. Update backend pages that do not support about with wording used previously for ftp - it is in passive voice but I can live with it. (my own wording and fault). The note is applied to a limitations section. If one does not already exist it is created (even if there are other limitations with their own sections)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Limitation consistent with
rclone about remote:and mentioned in the rclone forumWhat is the purpose of this change?
Reflect issue from forum in rclone documentation
Was the change discussed in an issue or in the forum before?
https://forum.rclone.org/t/rclone-union-mfs-most-free-space-not-working-for-ftp/20346
Checklist