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

QT5 support #65

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

QT5 support #65

wants to merge 2 commits into from

Conversation

vyrus001
Copy link

moved QT support to QT5, tested on OSX 10.12.6 (all dependancies acquired via homebrew) and gentoo-hardened

@ChrisTruncer
Copy link
Contributor

Hi @citronneur. I'm interested in seeing if this pull request is something you might get added in? I use RDPY in EyeWitness, and thanks to @vyrus001 I was looking into moving to pyqt5, but I'd also need RDPY updated as well. Just wanted to see if this was something you would review. Thanks for the help!

@rhertzog
Copy link

@citronneur FWIW Qt4 is no longer supported upstream so PyQt4 is actively removed from the various linux distributions, it would be nice to have a version of rdpy based on Qt5 so that we can keep maintaining a Kali package for rdpy.

For better cross-referencing, I'm pointing out that RedSiege/EyeWitness#301 is the eyewitness pull request adding Qt5 support to that software but which depends on this pull request.

@rhertzog rhertzog mentioned this pull request Aug 27, 2019
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.

3 participants