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

Re: [PATCH] Makefile: add NEEDS_LIBRT to optionally link with librt

From
Junio C Hamano <gitster@pobox.com>
Date
Jul 7, 2016, 21:14 UTC
Message-ID
<xmqqwpkxnoae.fsf@gitster.mtv.corp.google.com>
In-Reply-To
<20160707204554.14961-1-rdwampler@gmail.com>
Ronald Wampler <rdwampler@gmail.com> writes:
> I am not sure if this the correction solution. Another option I
> considered was to wrap the EXTLIBS += -lrt is an ifndef NO_RT and only
> defining NO_RT for Mac OS X in config.mak.uname.

That alternative would make the resulting code noisier/uglier with nested ifdef, I would imagine, but it would be of less impact to the existing users. But my gut feeling is that the patch you sent is probably a better solution for the longer term.

Thanks.
Previous: Ronald WamplerNext: Eric Wong
Message 2 of 5 in “Makefile: add NEEDS_LIBRT to optionally link with librt”
  1. Makefile: add NEEDS_LIBRT to optionally link with librtRonald Wampler, Jul 7, 2016
  2. Junio C HamanoJul 7, 2016
  3. config.mak.uname: define NEEDS_LIBRT under Linux, for nowEric Wong, Jul 10, 2016
  4. brian m. carlsonJul 11, 2016
  5. Junio C HamanoJul 11, 2016

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.