Skip to content

Update folder-caseSensitiveFS.rst #925

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

Closed
wants to merge 1 commit into from
Closed

Conversation

camoz
Copy link
Contributor

@camoz camoz commented May 6, 2025

Add Android to list of systems that are case-insensitive.

(The last part of the sentence that was removed was kind of redundant.)

@calmh
Copy link
Member

calmh commented May 9, 2025

It's not really redundant as you may well run with case insensitive FAT on Linux etc.

@camoz
Copy link
Contributor Author

camoz commented May 9, 2025

Yeah, that's why I wrote "kind of". The sentence talks about case(in)sensitive systems and case(in)sensitive filesystems, which makes it more complicated. I only removed the second part of the sentence, because with it, it became unnecessarily long and a bit weird to read, IMO. Feel free to suggest anything better or just dismiss this PR. I just thought it would be nice to have Android included there, since it's a popular platform for syncthing.

Edit: If you assume case(in)sensitive systems == case(in)sensitive filesystems, then it is completely redundant though.

Add Android and iOS to list of operating systems that are
case-insensitive by default.
@calmh
Copy link
Member

calmh commented May 26, 2025

I don't feel that adding two more operating systems, with one only being tangentially support by Syncthing to begin with, really adds much clarity here; it mostly makes the sentence longer.

@calmh calmh closed this May 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants