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

DSView is now the default app for .dsl files and added to 'Open With' #753

Open
wants to merge 2 commits into
base: dev-1.3.2
Choose a base branch
from

Conversation

kasra3422
Copy link

Issue:
In Ubuntu and likely other Linux distributions, DSView is not listed in the 'Open With' applications. Solution:

  1. Update 'Exec=DSView' to 'Exec=DSView %F' in the DSView.desktop file to add the application to the 'Open With' list.
  2. Add 'MimeType=applications/dsl' to the DSView.desktop file to set the application as the default for opening .dsl files.

@tedy58
Copy link

tedy58 commented Jun 8, 2024

Thanks @kasra3422 for the help hint!
one more switch could be use instead of %F (%U) which do the same.
I am using Debian 12 and desktop file is located normally in
/usr/share/applications/dsview.desktop and must be edited with super user privileges
with the following contect:

[Desktop Entry]
Type=Application
Encoding=UTF-8
Name=DSView
Comment=GUI Program for DreamSourceLab USB-based Instruments
TryExec=DSView
Exec=DSView %U
Icon=dsview
Terminal=false
Categories=Development;Electronics;Qt;
MimeType=applications/dsl

@kasra3422
Copy link
Author

Thanks, @tedy58. Yes, I checked the available switches and updated the PR. The %U switch supports both files and URLs.

%F: A list of files. Use for apps that can open several local files at once. Each file is passed as a separate argument to the executable program.

%U: A list of URLs. Each URL is passed as a separate argument to the executable program. Local files may either be passed as file: URLs or as file path.

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

Successfully merging this pull request may close these issues.

2 participants