Skip to content

binwalk: update to 3.1.0.#60668

Open
rayfadh wants to merge 1 commit into
void-linux:masterfrom
rayfadh:binwalk
Open

binwalk: update to 3.1.0.#60668
rayfadh wants to merge 1 commit into
void-linux:masterfrom
rayfadh:binwalk

Conversation

@rayfadh
Copy link
Copy Markdown
Contributor

@rayfadh rayfadh commented May 21, 2026

Testing the changes

  • I tested the changes in this PR: YES

Local build testing

  • I built this PR locally for my native architecture, x64-glibc

@Duncaen, I updated binwalk to 3.1.0, just wanna let ya know.

@rayfadh rayfadh force-pushed the binwalk branch 3 times, most recently from ba72c31 to a7d0560 Compare May 21, 2026 16:09
@rayfadh
Copy link
Copy Markdown
Contributor Author

rayfadh commented May 21, 2026

Just a simple test.
image

@rayfadh rayfadh marked this pull request as ready for review May 21, 2026 16:15
@rayfadh
Copy link
Copy Markdown
Contributor Author

rayfadh commented May 22, 2026

image

Comment thread srcpkgs/binwalk/template Outdated
Comment thread srcpkgs/binwalk/template Outdated

post_extract() {
vsed -i -e 's;/etc/bash_completion.d/%s;%s.bash;' setup.py
_setup_env() {
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.

do this instead (adjust for specific cc version)

post_patch() {
# https://github.com/rust-lang/cc-rs/issues/1469
cargo update --package cc:1.2.16 --precise 1.2.14

Copy link
Copy Markdown
Contributor Author

@rayfadh rayfadh May 22, 2026

Choose a reason for hiding this comment

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

Turns out it doesn't need the CC workaround at all to build. IIRC it was for cross-compilation, CMIIW.
image

@rayfadh rayfadh marked this pull request as draft May 22, 2026 16:19
@rayfadh rayfadh force-pushed the binwalk branch 2 times, most recently from 9f2eb3f to fe40585 Compare May 22, 2026 17:25
@rayfadh rayfadh marked this pull request as ready for review May 22, 2026 17:34
Comment thread srcpkgs/binwalk/template Outdated
Comment thread srcpkgs/binwalk/template Outdated
pre_build() {
# strip kaleido binary download feature from plotly
sed -i 's/features = \["kaleido", "kaleido_download"\]//' Cargo.toml
cargo vendor vendor
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.

why is cargo vendor needed?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Oh right, I'm sorry. Just realized I was looking the main branch's Cargo.toml, will strip the whole pre_build() part.

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.

2 participants