Skip to content

Conversation

@leigh123linux
Copy link
Contributor

Fixes: #2708

@mtwebster mtwebster merged commit 8ca3d47 into linuxmint:master Apr 25, 2022
@eyalroz
Copy link

eyalroz commented Apr 25, 2022

So what exactly changes with this PR merged?

@leigh123linux
Copy link
Contributor Author

@eyalroz It adds support for kernel exfat.
Microsoft files systems can't handle some characters, see

#define FAT_FORBIDDEN_CHARACTERS "/:*?\"<>\\|"

@eyalroz
Copy link

eyalroz commented Apr 26, 2022

@leigh123linux : I know, I filed #2708 ; I mean, in what sense is support added? What happens now when you copy to/from exfat and some characters are unsupported?

@mtwebster
Copy link
Member

Any invalid characters are replaced with an underscore _.

@eyalroz
Copy link

eyalroz commented Apr 26, 2022

@mtwebster : Ah, ok, now I get it.

What if there's a clash with another file? e.g. hello_world.txt and hello:world.txt?

@mtwebster
Copy link
Member

It will prompt you to rename the second file. I'm not crazy about this - I'd prefer it add the normal (Copy) suffix to the name, but I imagine the actual frequency of this happening will be low. Plus the current behavior has been in place for many other file systems for a while.

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.

Feature request: Add native exfat character support.

3 participants