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

Re: [PATCH] Makefile: remove the NO_R_TO_GCC_LINKER flag

From
Jeff King <peff@peff.net>
Date
May 17, 2019, 00:23 UTC
Message-ID
<20190517002303.GB13525@sigill.intra.peff.net>
In-Reply-To
<xmqqd0ki3s0q.fsf@gitster-ct.c.googlers.com>
On Fri, May 17, 2019 at 08:21:09AM +0900, Junio C Hamano wrote:
Show 15 quoted lines
> Jeff King <peff@peff.net> writes:
> 
> > Far be it from me to care about AIX, but it seems like this is ripe for
> > regressions, because we don't know which platforms were relying on "-R"
> > instead of "-Wl,-rpath", and now everybody will be using the latter by
> > default.
> 
> I do not have a stake in AIX, either, but I had the same reaction.
> 
> Those who found having to give NO_R_TO_GCC_LINKER=NoThanks annoying
> could have passed CC_LD_DYNPATH=-Wl,-rpath instead, but that is not
> much shorter or sweeter X-<; with this change, they do not have to
> do anything, but those who are broken can pass CC_LD_DYNPATH=-R to
> unbreak it.  So it may not be the end of the world, but this move
> certainly smells like robbing non-GCC users to pay GCC users to me.

Thanks, I wasn't quite clear on whether there was an escape hatch for people who were broken (I took Ævar's "perhaps we should follow this up" meaning we needed to add new knobs, but it is really just "perhaps we should set this existing knob by default on AIX"). So that at least quells the worst of my worries.

After that, I agree it's now just a question of the default. I don't have a sense whether the majority is helped or hurt here (I admit that I did not even know about these flags, despite compiling with gcc for the past 13 years, so possibly everybody was perfectly happy with the existing behavior).

-Peff
Previous: Junio C HamanoNext: Ævar Arnfjörð Bjarmason
Message 14 of 19 in “Improving HP-UX support”
  1. Improving HP-UX supportOsipov, Michael, May 8, 2019
  2. Ævar Arnfjörð BjarmasonMay 9, 2019
  3. Osipov, MichaelMay 9, 2019
  4. sha1dc: update from upstreamÆvar Arnfjörð Bjarmason, May 13, 2019
  5. Osipov, MichaelMay 14, 2019
  6. Ævar Arnfjörð BjarmasonMay 15, 2019
  7. Osipov, MichaelMay 16, 2019
  8. Ævar Arnfjörð BjarmasonMay 16, 2019
  9. Osipov, MichaelMay 16, 2019
  10. configure: Detect linking style for HP aCC on HP-UXÆvar Arnfjörð Bjarmason, May 16, 2019
  11. Makefile: remove the NO_R_TO_GCC_LINKER flagÆvar Arnfjörð Bjarmason, May 16, 2019
  12. Jeff KingMay 16, 2019
  13. Junio C HamanoMay 16, 2019
  14. Jeff KingMay 17, 2019
  15. Makefile: remove the NO_R_TO_GCC_LINKER flagÆvar Arnfjörð Bjarmason, May 17, 2019
  16. Junio C HamanoMay 19, 2019
  17. Jeff KingMay 19, 2019
  18. Osipov, MichaelJun 7, 2019
  19. Junio C HamanoJun 7, 2019

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.