Skip to content

fix install-api default paths for Darwin#17

Merged
ttimasdf merged 1 commit intoMCPPhalanx:mainfrom
evanrichter:patch-1
May 13, 2025
Merged

fix install-api default paths for Darwin#17
ttimasdf merged 1 commit intoMCPPhalanx:mainfrom
evanrichter:patch-1

Conversation

@evanrichter
Copy link
Copy Markdown
Contributor

No description provided.

@ttimasdf
Copy link
Copy Markdown
Member

Thanks for the PR! It looks great, but unfortunately, I don’t have access to a Mac right now, so I can’t test it. Could you share a screenshot or console log of binaryninja-mcp install-api running on a Mac?

@evanrichter
Copy link
Copy Markdown
Contributor Author

image

As you can see the first 2 commands fail (these are the default paths) and the last command succeeds (my proposed change)

@ttimasdf ttimasdf merged commit 8a3b5d8 into MCPPhalanx:main May 13, 2025
3 checks passed
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