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

Desktop.browse() is not always supported #51

Closed
infeo opened this issue Nov 26, 2020 · 0 comments
Closed

Desktop.browse() is not always supported #51

infeo opened this issue Nov 26, 2020 · 0 comments
Labels
Milestone

Comments

@infeo
Copy link
Member

infeo commented Nov 26, 2020

In version 1.2.5 the reveal command for in the AbstractMount class was changed from java.awt.Desktop.open() to ...browse(), which is not supported on all systems. (e.g. Linux system with KDE):

public void revealInFileManager() throws CommandFailedException {
if (Desktop.isDesktopSupported() && Desktop.getDesktop().isSupported(Desktop.Action.BROWSE)) {
try {
Desktop.getDesktop().browse(mountPoint.toUri());
} catch (IOException e) {
throw new CommandFailedException(e);
}
} else {
throw new CommandFailedException("API to browse files not supported.");
}
}

In tests the old open() command still does the job and hence should be used again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant