lemnos / tt

A terminal based typing test.
MIT License
714 stars 48 forks source link

Fix `make install` on macOS #9

Closed AulonSal closed 3 years ago

AulonSal commented 3 years ago

install -D is not available on macOS, so we create the directories with install -d before installing. install -t is also not present but turns out we didn't really need it as it's only useful if you want to specify the destination directory before the source, so I just removed the t flag (EDIT: for posterity, the actual reason we don't need to use it is because we already created the directories so install doesn't get confused about whether the last argument is a DEST or DIRECTORY, you need it if the target directory does not exist)

Really, this is my own fault, when changing the Makefile I didn't take other OSs into account, wrongly assuming the coreutils would work the same.

Fixes #7

lemnos commented 3 years ago

Looks good, sorry for the delay.