rrthomas / cw

Colorizing wrapper generator for shell commands, with many pre-defined wrappers
GNU General Public License v3.0
43 stars 2 forks source link

Installing via luarocks does not install wrapper scripts outside rock directory #1

Closed antitoxic closed 11 years ago

antitoxic commented 11 years ago

I just did an install via luarocks on Arch linux but libexec was actually a:

/usr/lib/luarocks/rocks/cw/2.0.0-1/libexec/cw:

Any idea why?

I don't like having a version in the path to the aliases - also means i need to change it when it updates?

rrthomas commented 11 years ago

That path looks like what I'd expect for the rock installation. You should also find the wrappers under /usr/libexec/cw.

antitoxic commented 11 years ago

I thought as well it's todo with rock installation. However I did a global find + manually checked /usr/libexec/cw doesn't exist.

Could be again down to my distro (Arch) - would be a bit annoying if so.

rrthomas commented 11 years ago

I can confirm this is a bug in the rockspec. Thanks very much for reporting it; I'll have a fix out ASAP.

rrthomas commented 11 years ago

(A manual fix is to copy the files to the correct location as above.)

rrthomas commented 11 years ago

Unfortunately, it seems LuaRocks has no support for copying the wrapper scripts to the libexec directory. I can't see a straightforward way around this. The best I can do for copying into an unversioned directory is to have the files copied to a subdirectory of the Lua files tree, but that is rather ugly. Hence for the present I will simply document this problem in the README, and ask for advice from the luarocks developers.

rrthomas commented 11 years ago

The 2.0.1 release should fix your problem (see the README). Unfortunately I can't get rid of the versioned directory, so instead I've introduced a script to print out the correct path, so you don't need to change your shell configuration each time you update cw. Thanks for the bug report!

antitoxic commented 11 years ago

Thank you, Reuben!