Page MenuHomePhabricator

Prevent the addition of files with names that aren't supported on Windows
Closed, DeclinedPublic

Description

A colon in a file name is forbidden on a windows machine.
When a repository contains one or more files with a colon in its name, cloning of that repository fails.

Please add a pre commit hook to avoid adding files with a colon in its name.
Maybe there are more letters to ignore. Thanks.


See Also:
T67139: Rename test files with colon to a proper name (invalid sign on windows machine)

Details

Reference
bz65140

Event Timeline

bzimport raised the priority of this task from to Low.Nov 22 2014, 3:16 AM
bzimport set Reference to bz65140.
bzimport added a subscriber: Unknown Object (MLST).

I wonder if this wouldn't be better implemented as a linter? I remember this was also a problem when there were files named like '*.wikimedia.org' that had something to do with the domains somewhere in operations repo.

The following characters are not allowed in Windows file names:

\ / : * ? " < > |

Note both kinds of slashes as directory separators.

The issue is affected CommonsMetadata (bug 65139). I am not sure it is worth triggering a job for every single patchset being sent just to catch it.

This could be something added to Arcanist when we switch to Phabricator for code review.

I am declining this one, mostly because I have no clue how we can enforce it on all repositories.