search

ponty / PyVirtualDisplay

Python wrapper for Xvfb, Xephyr and Xvnc
BSD 2-Clause "Simplified" License
709 stars 78 forks source link

Don't look for xauth unless use_xauth=True #23

Closed mbrunthaler closed 7 years ago

mbrunthaler commented 7 years ago

Fixes an issue where xauth installation is checked even if we don't need it.

coveralls commented 7 years ago

Coverage Status

Coverage decreased (-8.8%) to 84.722% when pulling b828dfa81f6db7430f4286d338f07f8d2016be49 on mbrunthaler:patch-1 into be778a94d73bedf02c8757dd43256ff9f26c9089 on ponty:master.

The-Compiler commented 7 years ago

cc @Kingdread who originally implemented xauth

Kingdread commented 7 years ago

Looks like I made a mistake when implementing it and this PR fixes it. Checking if xauth makes no sense anyway, as xauth is the module that is imported anyway and the check will always pass.