Incorrect information in /documentation/files/files.md regarding changing the value of config.file.root

[kallistisoft]

Build environment: All Moddable SDK version: 4.9 Target device: Any

Description The documentation for overriding the default config.file.root value is incorrect. Currently in /documentation/files/files.md the section "Host File System Configuration" has this code fragment:

"defines": {
    "file":{
        "root": "#/myroot"
    }
}

The correct code fragment is:

"config": {
    "file":{
        "root": "/myroot"
    }
}

Errata Submitting as an issue to avoid sending a two line PR :)

[kallistisoft]

Additional question; in /modules/files/file/manifest.json

The default file root is /tmp/ for linux and wasm builds, whereas for windows its ./ and on macos it's /Users/Shared/

Is there a compelling rational for this discrepancy with linux? Seems like it should also be ./ for consistency and principle of least surprise?

[phoddie]

@kallistisoft – thanks for the reports. I'll update the Files document to correct that. The convention is to have the root end with a / – so you can write config.file.root + "foo.json", so it will be /myroot/.

The file root for desktop computers is a little bit arbitrary and, admittedly, inconsistent. But, I'm not sure what would be least surprising; ;) The idea is to have the file system be in some sort of local sandbox, since that better reflects the file system on an embedded microcontroller.