git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: git-instaweb should 'just work' on Mac OS X

From
nathan spindel <nathans@gmail.com>
Date
May 10, 2008, 23:19 UTC
Message-ID
<F08C97B0-A7EE-4A32-9657-DCBDD291502A@gmail.com>
In-Reply-To
<D1BFDAC1-A959-44BD-A562-D787A68612B1@sb.org>
On May 10, 2008, at 3:49 PM, Kevin Ballard wrote:
Show 44 quoted lines
> On May 10, 2008, at 2:27 PM, nathan spindel wrote:
>
>> I've investigated what it would take for git-instaweb to 'just  
>> work' on
>> Mac OS X 10.5 when running the command with no arguments. This  
>> would be
>> of benefit to many Mac git users since they shouldn't be expected to
>> troubleshoot this problem. Here are the current issues at play:
>>
>> - lighttpd does not come with Mac OS X by default, so git-instaweb
>>  currently exits with "lighttpd not found...".
>>
>> - git-instaweb doesn't fallback to using other web servers on the  
>> system
>>  if the default command doesn't exist.
>>
>> - Mac OS X does come with webrick and 'git-instaweb -d webrick'  
>> _does_ just
>>  work. It's probably slower than Apache though (?).
>>
>> - Mac OS X also comes with mongrel, which is a great alternative, but
>>  git-instaweb doesn't support it.
>>
>> - Mac OS X's Apache installation is different than what git-instaweb
>>  expects in a couple areas:
>>  - the Apache binary is called 'httpd', not 'apache2', and git- 
>> instaweb
>>    only looks for the latter.
>>  - the modules on Mac OS X live in /usr/libexec/apache2 but by  
>> default
>>    git-instaweb looks for /usr/lib/apache2/modules.
>>  - Apache attempts to create its 'accept serialization lock file' at
>>    /private/var/run/ but that path is only writeable by root (a  
>> workaround
>>    is to add a LockFile directive to the conf file that places it
>>    in $fqgitdir/gitweb/tmp).
>>
>> I'm interested in fixing this to make the experience smoother for  
>> Mac users.
>> What is the community's preferred avenue of fixing this?
>
> If I were to fix this, I'd suggest making apache2 work properly. I  
> patched the git-svn tests to do this a while back, so you could look  
> at what's I did there (3644da7214).
Thanks Kevin. Your patch is similar to how I imagined fixing apache2.

Would you suggest fixing the other non-starter issue (git-instaweb doesn't fallback to other daemons if the default doesn't exist)? Or maybe changing the default to apache2 on Mac OS X installs (that seems like a poorer fix)?

-nathan
Previous: Kevin BallardNext: Kevin Ballard
Message 3 of 4 in “git-instaweb should 'just work' on Mac OS X”
  1. nathan spindelMay 10, 2008
  2. Kevin BallardMay 10, 2008
  3. nathan spindelMay 10, 2008
  4. Kevin BallardMay 10, 2008

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.