Fossil SCM

empty-dirs require every directory in the path to be on a separate line in order to work

Open

03df56ee7a0f8fa… · opened 12 years, 5 months ago

Type
Code_Defect
Priority
Severity
Minor
Resolution
Open
Subsystem
Created
Oct. 16, 2013 9:21 a.m.

If you commit a .fossil-settings\empty-dir file with, say, a path like this:


one\two\three

the fresh checkout will fail with a message:

"couldn't create directory one\two\three as required by empty-dirs setting"

if the directories "one" and "two" are not already existing.

However, if you provide three lines in .fossil-settings.empty-dir file:


one
one\two
one\two\three

in this order, it will always work regardless. This happens on Windows, if it matters.

This bug is of minor severity, but a major pain when maintaining deep directory structures that are to be filled in later (i.e. code scaffolding, project documentation & phases etc.). It looks like an easy fix as well.

Comments (4)

nobody 12 years, 5 months ago

If you commit a .fossil-settings\empty-dir file with, say, a path like this:


one\two\three

the fresh checkout will fail with a message:

"couldn't create directory one\two\three as required by empty-dirs setting"

if the directories "one" and "two" are not already existing.

However, if you provide three lines in .fossil-settings.empty-dir file:


one
one\two
one\two\three

in this order, it will always work regardless. This happens on Windows, if it matters.

This bug is of minor severity, but a major pain when maintaining deep directory structures that are to be filled in later (i.e. code scaffolding, project documentation & phases etc.). It looks like an easy fix as well.

jan.nijtmans 12 years, 5 months ago

Does it work when using forward slashes? I don't think that backward slashes should be permitted in the "empty-dirs" setting.

ss 12 years, 5 months ago

No, it doesn't matter - forward or backward slashes, same result.

This has been noticed by other people too, see this thread:

http://www.mail-archive.com/[email protected]/msg08038.html

jan.nijtmans 12 years, 5 months ago

Thanks! Then I agree that it would be good to fix that. Feel free to submit a patch, if you like.

Keyboard Shortcuts

Open search /
Next entry (timeline) j
Previous entry (timeline) k
Open focused entry Enter
Show this help ?
Toggle theme Top nav button