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

Re: [PATCH 1/1] Introduce new build variables INSTALL_MODE_EXECUTABLE and INSTALL_MODE_DATA.

From
Junio C Hamano <gitster@pobox.com>
Date
Jan 31, 2013, 15:48 UTC
Message-ID
<7vy5f9ica7.fsf@alter.siamese.dyndns.org>
In-Reply-To
<20130131132526.GA10622@unpythonic.net>
Jeff Epler <jepler@unpythonic.net> writes:
Show 8 quoted lines
> 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.

Previous: Jeff EplerNext: Junio C Hamano
Message 3 of 7 in “Introduce new build variables INSTALL_MODE_EXECUTABLE and INSTALL_MODE_DATA.”
  1. 1/1 Introduce new build variables INSTALL_MODE_EXECUTABLE and INSTALL_MODE_DATA.TJ, Jan 31, 2013
  2. Jeff EplerJan 31, 2013
  3. Junio C HamanoJan 31, 2013
  4. Junio C HamanoJan 31, 2013
  5. TJJan 31, 2013
  6. Junio C HamanoFeb 25, 2013
  7. TJFeb 25, 2013

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.