Skip to content

fix: use correct Homebrew install syntax#199

Merged
forketyfork merged 1 commit intomainfrom
fix/homebrew-install-syntax
Feb 4, 2026
Merged

fix: use correct Homebrew install syntax#199
forketyfork merged 1 commit intomainfrom
fix/homebrew-install-syntax

Conversation

@forketyfork
Copy link
Owner

Summary

Fixes the Homebrew formula installation error:

ArgumentError: wrong number of arguments (given 2, expected 0)

Solution

The previous code incorrectly used install source, destination as a standalone function. The correct Homebrew pattern is destination.install source where install is called on the destination Pathname.

Changed from:

install buildpath/"zig-out/bin/architect", macos/"architect"

To:

macos.install buildpath/"zig-out/bin/architect"

This matches the pattern already used in the same formula at line 71:

resources.install "assets/macos/#{app_name}.icns"

Test plan

  • brew install --build-from-source forketyfork/architect/architect completes successfully

The install method is called on the destination directory, not standalone.
@forketyfork forketyfork merged commit f482c54 into main Feb 4, 2026
4 checks passed
@forketyfork forketyfork deleted the fix/homebrew-install-syntax branch February 4, 2026 13:16
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.

1 participant