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 Desktop Shortcut feature - usage: #18

Closed
wants to merge 1 commit into from
Closed

Added Desktop Shortcut feature - usage: #18

wants to merge 1 commit into from

Conversation

sidewinder040
Copy link
Contributor

Desktop Shortcut feature

Hi Martin,
quickemu --shortcut --vm <VMNAME>.conf
.desktop added to user profile ~/.local/share/applications folder
The icon for virgil is used.
When the VM is --deleted, the .desktop file is removed.
Updated the README.md also.

TODO Proposal - Add parameters options to be added to .conf file:

  • SHORTCUT=1 # Add shortcut when vm is created
  • ICON= # User can add distro logo icon from web

This is my first Git Pull request, i hope I've submitted it correctly.

Regards,
Mark Crouch

quickemu --shortcut --vm <VMNAME>.conf
.desktop added to user profile ~/.local/share/applications folder
The icon for virgil is used.
When the VM is --deleted, the .desktop file is removed.

** Maybe TODO - In .conf file:
** SHORTCUT=1 # Add shortcut when vm is created
** ICON=<path to custom icon> # User can add distro logo icon from web
@@ -441,6 +465,10 @@ while [ $# -gt 0 ]; do
VM="${2}"
shift
shift;;
-shortcut|--shortcut)
SHORTCUT=1
echo "shortcut is" ${SHORTCUT}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Debug Code - Please delete

@AnErrupTion
Copy link

Seems a nice thing, gotta add this into my fork lel

Terminal=true
Exec=${LAUNCHER_DIR}/${LAUNCHER} --vm ${VM}
Name=${VMNAME}
Icon=/snap/qemu-virgil/74/meta/gui/icon.png
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Icon=/snap/qemu-virgil/74/meta/gui/icon.png
Please change to:
Icon=/snap/qemu-virgil/current/meta/gui/icon.png

@sidewinder040
Copy link
Contributor Author

I've had time to process the changes needed to tidy up, so I'll resubmit this feature pull request

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