public inbox for ecos-maintainers@sourceware.org
 help / color / mirror / Atom feed
From: Bart Veer <bartv@ecoscentric.com>
To: John Dallaway <john@dallaway.org.uk>
Cc: ecos-maintainers@ecos.sourceware.org
Subject: Re: Building RedBoot for SH3 targets with new toolchain
Date: Thu, 22 Jan 2009 14:40:00 -0000	[thread overview]
Message-ID: <pn3afbz9ne.fsf@delenn.bartv.net> (raw)
In-Reply-To: <497866C6.80009@dallaway.org.uk> (message from John Dallaway on 	Thu, 22 Jan 2009 12:29:58 +0000)

>>>>> "John" == John Dallaway <john@dallaway.org.uk> writes:

    John> Hi Andrew
    John> Andrew Lunn wrote:

    >>> Taking all these things into account, plus the desire to avoid
    >>> further delays to the eCos 3.0 release, I think it preferable
    >>> to stick with the stock mpfr and gmp DLLs for the
    >>> Cygwin-hosted toolchains. We must document the need to install
    >>> the relevant Cygwin packages though, and add an entry to the
    >>> FAQ.
    >> 
    >> Would it be possible to detect at build time if these DLLs are
    >> installed? But a custom build rule with the highest priority in
    >> infra, or HAL which will test if this files exist and stop and
    >> print an useful error message if they are not installed?
    >> 
    >> The way cygwin is silently failing is not nice...

    John> The failure mode is certainly not nice. I expect that the
    John> mpfr and gmp DLLs are loaded explicitly by the compiler
    John> using LoadLibrary() and failure is not being reported at the
    John> application level.

    John> We could emit a _warning_ message using a custom rule. An
    John> _error_ message would be too strong since older toolchains
    John> do not use mpfr and gmp. However, it seems cleaner to keep
    John> such testing out of the build process and perform the tests
    John> in the eCos install script instead when the relevant
    John> toolchains are being installed. Having said that, I
    John> appreciate that experienced eCos developers may not use the
    John> install script.

Just how much does it save to link mpfr and gmp dynamically instead of
statically? Ditto for intl and iconv? If it only makes a small
difference I would much prefer to keep things robust and use static
links - and as a useful side effect, avoid problems if updated cygwin
versions of these packages have compatibility issues. Or when moving
toolchains between machines with slightly different cygwin setups. And
no doubt there are plenty of other failure scenarios.

I would not mind so much if the users were given sensible error
messages that explained exactly how to resolve the problem, but silent
failures are going to give a really bad impression to anybody just
starting out with eCos. Especially since the chances are high that any
installation instructions will have received, at best, a cursory
glance.

Bart

-- 
Bart Veer                                   eCos Configuration Architect
eCosCentric Limited    The eCos experts      http://www.ecoscentric.com/
Barnwell House, Barnwell Drive, Cambridge, UK.      Tel: +44 1223 245571
Registered in England and Wales: Reg No 4422071.

  reply	other threads:[~2009-01-22 14:40 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-01-21 19:48 John Dallaway
2009-01-21 20:30 ` Jonathan Larmour
2009-01-21 22:08   ` John Dallaway
2009-01-21 22:33     ` Jonathan Larmour
2009-01-21 23:34       ` John Dallaway
2009-01-22  0:29         ` Jonathan Larmour
2009-01-22 10:15           ` John Dallaway
2009-01-22 10:31             ` Andrew Lunn
2009-01-22 12:30               ` John Dallaway
2009-01-22 14:40                 ` Bart Veer [this message]
2009-01-22 16:59                   ` John Dallaway

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=pn3afbz9ne.fsf@delenn.bartv.net \
    --to=bartv@ecoscentric.com \
    --cc=ecos-maintainers@ecos.sourceware.org \
    --cc=john@dallaway.org.uk \
    /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.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).