On Mar 11, 2014, at 3:01 PM, Guy Harris <g...@alum.mit.edu> wrote:

> 
> On Mar 11, 2014, at 11:18 AM, Hadriel Kaplan <hadriel.kap...@oracle.com> 
> wrote:
> 
>> I see you have the 'pip install git-review' stuff on that page... fwiw, I've 
>> never gotten git-review to work on my Mac.  I got it installed with pip, but 
>> issuing any 'git review' command doesn't work.  I don't *think* it's a path 
>> problem, but instead whatever mechanics git-review did to hook itself into 
>> git's command parser
> 
> Git's command parser appears to be something similar to
> 
>       system("git-" || the arguments to git);
> 
>> (through Python it seems)
> 
>       $ which git-review
>       /usr/local/bin/git-review
>       $ file `which git-review`
>       /usr/local/bin/git-review: Python script, ASCII text executable
> 
> so git-review is just a Python script with a #! header, so that an 
> exec-family call (which includes posix_spawn() on systems, such as newer 
> versions of OS X, that have it) on that file will run the Python interpreter 
> on it.

Ahhh. So then it *is* a path problem.  Naturally 'pip install' put the 
"installed" binary in:
/opt/local/Library/Frameworks/Python.framework/Versions/2.7/bin/
on my machine.  Because, ya know, it's so useful there.
:)

Thanks for the info - it workie now.

-hadriel

___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev@wireshark.org>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-requ...@wireshark.org?subject=unsubscribe

Reply via email to