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

Re: [PATCH 2/4] Makefile: add NEEDS_RESOLV to optionally add -lresolv to compile arguments

From
Jeff King <peff@peff.net>
Date
May 28, 2009, 19:35 UTC
Message-ID
<20090528193537.GA14324@coredump.intra.peff.net>
In-Reply-To
<vVyKoW1VAw2ieNPkJ-SGMNqgFzp_toRpDdk-ZeAp3qzdczSzN4yckg@cipher.nrlssc.navy.mil>
On Thu, May 28, 2009 at 02:32:20PM -0500, Brandon Casey wrote:
Show 9 quoted lines
> >> This library is required on Solaris since hstrerror resides in libresolv.
> >> Additionally, on Solaris 7, inet_ntop and inet_pton reside there too.
> > 
> > Patch works for me on Solaris 8, though it is largely a non-issue:
> > hstrerror is only used at all if NO_IPV6 is set, and I don't set that
> > for my build.
> 
> Ah, in that case I guess NEED_RESOLV should only go in the 5.7 specific
> configuration section.

Well, not exactly. Getting rid of NO_HSTRERROR and then compiling with NO_IPV6 still requires NEED_RESOLV. It's just that NO_HSTRERROR isn't even useful unless NO_IPV6 is defined.

-Peff
Previous: Brandon CaseyNext: Jeff King
Message 9 of 10 in “Makefile: use /usr/ucb/install on SunOS platforms rather than ginstall”
  1. 1/4 Makefile: use /usr/ucb/install on SunOS platforms rather than ginstallBrandon Casey, May 28, 2009
  2. 2/4 Makefile: add NEEDS_RESOLV to optionally add -lresolv to compile argumentsBrandon Casey, May 28, 2009
  3. 3/4 Makefile: add section for SunOS 5.7Brandon Casey, May 28, 2009
  4. 4/4 Makefile: introduce SANE_TOOL_PATH for prepending required elements to PATHBrandon Casey, May 28, 2009
  5. Jeff KingMay 28, 2009
  6. Junio C HamanoMay 29, 2009
  7. Jeff KingMay 28, 2009
  8. Brandon CaseyMay 28, 2009
  9. Jeff KingMay 28, 2009
  10. Jeff KingMay 28, 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.