Skip to content

Use -o rather than the undocumented --executable on Linux/macOS - #4

Merged
DNSGeek merged 1 commit into
DNSGeek:claude/issue-1-20260930-0310from
agorangetek:fix-c-backend-output-switch
Oct 1, 2026
Merged

DNSGeek merged 1 commit into
DNSGeek:claude/issue-1-20260930-0310from
agorangetek:fix-c-backend-output-switch

Conversation

@agorangetek

Copy link
Copy Markdown

Based on claude/issue-1-20260930-0310, so this is a one-line amendment to #3 rather than a competing fix — merging it updates that PR, which then carries the whole correction to main.

What changes. --executable → -o for the non-Windows branch. /EXE on Windows is untouched.

Why. PureBasic's reference documents the output switch as -o, --output, /OUTPUT <filename>. --executable appears nowhere: not in pbcompiler -h on 6.41, and not in the manual. I checked that 6.41 does accept it and does write to the path it is given, so this is not a live bug — it is an undocumented alias, with nothing promising it on another version or platform. -o is the same switch in its documented short form: it is the one the switch table lists, and the one the manual's Windows example pairs with /EXE.

Checked. npm install && npm run compile → tsc -p ./ exits 0 on this branch. Unlike the original patch, this has been through the type-checker.

I could not test the Windows /EXE path — no Windows here. Worth noting the manual is inconsistent about it: the cross-platform table gives /OUTPUT, while the Windows example uses /EXE "C:\Project\project.dll". Presumably both are accepted, but that branch still wants a run on Windows before merge.

PureBasic's own reference documents the output switch as
"-o, --output, /OUTPUT <filename>" and the Windows example uses /EXE, but
"--executable" appears nowhere: not in `pbcompiler -h` and not in the manual.
It is accepted by 6.41 and does honour the path, so this is not a live bug --
it is an undocumented alias with nothing promising it on other versions.

-o is the same switch in its documented short form, so the non-Windows branch
now uses it. The /EXE choice for Windows is unchanged and matches the manual's
Windows example.
@DNSGeek
DNSGeek merged commit 691e4d4 into DNSGeek:claude/issue-1-20260930-0310 Oct 1, 2026
1 check passed
@agorangetek
agorangetek deleted the fix-c-backend-output-switch branch October 5, 2026 12:23
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