-
Notifications
You must be signed in to change notification settings - Fork 69
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
Change from PySide to PyQT #11
Comments
Fixes work for IDA on 64Bit Windows (there for IDA 32Bit and 64Bit)... Apperantly not in a 32-Bit Windows VM :/ |
@rafaelscheel I haven't tested it on IDA with PyQt yet, thanks for taking a look. I'll try to free up some time in July to make sure it works alright. What's the crash you see? |
@aaronportnoy Yes I fixed it, so it runs now with PyQt (not as before PySide). I will fork the project on Friday and upload the changes (Its a no-brainer, but still many small changes). This should make it eazy to keep both versions with PySide and PyQt. Regarding the crash: I will document / upload the crashes as well on Friday. It works perfectly on x64 Machines (IDA x64 but also x32). It crashes on Exit on my x32 Win7 VM. I will upload the crash dump and so on as well latest on friday. |
@aaronportnoy just made the pull request. Regarding the crash, i uploaded the ida dmp here: Might not be too relevant, lets see if others have the crash too. |
puh-lease implement these changes, having multiple idas installed screws my workflow :D |
Hey
Apperantly IDA uses now PyQT and not PySide anymore (http://www.hexblog.com/?p=906).
I adjusted the code a bit, so that it works with PyQt now, but get an crash on exit (hehehe). Did anybody already successfully change from PySide to PyQt? Just to make sure we don't do the work twice :P
Cheers,
Rafael
The text was updated successfully, but these errors were encountered: