Re: ATHTTPD Server

[Jonathan Larmour <jifl at eCosCentric dot com>]

Hi Anthony,

Great work, thanks.

Anthony Tonizzo wrote:
> Here is the patch for a new web server for ecos.
> I won't list the features since the docbook documentation
> is included.
>
> I have been using it for a while, and I am happy with it,
> but I am sure it would benefit from other people trying it.
> I am more than willing to help out early users setting it
> up and working out the kinks.

A few minor things that I've fixed up:

- the CDL include_dir is cyg/httpd - I think this should be cyg/athttpd. 
It's not like the API is generic between httpd implementations.

- All the CDL options should contain ATHTTPD rather than just HTTPD. 
Otherwise, despite the "requires !CYGPKG_HTTPD" at the top, there will be 
an error even loading the package should there in future be options that 
conflict with the other HTTPD. It's just to follow the name spacing principle.

- There should be a CYGPKG_ATHTTPD_OPTIONS component. You can see the 
existing HTTPD package CDL for an example. I'm surprised this is all 
working correctly without having had to build the package with -D__ECOS in 
fact.

- A TCL interpreter! Bart will be pleased :).

- The MD5 code is under a license that users will need to be aware of, so I 
have documented this.

- I've removed the include of <cyg/sntp/sntp.h> as it appears to be unused.

That's it! I've checked it in with the above mods. Thanks again.

Jifl
--
eCosCentric    http://www.eCosCentric.com/    The eCos and RedBoot experts
------["The best things in life aren't things."]------      Opinions==mine