Skip to content

patchelf: update to 0.19.1 - #61906

Merged
Duncaen merged 1 commit into
void-linux:masterfrom
wbp11:patchelf-update-0.19.1
Aug 8, 2026
Merged

patchelf: update to 0.19.1#61906
Duncaen merged 1 commit into
void-linux:masterfrom
wbp11:patchelf-update-0.19.1

Conversation

@wbp11

@wbp11 wbp11 commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

Testing the changes

  • I tested the changes in this PR: YES

Local build testing

  • I built this PR locally for my native architecture: x86_64-glibc
  • I built this PR locally for these architectures (if supported. mark crossbuilds):
    • x86_64-musl (masterdir)
    • i686 (masterdir)
    • aarch64 (cross)
    • aarch64-musl (cross)
    • armv6l (cross)
    • armv6l-musl (cross)
    • armv7l (cross)
    • armv7l-musl (cross)

Comment thread srcpkgs/patchelf/template
@wbp11

wbp11 commented Aug 8, 2026

Copy link
Copy Markdown
Contributor Author

@Duncaen I applied your change but now the lint phase is failing and I'm not sure why.

@Duncaen

Duncaen commented Aug 8, 2026

Copy link
Copy Markdown
Member

@Duncaen I applied your change but now the lint phase is failing and I'm not sure why.

Please squash the commits. The lint complains about the commit title not being right.

@wbp11

wbp11 commented Aug 8, 2026

Copy link
Copy Markdown
Contributor Author

@Duncaen I applied your change but now the lint phase is failing and I'm not sure why.

Please squash the commits. The lint complains about the commit title not being right.

@Duncaen I applied the change directly on github and I can't find any option to squash those commits. And since I applied the change on github I'm assuming I can't do it locally now? Would it be better to close the PR and create a new one?

@Duncaen
Duncaen force-pushed the patchelf-update-0.19.1 branch from 0e0e220 to 8b36055 Compare August 8, 2026 23:37
@Duncaen

Duncaen commented Aug 8, 2026

Copy link
Copy Markdown
Member

@Duncaen I applied your change but now the lint phase is failing and I'm not sure why.

Please squash the commits. The lint complains about the commit title not being right.

@Duncaen I applied the change directly on github and I can't find any option to squash those commits. And since I applied the change on github I'm assuming I can't do it locally now? Would it be better to close the PR and create a new one?

You should be able to fetch and checkout the same changes and force push them after rebasing. I did it quickly tho.

@Duncaen
Duncaen merged commit 8ad5594 into void-linux:master Aug 8, 2026
8 checks passed
@wbp11

wbp11 commented Aug 8, 2026

Copy link
Copy Markdown
Contributor Author

@Duncaen I applied your change but now the lint phase is failing and I'm not sure why.

Please squash the commits. The lint complains about the commit title not being right.

@Duncaen I applied the change directly on github and I can't find any option to squash those commits. And since I applied the change on github I'm assuming I can't do it locally now? Would it be better to close the PR and create a new one?

You should be able to fetch and checkout the same changes and force push them after rebasing. I did it quickly tho.

Thanks! Sorry about the hassle.

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