From: Junio C Hamano <gitster@pobox.com>
Date: Thu, 31 Jan 2013 15:48:32 GMT
Subject: Re: [PATCH 1/1] Introduce new build variables INSTALL_MODE_EXECUTABLE and INSTALL_MODE_DATA.
Message-ID: <7vy5f9ica7.fsf@alter.siamese.dyndns.org>
In-Reply-To: <20130131132526.GA10622@unpythonic.net>

Jeff Epler <jepler@unpythonic.net> writes:

> I was not familiar with this behavior of 'install -d' that it tries to change
> the mode of an existing directory, but GNU coreutils 8.12.197-032bb
> certainly behaves as TJ reports.
>
> As a possible alternative, what about
>     [ -d $(DESTDIR)$(main1dir) ] || $(INSTALL) -d -m 755 $(DESTDIR)$(man1dir)
> so that $(INSTALL) is not called when the target directory exists
> already.

That can cut both ways, though.  If it exists and its permission is
too tight (say 0750), $(INSTALL) -d -m 755 ought to loosen for
others, but with the additional test, it won't be given a chance to
do so.