search

plasma-umass / browsix

Browsix is a Unix-like operating system for the browser.
Other
3.15k stars 181 forks source link

rm: fix '-r' flag #30

Closed rvolosatovs closed 8 years ago

rvolosatovs commented 8 years ago

The '-r' flag was implmented wrong, rewrote the function using recursion to loop over dir contents. The check for '.' and '..' is unneeded given the node.js specification of 'fs.readdir'

rm -rf / seem to be failing due to missing '.deletedFiles.log' after first iterations of the function.

Removed unnecessary comment in xargs

bpowers commented 8 years ago

This looks great, thanks! I worked around the .deletedFiles.log problem for now with a hack - I think it is a BrowserFS bug, but we're on an old version so I want to try to upgrade before reporting it upstream to John.