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

Re: [patch 02/15] const-expr.patch

From
GVGary V. Vaughan <git@mlists.thewrittenword.com>
Date
Apr 25, 2010, 08:38 UTC
Message-ID
<20100317060108.GB27121@thor.il.thewrittenword.com>
In-Reply-To
<32541b131003161643g1a1cbf6ck3089ae5fb016a985@mail.gmail.com>

[It seems like all my replies to this list don't make it through, even though I'm able to post a new thread. Please forward on my behalf.]

On Tue, Mar 16, 2010 at 07:43:54PM -0400, Avery Pennarun wrote:
Show 20 quoted lines
> On Tue, Mar 16, 2010 at 3:31 AM, Johannes Sixt <j.sixt@viscovery.net> wrote:
> > Gary V. Vaughan schrieb:
> >> Unfortunately, there are still plenty of production systems with
> >> vendor compilers that choke unless all compound declarations can be
> >> determined statically at compile time, for example hpux10.20 (I can
> >> provide a comprehensive list of our supported platforms that exhibit
> >> this problem if necessary).
> >
> > Yes, a comprehensive list would be appreciated. This change is an
> > uglification that I personally would prefer to stay out of the code base
> > unless many consumers of git are hurt.
> >
> > The problem with this non-feature is that it is all too easy that new code
> > introduces new incompatibilities.
> 
> This is probably a stupid question, but why not just build it using
> gcc on systems with a broken vendor compiler?  You don't have to
> distribute gcc just to distribute binaries built with it, so it seems
> like there's no downside... and less suffering for the build
> maintainer.

More often than not the vendor compiler produces better code than gcc on any given architecture, and one of the features of the TWW package builds is that in the vast majority of cases we port the upstream code to build using the vendor compiler, because this is what our customers want.

Cheers,
-- 
Gary V. Vaughan (gary@thewrittenword.com)
Previous: Avery PennarunNext: Gary V. Vaughan
Message 8 of 37 in “Portability Patches v3”
  1. 00/15 Portability Patches v3Gary V. Vaughan, Mar 16, 2010
  2. 01/15 user-cppflags.patchGary V. Vaughan, Mar 16, 2010
  3. Junio C HamanoMar 16, 2010
  4. 02/15 const-expr.patchGary V. Vaughan, Mar 16, 2010
  5. Johannes SixtMar 16, 2010
  6. Gary V. VaughanMar 16, 2010
  7. Avery PennarunMar 16, 2010
  8. Gary V. VaughanApr 25, 2010
  9. 03/15 pthread.patchGary V. Vaughan, Mar 16, 2010
  10. Junio C HamanoMar 16, 2010
  11. 04/15 diff-export.patchGary V. Vaughan, Mar 16, 2010
  12. Junio C HamanoMar 16, 2010
  13. 05/15 diff-test_cmp.patchGary V. Vaughan, Mar 16, 2010
  14. Junio C HamanoMar 16, 2010
  15. Gary V. VaughanApr 25, 2010
  16. 06/15 diff-defaults.patchGary V. Vaughan, Mar 16, 2010
  17. Junio C HamanoMar 16, 2010
  18. Gary V. VaughanApr 25, 2010
  19. 07/15 host-IRIX.patchGary V. Vaughan, Mar 16, 2010
  20. Junio C HamanoMar 16, 2010
  21. Gary V. VaughanApr 25, 2010
  22. Brandon CaseyMar 16, 2010
  23. Gary V. VaughanApr 25, 2010
  24. 08/15 host-HPUX10.patchGary V. Vaughan, Mar 16, 2010
  25. 09/15 host-HPUX11.patchGary V. Vaughan, Mar 16, 2010
  26. 10/15 host-OSF1.patchGary V. Vaughan, Mar 16, 2010
  27. 11/15 no-hstrerror.patchGary V. Vaughan, Mar 16, 2010
  28. Junio C HamanoMar 16, 2010
  29. Gary V. VaughanApr 25, 2010
  30. 12/15 no-inet_ntop.patchGary V. Vaughan, Mar 16, 2010
  31. 13/15 no-socklen_t.patchGary V. Vaughan, Mar 16, 2010
  32. Junio C HamanoMar 16, 2010
  33. Gary V. VaughanApr 25, 2010
  34. 14/15 no-ss_family.patchGary V. Vaughan, Mar 16, 2010
  35. 15/15 no-inline.patchGary V. Vaughan, Mar 16, 2010
  36. Sverre RabbelierMar 16, 2010
  37. Brandon CaseyMar 16, 2010

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.