Re: Trunk bootstrap failure [Cygwin]

[Angelo Graziosi <angelo.graziosi@alice.it>]

Il 09/07/2010 19.08, Dan Nicolaescu ha scritto:
> Angelo Graziosi<angelo.graziosi@alice.it>  writes:
>
>> Il 09/07/2010 1.07, Dan Nicolaescu ha scritto:
>>> Angelo Graziosi<angelo.graziosi@alice.it>   writes:
>>>
>>>> Il 08/07/2010 7.54, Dan Nicolaescu ha scritto:
>>>>> Angelo Graziosi<angelo.graziosi@alice.it>    writes:
>>>>>
>>>>>> Since 'start_of_text' is not used by Cygwin build, shouldn't 'ecrt0.c'
>>>>>> be completely unnecessary?
>>>>>
>>>>> That's exactly the reason for this discussion: getting rid of ecrt0.c.
>>>>
>>>> Hmm... bootstrapping rev. 100753 *without* patches (i.e. using
>>>> ecrt0.c), shows that:
>>>>
>>>> $ grep -R start_of_text emacs/inst/usr/local/emacs
>>>>
>>>> is *empty* (emacs/inst/usr/local/emacs is the Emacs installation
>>>> directory). This would mean that the function 'start_of_text' is not
>>>> compiled any more in the Cygwin build. Right?
>>>
>>> Sure, before yesterday it was just sitting there unused.
>>>
>>
>> What about 'ecrt0.c'? It seems that it happens the same for it.
>>
>> For completeness, the bootstrap (rev. 100755) I did applying *only*
>> this patch:
>>
>> =====================================================
>> --- emacs.orig/configure        2010-07-02 11:27:38.000000000 +0200
>> +++ emacs/configure     2010-07-06 10:45:21.656250000 +0200
>> @@ -5864,7 +5864,7 @@
>>   case $opsys in
>>     cygwin )
>>       LIB_MATH=
>> -    START_FILES='ecrt0.o'
>> +    START_FILES='pre-crt0.o'
>>       ;;
>>     darwin )
>>       ## Adding -lm confuses the dynamic linker, so omit it.
>> =====================================================
>>
>> is still working.
>
> As expected.  Nothing in this area has changed since you tried this a
> couple of days ago.
> I'd like to make this change.
> And in order to plan for the unexpected: if in the next few months you
> get weird memory errors, are you willing to test again with this patch reverted?

Obviously, but if you prefer, you can leave the things as they stand and 
I add that patch to my set of personal patches with which I bootstrap 
Emacs...

Ciao,
Angelo.

---
Tu proverai si' come sa di sale
lo pane altrui, e come e' duro calle
lo scendere e 'l salir per l'altrui scale.

                   DANTE, Paradiso, xvii 58-60