Skip to content
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

[1pt] feedback: Module fio | Tarantool #1839

Closed
TarantoolBot opened this issue Feb 16, 2021 · 1 comment · Fixed by #2050
Closed

[1pt] feedback: Module fio | Tarantool #1839

TarantoolBot opened this issue Feb 16, 2021 · 1 comment · Fixed by #2050
Assignees
Labels
typo [nature] Just a typo. Typically tickets created by online bug reporter.

Comments

@TarantoolBot
Copy link
Collaborator

<…>rectories.
The result is similar to the rmdir -r shell command.
|The directory should not be empty.|

https://www.tarantool.io/en/doc/latest/reference/reference_lua/fio/#fio-rmtree

Why should it? Seems it's not true.

rosik@rlap:/tmp/tmp.MFOEXsE5TA$ mkdir tmp
rosik@rlap:/tmp/tmp.MFOEXsE5TA$ ls -al tmp
total 8
drwxrwxr-x 2 rosik rosik 4096 Feb 16 20:26 .
drwx------ 4 rosik rosik 4096 Feb 16 20:26 ..
rosik@rlap:/tmp/tmp.MFOEXsE5TA$ tarantool -l fio
Tarantool 2.8.0-8-g4108070c3
type 'help' for interactive help
tarantool> fio.rmtree('tmp')
---
- true
...

tarantool> ^C
rosik@rlap:/tmp/tmp.MFOEXsE5TA$ ls -al tmp
ls: cannot access 'tmp': No such file or directory

It was empty, but it's removed successfully. Also, there's no such thing as rmdir -r, but there is rm -rf.

@rosik

May be related to #1063
See also 034361f

@NickVolynkin
Copy link
Contributor

@rosik thanks!

@NickVolynkin NickVolynkin added the typo [nature] Just a typo. Typically tickets created by online bug reporter. label Feb 17, 2021
@NickVolynkin NickVolynkin changed the title feedback: Module fio | Tarantool [1pt] feedback: Module fio | Tarantool Mar 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
typo [nature] Just a typo. Typically tickets created by online bug reporter.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants