Skip to content

Fix bindir value missing exec_prefix - #256

Closed
x-xiang wants to merge 1 commit into
JoeDog:masterfrom
x-xiang:master
Closed

Fix bindir value missing exec_prefix#256
x-xiang wants to merge 1 commit into
JoeDog:masterfrom
x-xiang:master

Conversation

@x-xiang

@x-xiang x-xiang commented Sep 29, 2025

Copy link
Copy Markdown

bindir defaults to ${exec_prefix}/bin. If exec_prefix is not defined, utils/bombardment mistakenly invokes /bin/siege instead of the intended binary path. This change corrects the logic error.

@x-xiang x-xiang closed this Aug 18, 2026
@JoeDog

JoeDog commented Aug 19, 2026 via email

Copy link
Copy Markdown
Owner

@x-xiang

x-xiang commented Aug 19, 2026

Copy link
Copy Markdown
Author

@JoeDog No further review is needed for this PR. The issue has already been fixed in the latest 4.2.0 release by this commit (2eeb9a2), so I’ve closed the PR. Thanks!

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