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

Change from PySide to PyQT #11

Open
rafaelscheel opened this issue Jun 27, 2016 · 5 comments
Open

Change from PySide to PyQT #11

rafaelscheel opened this issue Jun 27, 2016 · 5 comments

Comments

@rafaelscheel
Copy link

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

@rafaelscheel
Copy link
Author

Fixes work for IDA on 64Bit Windows (there for IDA 32Bit and 64Bit)... Apperantly not in a 32-Bit Windows VM :/

@aaronportnoy
Copy link
Owner

@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?

@rafaelscheel
Copy link
Author

@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.

@rafaelscheel
Copy link
Author

@aaronportnoy just made the pull request. Regarding the crash, i uploaded the ida dmp here:
https://drive.google.com/open?id=0ByQKtdCNEYS4dWFYMkFRVjdkVHM

Might not be too relevant, lets see if others have the crash too.

@kokanin
Copy link

kokanin commented Sep 25, 2016

puh-lease implement these changes, having multiple idas installed screws my workflow :D

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

No branches or pull requests

3 participants