Skip to content

Conversation

@iarata
Copy link

@iarata iarata commented Oct 23, 2025

Clarified installation instructions for macOS by mentioning that LLD is provided in a separate formula in Homebrew.

Description

  • Adds lld to brew install llvm for building .msi on macOS. This is because LLD is now being offered in a separate formula on brew

Clarified installation instructions for macOS by mentioning that LLD is provided in a separate formula in Homebrew.
@iarata iarata requested a review from a team as a code owner October 23, 2025 23:15
@github-project-automation github-project-automation bot moved this to 🪵 Backlog in Documentation Oct 23, 2025
@netlify
Copy link

netlify bot commented Oct 23, 2025

Deploy Preview for tauri-v2 ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 288f545
🔍 Latest deploy log https://app.netlify.com/projects/tauri-v2/deploys/68faf25786b5410008352b57
😎 Deploy Preview https://deploy-preview-3553--tauri-v2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 99 (🔴 down 1 from production)
Accessibility: 98 (no change from production)
Best Practices: 100 (no change from production)
SEO: 100 (no change from production)
PWA: 80 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

```

On macOS you also have to add `/opt/homebrew/opt/llvm/bin` to your `$PATH` as suggested in the install output.
On macOS you also have to add `/opt/homebrew/opt/llvm/bin` to your `$PATH` as suggested in the install output (LLD is now provided in a separate formula in Homebrew).
Copy link
Member

Choose a reason for hiding this comment

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

What is this trying to convey? Do users need to add lld manually to the path as well?

Copy link
Author

@iarata iarata Oct 24, 2025

Choose a reason for hiding this comment

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

No they just need to install it separately as it's no longer installed along LLVM
Perhaps the description edits wasn't clear enough 🤔

Copy link
Member

Choose a reason for hiding this comment

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

Yeah, i think it's fine to just remove this change. The edit of the brew install line should be enough.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🪵 Backlog

Development

Successfully merging this pull request may close these issues.

3 participants