Re: [PATCH 1/3] git-gui: Adapt discovery of oguilib to execdir 'libexec/git-core'

From: Steffen Prohaska <prohaska@zib.de>
Date: Jul 30, 2008, 05:39 UTC
Message-ID: <59D4282F-7744-4FE2-92FC-11E72386BDEF@zib.de>
In-Reply-To: <20080730052517.GF7225@spearce.org>

On Jul 30, 2008, at 7:25 AM, Shawn O. Pearce wrote:

Show 20 quoted lines

> Steffen Prohaska <prohaska@zib.de> wrote:
>> Isn't only the computation of sharedir based on gitexecdir wrong?
>>
>>> ifndef sharedir
>>> 	sharedir := $(dir $(gitexecdir))share
>>
>> and could be replaced with this (instead of your patch):
>>
>> ifndef sharedir
>> +ifeq (git-core,$(notdir $(gitexecdir)))
>> +       sharedir := $(dir $(patsubst %/,%,$(dir $(gitexecdir))))share
>> +else
>>        sharedir := $(dir $(gitexecdir))share
>> endif
>> +endif
>
> Oh, damn good catch.  Thanks.
>
> How about this then?  Its your patch above, my message, and me
> forging your SOB...

looks good.  SOB ok.

Thanks, Steffen