Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add SketchTool latest #4240

Merged
merged 1 commit into from
May 9, 2014
Merged

Add SketchTool latest #4240

merged 1 commit into from
May 9, 2014

Conversation

silvenon
Copy link
Contributor

@silvenon silvenon commented May 7, 2014

A companion CLI tool for Sketch. http://bohemiancoding.com/sketch/tool/

@vitorgalvao
Copy link
Member

Please squash your commits.

@silvenon
Copy link
Contributor Author

silvenon commented May 7, 2014

Done.

@rolandwalker
Copy link
Contributor

Hi @silvenon! Thanks!

I'm merging this (happily), but it would also be cool to look into whether sketchtool resources.bundle could be located somewhere other than /usr/local/bin. Technically that is anti-Unix. Possibly that is an upstream decision that we cannot change.

rolandwalker added a commit that referenced this pull request May 9, 2014
@rolandwalker rolandwalker merged commit 92353be into Homebrew:master May 9, 2014
@vitorgalvao
Copy link
Member

@rolandwalker I wondered about that as well. The README inside states

You can put the tool anywhere you like (eg /usr/local/bin), but the "sketchtool resources.bundle" file currently has to be in the same directory.

Considering we symlink the tool it should be possible, unless it fails when symlinked, but in that case even linking both shouldn’t work.

rolandwalker added a commit to rolandwalker/homebrew-cask that referenced this pull request May 9, 2014
@rolandwalker
Copy link
Contributor

OK. Let's just document the oddity.

@Homebrew Homebrew locked and limited conversation to collaborators May 8, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants