goggle / flatten Goto Github PK

In the file filesystem/tree.go there is a function create, which creates a file system tree in memory. At the moment it is completely inefficient. This code should be optimized.

~/gopath/src/github.com/goggle/flatten$ ls -CF
dest/        flatten/  LICENSE  osabstraction/  test/
filesystem/  flattn*   main.go  README.md

~/gopath/src/github.com/goggle/flatten$ ./flattn test/ dest/ -s
Could not simulate the process. The following error occured:
test is not a directory

~/gopath/src/github.com/goggle/flatten$ ./flattn /home/anestis/gopath/src/github.com/goggle/flatten/test/ /home/anestis/gopath/src/github.com/goggle/flatten/dest/ -s
/home/anestis/gopath/src/github.com/goggle/flatten/dest
├── 11
├── 22
└── 33

These two commands are the same and should have the same result, but when relative syntax is used, the program doesn't recognize the directories.

Add a section in the readme file, which describes how to install the tool by using the go tool chain.

I'm not happy with the default behavior of the command-line interface.

It's annoying to be asked to simulate the actions on the file system first, and since "only" the directory tree information can be lost and not the data itself, we should make if default to immediately perform the actions on the file system. This also should make the "--force" option obsolete. The "--simulate-only" option should be kept as it is.

Also, the "--include-source-files" option should be default behavior in my opinion. We could make a "--exclude-source-files" option instead.