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

Re: [RFC/PATCH] build: avoid possible confusion between GNU/XPG4 make on Solaris

From
Stefano Lattarini <stefano.lattarini@gmail.com>
Date
Mar 2, 2012, 19:00 UTC
Message-ID
<4F5118CF.6030402@gmail.com>
In-Reply-To
<7vhay6etqc.fsf@alter.siamese.dyndns.org>
On 03/02/2012 07:35 PM, Junio C Hamano wrote:
Show 25 quoted lines
> Stefano Lattarini <stefano.lattarini@gmail.com> writes:
> 
>> On a Solaris 10 system with Solaris XPG4 make installed as /usr/xpg4/bin/make,
>> GNU make installed as /usr/local/bin/make, and with /usr/local/bin appearing
>> in $PATH *before* /usr/xpg4/bin, I was seeing errors like this upon invoking
>> "make all":
> 
> After reading this explanation, my first reaction is that the prefixing of
> path _is_ what is wrong.  The prefixing is done to help a subset of
> Solaris users who are unaware of /usr/xpg4/bin that are more POSIX than
> what they have in /usr/bin, and that is what is hurting people like you
> who know what you are doing and have suitable tools in other places, like
> you do in /usr/local/bin.
> 
> And the real fix for your problem is _not_ an ugly override of $(MAKE)
> like you do in this patch, I think.  After all, somebody else who have a
> tool in /usr/local/bin that is saner than what is in /usr/xpg4/bin may
> suffer from the same issue for commands other than "make".
> 
> So the real solution would probably be to let you override how the
> BROKEN_PATH_FIX works, no?
> 
> Ah... and I think we already have such a solution in our Makefile.  Can't
> you override SANE_TOOL_PATH in your config.mak instead?
>

Yes and no. While in hindsight I agree that using this already-provided solution is much better than my ugly automatic munging of $(MAKE) (so yes, let's scrap the present patch), I also think that if one is setting up the configuration of the Git tree using the the autoconf-generated configure script (as I do), he wants configure to Do The Right Thing automatically in this regard too -- thus recognizing that /usr/xpg4/bin is already in $PATH before /bin and /usr/bin, and setting up an appropriate default for SANE_TOOL_PATH in 'config.make.autogen' accordingly.

What would you think of a patch in this direction?
Thanks,
  Stefano
Previous: Junio C HamanoNext: Junio C Hamano
Message 6 of 12 in “build: avoid possible confusion between GNU/XPG4 make on Solaris”
  1. build: avoid possible confusion between GNU/XPG4 make on SolarisStefano Lattarini, Mar 2, 2012
  2. Thomas RastMar 2, 2012
  3. Stefano LattariniMar 2, 2012
  4. build: avoid possible confusion between GNU/XPG4 make on SolarisStefano Lattarini, Mar 2, 2012
  5. Junio C HamanoMar 2, 2012
  6. Stefano LattariniMar 2, 2012
  7. Junio C HamanoMar 2, 2012
  8. configure: allow user to prevent $PATH "sanitization" on SolarisStefano Lattarini, Mar 9, 2012
  9. Junio C HamanoMar 9, 2012
  10. Junio C HamanoMar 9, 2012
  11. Stefano LattariniMar 9, 2012
  12. Junio C HamanoMar 9, 2012

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.