Open
Conversation
Previously it would refuse to load on IDA 8.0 and above, fixed that. Additionally, it will print a message warning that the plugin was built for IDA 7.5.
Maple38
changed the title
No idea how I managed to mess up this badly
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
Author
|
@joydo very sorry to bother, but this plugin is very cool and people still use it, but in its current state it's incredibly inconvenient. The plugin refuses to load on IDA versions < X.5, so if someone downloads it they have to either come here, read the pull requests, and modify the code, or figure it out themselves. If you could take the ~20 seconds to merge this pull request, that would be awesome for future users :) If you're reading this I'm assuming you've merged the request... so thank you, and have a great day (you deserve it). |
dungkkk7
pushed a commit
to dungkkk7/d810
that referenced
this pull request
Mar 2, 2026
- 🧭 Add repo-local speedups module: src/d810/speedups.py with get_speedups_dir(), ensure_speedups_on_path(), install_speedups(), and a main() entrypoint. It installs z3-solver into a private .d810-speedups directory (configurable via D810_SPEEDUPS_DIR) and prepends it to sys.path for isolation from IDA's DLLs. 🚀🔒 - 🧰 Add CLI helper: tools/install_speedups.py to trigger the speedups install flow via a simple command. handy for quick setup with a single command. 🧪🧹 - 📦 Update packaging: pyproject.toml - Move z3-solver out of core dependencies (main) and into the new optional-speedups dependency set. - Add a new project.scripts entry: install-speedups = "tools.install_speedups:main" for easy invocation. 🧭🪄 - Clarify speedups packaging as an optional feature so environments without speedups aren’t forced to install them. 🧩 - 📖 Documentation alignment: README.md updated to reflect the new repo-local speedups installation flow, including how to use the helper and where dependencies live. 📝✨ Why this matters: isolates optional dependencies (like z3-solver) to a repo-local folder to avoid conflicts with IDA Pro’s own runtime, provides a straightforward install path, and keeps core installs lean. 🧠💡 Fixes joydo#8
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
1 participant
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.
Previously it would refuse to load on IDA 8.0 and above, fixed that. Additionally, it will print a message warning that the plugin was built for IDA 7.5.