Skip to content

Update rizin to latest dev - #3671

Open
PremadeS wants to merge 12 commits into
rizinorg:devfrom
PremadeS:update-rizin-dev
Open

PremadeS wants to merge 12 commits into
rizinorg:devfrom
PremadeS:update-rizin-dev

Conversation

@PremadeS

Copy link
Copy Markdown
Collaborator

Your checklist for this pull request

  • I've read the guidelines for contributing to this repository
  • I made sure to follow the project's coding style
  • I've updated the documentation with the relevant information (if needed)
  • I've used AI tools to generate fully or partially these code changes and I'm sure the changes are not copyrighted by somebody else.

Detailed description

Updates rizin to latest dev branch
Major changes in rizin include:

  • Removing global RzCons instance
  • Refactoring RzIterator

Required for: #3670 and #3654

Test plan (required)

CI is green

Closing issues

Comment thread dist/bundle_rz_frida.ps1

if (-not (Test-Path -Path 'rz-frida\meson.build' -PathType Leaf)) {
git clone https://github.com/rizinorg/rz-frida.git --depth 1 rz-frida
git clone https://github.com/rizinorg/rz-frida.git --depth 1 --branch "dev" rz-frida

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

you should take the branch name from the argument, like we do for jsdec etc..

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

you should take the branch name from the argument, like we do for jsdec etc..

ps1 (and shell) script for jsdec also uses hardcoded "dev" if that's what you mean

I'm planning on making these scripts (and CI) less rigid by removing hardcoded values. Making a release for cutter is a nightmare right now, we need a more streamlined approach having all of the configurable values in a single file would be nice

@PremadeS
PremadeS requested a review from wargio October 29, 2026 17:11

This branch has not been deployed

No deployments
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.

4 participants