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

Update to angr==9.2.77 #417

Merged
merged 29 commits into from
Feb 22, 2024
Merged

Conversation

ANogin
Copy link
Contributor

@ANogin ANogin commented Feb 9, 2024

One sentence summary of this PR (This should go in the CHANGELOG!)
Update to recent angr==9.2.77, which also necessitates Python >= 3.8 and capstone==5.0.0.post1

Link to Related Issue(s)
N/A

Please describe the changes in your request.

Anyone you think should look at this, specifically?
@EdwardLarson

Note:

@ANogin ANogin marked this pull request as ready for review February 18, 2024 03:24
@ANogin ANogin changed the title Update to latest angr Update to angr==9.2.77 Feb 18, 2024
Copy link
Contributor

@whyitfor whyitfor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See comments. @rbs-jacob, your feedback would be appreciated as well.

disassemblers/ofrak_angr/ofrak_angr_test/test_unpackers.py Outdated Show resolved Hide resolved
docs/getting-started.md Outdated Show resolved Hide resolved
@ANogin ANogin requested a review from whyitfor February 21, 2024 00:37
Copy link
Contributor

@whyitfor whyitfor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

docs/getting-started.md Outdated Show resolved Hide resolved
docs/environment-setup.md Outdated Show resolved Hide resolved
@whyitfor whyitfor merged commit 8fe5083 into redballoonsecurity:master Feb 22, 2024
5 checks passed
ANogin added a commit to ANogin/ofrak that referenced this pull request May 30, 2024
* Use python3.8 in docker images

* Require pytest<8.0

This is needed becase of pytest-dev/pytest#11890
TvoroG/pytest-lazy-fixture#65

* Update changelog

* Revert "Update changelog"

This reverts commit 500ee9b.

Making changes before having coffee :(

* Update to latest angr==9.2.89, which also necessitates Python >= 3.8 and capstone==5.0.0.post1

* Apply Edward's attempted fix to angr test failure

* Add a note on recommending Python 3.8

* Add a note on recommending Python 3.8

* Document the requirement of Python 3.8+

* Switch to angr 9.2.77

* `ofrak_core` also needs `pytest<8.0`

* ignore DataWord in test due to angr bug

* add another now missing block

* black linting

* Attempt to fix a capstone error

* Dropping the .altinstr_replacement section from the toolchain (redballoonsecurity#414)

* Dropping the .altinstr_replacement section from the toolchain

* Updated CHANGELOG

* Set the fallback font to monospace (redballoonsecurity#422)

* Set the fallback font to monospace

* Update CHANGELOG

* Display strings with numbers primarily as strings (redballoonsecurity#423)

* Display strings with numbers primarily as strings

* Update CHANGELOG

* Add typing support to ofrak_ghidra package (redballoonsecurity#421)

* Add typing to ofrak_ghidra package

* Add changelog

---------

Co-authored-by: Paul Noalhyt <paul@redballoonsecurity.com>

* Increase time limit on `test_comment_content`

* Fix a spurious "no current event loop" test error

* Explain 3.7 vs 3.8 better in the docs

* Cite specific versions of angr in comment

* Update docs/environment-setup.md

* Update docs/getting-started.md

---------

Co-authored-by: Edward Larson <edward@redballoonsecurity.com>
Co-authored-by: rbs-alexr <122491504+rbs-alexr@users.noreply.github.com>
Co-authored-by: Jacob Strieb <99368685+rbs-jacob@users.noreply.github.com>
Co-authored-by: Paul Noalhyt <paul.noalhyt@gmail.com>
Co-authored-by: Paul Noalhyt <paul@redballoonsecurity.com>
Co-authored-by: Wyatt <53830972+whyitfor@users.noreply.github.com>
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.

6 participants