On Mon, Mar 18, 2019 at 06:28:16PM +0100, Björn Kautler wrote:
> I think fast-import should refuse to import paths Git cannot handle
> properly later on, so that the migration fails early and the frontend
> that generates the fast-import stream can be fixed / configured to not
> include such invalid paths.
Yeah, that seems quite sensible to me[1].
If you (or anybody else) are interested in working on this, I suspect
the answer is to just sprinkle some calls to verify_path() in the right
spots. Probably in fast-import.c:file_change_m(), etc.
-Peff
[1] Stretching to think of a way this might backfire, I guess somebody
could be using Git as an intermediate format to then convert to
another system. But that seems terribly obscure, and at most I think
we should give that case an escape hatch to disable the check; it
should definitely be on by default.
