From: Dan Nicolaescu <dann@gnu.org>
To: Angelo Graziosi <angelo.graziosi@alice.it>
Cc: emacs <emacs-devel@gnu.org>
Subject: Re: Trunk bootstrap failure [Cygwin]
Date: Fri, 09 Jul 2010 21:36:52 -0400 [thread overview]
Message-ID: <yxqr5jci0cr.fsf@fencepost.gnu.org> (raw)
In-Reply-To: <4C37A436.1090400@alice.it> (Angelo Graziosi's message of "Sat\, 10 Jul 2010 00\:35\:34 +0200")
Angelo Graziosi <angelo.graziosi@alice.it> writes:
> 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...
I prefer to get rid of code that is only used by a single platfrom
(ecrt0.c), when the more commonly used code (pre-crt0.c) seems to do the job.
prev parent reply other threads:[~2010-07-10 1:36 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-07-05 23:35 Trunk bootstrap failure [Cygwin] Angelo Graziosi
2010-07-06 2:29 ` Dan Nicolaescu
2010-07-06 9:08 ` Angelo Graziosi
2010-07-06 15:19 ` Dan Nicolaescu
2010-07-07 9:12 ` Angelo Graziosi
2010-07-07 10:34 ` Eli Zaretskii
2010-07-07 12:11 ` Ken Brown
2010-07-07 16:22 ` Dan Nicolaescu
2010-07-08 0:16 ` Angelo Graziosi
2010-07-08 5:54 ` Dan Nicolaescu
2010-07-08 9:01 ` Angelo Graziosi
2010-07-08 10:09 ` Angelo Graziosi
2010-07-08 10:22 ` Eli Zaretskii
2010-07-08 10:32 ` Andreas Schwab
2010-07-08 23:07 ` Dan Nicolaescu
2010-07-09 13:49 ` Angelo Graziosi
2010-07-09 17:08 ` Dan Nicolaescu
2010-07-09 22:35 ` Angelo Graziosi
2010-07-10 1:36 ` Dan Nicolaescu [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=yxqr5jci0cr.fsf@fencepost.gnu.org \
--to=dann@gnu.org \
--cc=angelo.graziosi@alice.it \
--cc=emacs-devel@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.