weslly / ColorPicker

Color picker for Sublime Text
http://weslly.github.io/ColorPicker
511 stars 108 forks source link

ST3: Fix colorpicker binary missing on linux/osx #24

Closed ladios closed 11 years ago

ladios commented 11 years ago

Should fix the FileNotFoundError in #21 Tested with ST2 & 3 on Linux only, but OSX should be the same.

ladios commented 11 years ago

Well, it seems I didn't test it completely...

Traceback (most recent call last): File "/home/ladios/.config/sublime-text-3/Installed Packages/ColorPicker.sublime-package/sublimecp.py", line 355, in update_binary File "X/os.py", line 258, in makedirs File "X/os.py", line 258, in makedirs File "X/os.py", line 266, in makedirs PermissionError: [Errno 13] Permission denied: 'User'

ladios commented 11 years ago

Previously I forgot to remove package folder. Turns out the sublime.packages_path() behaves differently at runtime, when it's a archive format of package. It really works now. Should I squeeze them into one commit?

weslly commented 11 years ago

Thanks for the fix and sorry for the delay. I'll merge it as soon as I test if it works at windows.