sindresorhus / quick-look-plugins

List of useful Quick Look plugins for developers
18.09k stars 534 forks source link

Update readme.md #2

Closed jacobvanorder closed 10 years ago

jacobvanorder commented 10 years ago

QuickLook dir is in /Library not ~/Library

wachunei commented 10 years ago

Depends if it is for the current user or all users, this could be a flag on the install script, for example python install.py -afor all users (/Library directory)

mathiasbynens commented 10 years ago

+1 to defaulting to /Library instead of ~/Library. If you’re installing Quick Look plugins anyway you might as well make them available system-wide.

mathiasbynens commented 10 years ago

Note that using brew cask installs (well, links) into ~/Library instead of /Library by default.

samyakbhuta commented 10 years ago

If ~/Library/QuickLook don't exists, the installation (linking) fails. Below is the output for failed command.

==> Downloading https://qlcolorcode.googlecode.com/files/QLColorCode-2.0.2.tgz
######################################################################## 100.0%
==> Linking QuickLook plugin QLColorCode.qlgenerator to /Users/<username>/Library/QuickLook/QLColorCode.qlgenerator
Error: Command failed to execute!

==> Failed command:
/bin/ln '-hfs' '/opt/homebrew-cask/Caskroom/qlcolorcode/2.0.2/QLColorCode-2.0.2/QLColorCode.qlgenerator' '/Users/<username>/Library/QuickLook/QLColorCode.qlgenerator' 2>&1

==> Output of failed command:
ln: /Users/<username>/Library/QuickLook/QLColorCode.qlgenerator: No such file or directory

Needless to say, mkdir ~/Library/QuickLook makes it possible.

sindresorhus commented 10 years ago

@samyakbhuta can you open a ticket on Homebrew Cask?

samyakbhuta commented 10 years ago

The cask is not doing anything but to fetch and extract. See here.

mathiasbynens commented 10 years ago

Yeah, but the qlplugin line should take care of mkdir -p ~/Library/QuickLook (but it doesn’t).

samyakbhuta commented 10 years ago

sudo brew cask install qlcolorcode --qlplugindir="/Library/QuickLook" does install it in /Library/QuickLook. Not sure if sudo is recommended or not in this case, though.