Conversation
It is a Tcl/Tk script using wish, already installed on OS X.
# It's enough to just replace "false" with the main program this | ||
# formula installs, but it'd be nice if you were more thorough. | ||
# Test the test with `brew test tkdiff`. | ||
system "tkdiff --help" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove the comments here, and specify the path to the binary, i.e. system "#{bin}/tkdiff --help"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I removed comments, added #{bin}/, then tried to squash and change the pull request (to f1e5ff8).
Since I'm not super git-savvy, I don't know if the pull request has been changed properly. Let me know if I need to take further action.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, I'll squash it down to a single commit myself.
It is a Tcl/Tk script using wish, already installed on OS X. Response to review comments from Jack.
It works well on Lion. Thanks for the formula. |
Pulled. |
Thanks for homebrew! |
Closes Homebrew#10100. Signed-off-by: Jack Nagel <jacknagel@gmail.com>
Closes Homebrew#10100. Signed-off-by: Jack Nagel <jacknagel@gmail.com>
Closes Homebrew#10100. Signed-off-by: Jack Nagel <jacknagel@gmail.com>
It is a Tcl/Tk script using wish, already installed on OS X.