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

Re: mkstemps compat issue

From
Andreas Färber <andreas.faerber@web.de>
Date
Jun 7, 2009, 12:49 UTC
Message-ID
<DAD17464-336A-451A-AEFE-26481D6B39B2@web.de>
In-Reply-To
<ED882BBA-E0E7-4CCC-9C6B-72FEBED1C9E0@web.de>
Am 07.06.2009 um 14:32 schrieb Andreas Färber:
Show 7 quoted lines
> The merge commit b11cf09043f18b368ec0d988f064ea21247c843d introduced  
> a git_mkstemps function in path.c, using mkstemps for the  
> implementation. Unlike mkstemp that function is not part of POSIX,  
> so we seem to need a NO_MKSTEMPS option and corresponding  
> alternative code path.
>
> Any suggestions how to best replace mkstemps?

Sorry for the noise... NO_MKSTEMPS=YesPlease does work correctly if typed correctly. (Thanks for the hint!)

It was a little confusing seeing mkstemps being used unconditionally in path.c, but it's being #defined in git-compat-util.h, so everything is fine.

Regards,
Andreas
Previous: Andreas FärberNext: David Aguilar
Message 2 of 3 in “mkstemps compat issue”
  1. Andreas FärberJun 7, 2009
  2. Andreas FärberJun 7, 2009
  3. David AguilarJun 8, 2009

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.