-
-
Notifications
You must be signed in to change notification settings - Fork 628
fix: trash.cmd defaults to 'trash' on macos #2336
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
Conversation
--- Apply OS specific localisations to DEFAULT_OPTS | ||
local function localise_default_opts() | ||
if utils.is_macos then | ||
DEFAULT_OPTS.trash.cmd = "trash" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't that be gio trash
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is where I get confused. I was under the impression that gio would put the files in ~/.local/share/Trash/files
which is not recognised by macos.
Perhaps we should just update the doc, removing "Trash is unavailable" and noting the available homebrew options for macos users.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it might be time for me to use https://github.com/sickcodes/Docker-OSX
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here, we found that neither trash
nor gio
comes preinstalled on macOS so it's best to do what you said, which is to warn the users that D
is not available unless you install trash
from homebrew. We shouldn't use gio trash
on macOS as it moves the file to a different location than ~/.Trash
which is where the Bin icon is pointing to.
Updated to include the windows trash changes #2335
|
Hi, @alex-courtis, there's no such default trash command on Windows. For macOS/Linux, I also don't know if there is such a default command. I always install third party via nodejs or python. |
Thanks mate. Adding to the doc. |
sure thing |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
fixes #1366