Fix cp error on Mac OS - Githubissues

asdf-vm / asdf-elixir

Elixir plugin for asdf version manager

https://github.com/asdf-vm/asdf

MIT License

473 stars 70 forks source link

Fix cp error on Mac OS #100

Closed LouisMT closed 3 years ago

LouisMT commented 3 years ago

The cp command doesn't have a -T switch on Mac OS:

Build succeeded!
cp: illegal option -- T
usage: cp [-R [-H | -L | -P]] [-fi | -n] [-apvXc] source_file target_file
       cp [-R [-H | -L | -P]] [-fi | -n] [-apvXc] source_file ... target_directory

Stratus3D commented 3 years ago

Thanks for the PR @LouisMT ! Did you happen to test this on other platforms besides OSX? Curious if the -a flag is widely supported.

LouisMT commented 3 years ago

I'll check this soon!

juulSme commented 3 years ago

It's supported on ubuntu (20.04). Also it's mentioned on line 10059 of GNU coreutils changelog (https://fossies.org/linux/coreutils/ChangeLog) in an entry dated 2014, so I think it's been supported for quite a long time. Sorry for messing this up in #99

Stratus3D commented 3 years ago

Thanks @juulSme and @LouisMT, I went ahead and merged.