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

Added ImageShow support for xdg-open #5897

Merged
merged 2 commits into from Dec 23, 2021
Merged

Added ImageShow support for xdg-open #5897

merged 2 commits into from Dec 23, 2021

Conversation

m-shinder
Copy link
Contributor

@m-shinder m-shinder commented Dec 20, 2021

xdg-open is reliable way to open image with preferred application.
It works on linux and BSD with any desktop environment.

Debugging process should be comfortable.

Synopsis

xdg-open { file | URL }

xdg-open { --help | --manual | --version }

Use 'man xdg-open' or 'xdg-open --manual' for additional info.
@radarhere radarhere changed the title Add: XDGViewer which uses xdg-open Added ImageShow support for xdg-open Dec 23, 2021
@hugovk
Copy link
Member

@hugovk hugovk commented Dec 23, 2021

This is registered first and so will be used first.

Is it better than the other viewers?

@radarhere
Copy link
Member

@radarhere radarhere commented Dec 23, 2021

It's not a viewer in the normal sense - it is instructing the system to open the file with whatever program it thinks is appropriate. I suppose the macOS equivalent is 'open'.

@hugovk hugovk merged commit 1510efb into python-pillow:main Dec 23, 2021
49 checks passed
radarhere added a commit to radarhere/Pillow that referenced this issue Dec 23, 2021
radarhere added a commit to radarhere/Pillow that referenced this issue Dec 23, 2021
radarhere added a commit to radarhere/Pillow that referenced this issue Dec 23, 2021
hugovk added a commit that referenced this issue Dec 24, 2021
@radarhere radarhere mentioned this pull request Feb 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants