Skip to content

Commit 6ebbd8f

Browse files
authored
Respond to PR comments
* Use "conventional" instead of "correct" * Note the installation of realpath isn't strictly necessary
1 parent 663a2b4 commit 6ebbd8f

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

README.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,12 +27,12 @@ Then use the following commands:
2727

2828
### Notes for macOS
2929

30-
The correct install directory for macOS is `/usr/local/avr-rust` rather than the Linux default `/opt/avr-rust`, so before the steps below, you must:
30+
The conventional install directory for macOS is `/usr/local/avr-rust` rather than the Linux default `/opt/avr-rust`, so before the steps below, you must:
3131

3232
1. Edit `build/config.toml` to set `prefix = '/usr/local/avr-rust'`.
3333
2. Ensure `/usr/local/avr-rust` exists and has the correct permissions: `sudo mkdir /usr/local/avr-rust && sudo chown $USER:admin /usr/local/avr-rust`
3434

35-
Finally, `realpath` isn't included by default on macOS but is included in GNU Coreutils, so you need to `brew install coreutils` so the `rustup toolchain...` step works properly.
35+
Finally, `realpath` isn't included by default on macOS but is included in GNU Coreutils, so you can either `brew install coreutils` so the `rustup toolchain...` step works properly, or just use the explicit path (`rustup toolchain link avr-toolchain /usr/local/avr-rust`).
3636

3737
``` bash
3838
# Grab the avr-rust sources

0 commit comments

Comments
 (0)