Skip to content

cmdline: allow symlinks to binaries to be used for analysis - #303

Merged
waskyo merged 1 commit into
static-analysis-engineering:masterfrom
waskyo:support_simlinks_to_binaries
Oct 29, 2026
Merged

waskyo merged 1 commit into
static-analysis-engineering:masterfrom
waskyo:support_simlinks_to_binaries

Conversation

@waskyo

@waskyo waskyo commented Oct 27, 2026 •

Copy link
Copy Markdown
Collaborator

we now ask 'file' to follow symlinks. The analysis results are kept in the folder with the symlink, not where it points to.

Without this, if you pass chkx a symlink to a binary it will error out with:

********************************************************************************
********************************************************************************
File format ? not supported
********************************************************************************
********************************************************************************

we now ask 'file' to follow symlinks. The analysis
results are kept in the folder with the symlink, not
where it points to
@waskyo
waskyo requested a review from sipma October 27, 2026 16:05

@sipma sipma left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

That is pretty useful. Thank you!

@waskyo
waskyo merged commit d9d8dc6 into static-analysis-engineering:master Oct 29, 2026
4 checks passed
@waskyo
waskyo deleted the support_simlinks_to_binaries branch October 29, 2026 13:57
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