public inbox for ecos-devel@sourceware.org
 help / color / mirror / Atom feed
From: Simon Kallweit <simon.kallweit@intefo.ch>
To: John Dallaway <john@dallaway.org.uk>
Cc: eCos development list <ecos-devel@ecos.sourceware.org>,
	  John Eigelaar <john@kses.net>
Subject: Re: lwIP upgrade to CVS HEAD
Date: Wed, 13 May 2009 16:03:00 -0000	[thread overview]
Message-ID: <4A0AEF5E.3030602@intefo.ch> (raw)
In-Reply-To: <4A045625.9080706@dallaway.org.uk>

John Dallaway wrote:
> There's some scope for improving the dependency info. For example,
> CYGIMP_LWIP_MEMP_MEM_MALLOC could specify:
> 
>    requires { CYGINT_ISO_MEMALLOC != 0 }
>    default_value { CYGINT_ISO_MEMALLOC != 0 }
> 
> Many of the protocol components and options are also likely to "require"
> either CYGPKG_LWIP_TCP or CYGPKG_LWIP_UDP.
> 
> There may also be some scope for active_if constraints to clarify the
> relevance of certain options. For example, could CYGPKG_LWIP_NETCONF be
> made "active_if !CYGPKG_LWIP_DHCP"? This may not be possible if the
> static networking address definitions are still required for correct
> compilation when DHCP is enabled.

I'll have a look at them tomorrow. There are lots of small improvements 
to be made with dependencies and constraints.

> Finally, I would make the display strings of the numerous protocol
> support components and options just a little more verbose. For example:
> 
>   "DHCP" -> "DHCP support"
>   "ICMP" -> "ICMP support"
> 
> and so on.

Done.

> There are currently 27 items directly under the lwIP package node in the
> tree. Perhaps there is some scope for further grouping of these nodes to
> give the tree more depth and less breadth. For example, a "Protocol
> support" component of "flavor none" containing the various protocol
> components?

I have reorganized the whole CDL quite a bit. Please check out the new 
hierarchy and give me some feedback.

> OK. Send me a tarball of the updated package when you're ready to start
> collaborating and I'll check it in on a new branch.

It's still in the git repo. Please get the current version from there, 
I'll still send you a tarball once I'm ready.

I will also look into sequential support, as there are no words from 
John Eigelaar. I think I can get do the basics myself.

Simon

  reply	other threads:[~2009-05-13 16:03 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <49F83C8B.9000108@intefo.ch>
2009-05-05 10:31 ` John Dallaway
2009-05-05 11:29   ` Simon Kallweit
2009-05-08 15:56     ` John Dallaway
2009-05-13 16:03       ` Simon Kallweit [this message]
2009-05-14 13:27 David Fernandez
2009-05-14 13:31 ` Simon Kallweit

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=4A0AEF5E.3030602@intefo.ch \
    --to=simon.kallweit@intefo.ch \
    --cc=ecos-devel@ecos.sourceware.org \
    --cc=john@dallaway.org.uk \
    --cc=john@kses.net \
    /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).