[debian] Temporary: Don't install xenmou driver#353
Merged
jean-edouard merged 1 commit intoOpenXT:masterfrom Jul 25, 2019
Merged
[debian] Temporary: Don't install xenmou driver#353jean-edouard merged 1 commit intoOpenXT:masterfrom
jean-edouard merged 1 commit intoOpenXT:masterfrom
Conversation
The xenmou driver causes issues on Debian and Ubuntu systems. Debian systems will boot without mouse functionality and Ubuntu systems won't make it to the login screen due to a dereferenced NULL pointer. Note that this is a temporary bandaid and should be fixed when possible. OXT-1608 Signed-off-by: Nicholas Tsirakis <tsirakisn@ainfosec.com>
jean-edouard
approved these changes
Jul 24, 2019
Member
jean-edouard
left a comment
jean-edouard
left a comment
There was a problem hiding this comment.
This is fine, there are now better solutions that xenmou out there (I even wrote one of them :))
One important thing to note though, is that this will break seamless mouse switching (switching between VMs on different monitors by just moving the mouse).
I'm OK with that too. With your PR, xenmou is still getting built, the package still even gets copied to the guest rootfs, just not installed by default. So if a user out there cares about seamless mouse switching, they can just apt install openx-xenmou-dkms (or whatever it's called).
Will merge later this week to give time for potential reviews.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The xenmou driver causes issues on Debian and Ubuntu
systems. Debian systems will boot without mouse
functionality and Ubuntu systems won't make it to the
login screen due to a dereferenced NULL pointer.
Note that this is a temporary bandaid and should be
fixed when possible.
OXT-1608
Signed-off-by: Nicholas Tsirakis tsirakisn@ainfosec.com