Source-Changes-D archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/share/mk

On 11.12.2012 00:25, matthew green wrote:
Module Name:    src
Committed By:   pooka
Date:           Mon Dec 10 20:58:55 UTC 2012

Modified Files:

Log Message:
Deal with crti and crtn not present in some EXTERNAL_TOOLCHAIN's.  Also,
handle "missing" crtstuff internally instead of depending on external

what is this change for?

So that you don't get bogus dependencies.

using sed here is wrong, irrespective of the comment reason you've
given.  if TOOL_SED isn't available, we shouldn't even be processing
this file yet.  that seems like the fix you should look for here.

If using sed here is wrong, then using sed instead of TOOL_SED in the multiple configure scripts under src/tools is wrong too. Do you want to fix those first?

what is this EXTERNAL_TOOLCHAIN you're using?  i'm totally at a loss
what this change is useful for besides re-adding a host dependancy
on sed.

The solaris/cygwin toolchains, e.g.:

$ cc --print-file-name=crtbegin.o
$ cc --print-file-name=crtbeginS.o

Home | Main Index | Thread Index | Old Index