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

Improvements #8

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Improvements #8

wants to merge 4 commits into from

Conversation

Iceburgino
Copy link

  1. Change events from add and remove to bind and unbind - the never ones that happen a little bit later than add and sooner than remove, so that the device can be properly initialized.

  2. Add default behavior, if no vm domain is specified. In this case, the script will pick the longest-running vm.

  3. Add monitor script, that allows for finding ports quickly. The author is darkguy2008. I modified it a bit, so that it searches for what's necessary only.

  4. Add hotplug-setup script that copies usb-libvirt-hotplug to bin directory, finds usb ports and adds udev rules automatically.

1. Change events from add and remove to bind and unbind - the never ones that happen a little bit later than add and sooner than remove, so that the device can be properly initialized.

2. Add default behavior, if no vm domain is specified. In this case, the script will pick the longest running vm.

3. Add monitor script, that allows for finding ports quickly. The author is darkguy2008. I modified it a bit, so that it searches for what's necessary only.

4. Add hotplug-setup script that copies usb-libvirt-hotplug to bin directory, finds usb ports and adds udev rules automatically.
Minor bug fix
Minor bug fix.

- Added option to remove portlist file even if automatic udev rule addition is not selected.

- The script now removes usb.portlist file before finishing if it is empty.

- Now user can skip both questions with ENTER, and they will be "y" by default.
Fix a bug where if statement fails on some lines.
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.

1 participant