Kieran Sedgwick
Kieran Sedgwick
To reiterate, this implementation of `mv` doesn't allow the movement of multiple files in one command. If we want to include that it'll need one more revision.
@modeswitch The `mv` utility will automatically overwrite files during the move unless told not to. Do we want to keep this for our version?
This is a pretty good patch! @modeswitch Is this approach to `mv` what we want? I'm fairly sure `fs.rename` can do the majority of the work for us though I'm...
@modeswitch The `mv` utility can accept multiple source files to be moved to a directory. Do we want to support this?
The new version of this PR is #346
The [POSIX standard](http://pubs.opengroup.org/onlinepubs/009604599/utilities/mv.html) says that the `mv` utility relies on `rename()`, except for cross-filesystem moves. At the moment, this patch is doing the `rename()` work manually, but it's working. Should...
@modeswitch Here's the updated PR: https://github.com/filerjs/filer/pull/346 It's missing a test case, which shouldn't be too difficult to add. Other than that I believe it's ready for review.
@modeswitch Can you expand on this a little bit? I'm unclear on how a user would access these optional programs from their `Shell` instance.
@humphd So that second idea, which I'm _totally_ ignoring, would mean attaching the builtins to the FS instance and running them in the context of a Shell instance with `exec`?...
@modeswitch Woah! That's really cool. Okay, so I'll build a basic shell including the full FS API. Are there any other convenience functions you want other than `ls` and `cp`?