-
Notifications
You must be signed in to change notification settings - Fork 257
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
rm *.extension or file* deletes all files #10
Comments
and now I need to download all my music again... |
Linux shell works in the same way. I'm sorry for what happened to your music |
@codebam So if you execute |
I didn't understand the problem was that. I'm sorry, I'll solve this in the next release. Indeed, I'm very sorry for your music |
Yeah the problem was that anything prefixed with or at the end of with a * deleted all files in the current directory without warning. So another bug should be not to allow folder deletion without the -r flag, and only do it without prompt with -rf. Also, @licaon-kter not exactly sure what you were trying to say. I executed rm *.log which should have removed all the log files in the root of /sdcard, instead it deleted all files and folders inside /sdcard. No problem about the music @Andre1299, I can't really blame you since I was the one who ran the command knowing it might not be stable. |
@codebam That what I said, you try to remove a file and another file is deleted too. See AAA and BBB ;) |
It will be solved in next build |
Are you using different code than linux binary rm? |
@HassanHeydariNasab yes, I had to. you can find the whole source here: https://github.com/Andre1299/TUI-ConsoleLauncher/blob/master/app/src/main/java/ohi/andre/consolelauncher/commands/raw/rm.java |
in 4.3.b1 rm foo* doesn't work (File not found) but rm *bar works. |
actually, you can't do this. maybe in next versions. you are able to do only I need to use custom rm code to be able to use useful things like suggestions and custom directory positions |
What's your work different than passing arguments to rm linux binary? Do the suggestions,... here:
and do this: |
ok On Fri, Jun 10, 2016, 4:39 PM Hysyn Hejdyri Nysyb notifications@github.com
|
I think rewriting the rm command would be a bad idea, at that point you are sort of just rewriting the tools that are already there. The rm command has advanced options that aren't as simple as just removing a file. Why not pass the full typed string to the real rm command in the background and display its output by adding &> /data/local/tmp/tui-console_out to the end of the string you're passing? Just an idea, it's your app of course. |
No description provided.
The text was updated successfully, but these errors were encountered: