This is the mail archive of the
ecos-patches@sources.redhat.com
mailing list for the eCos project.
Re: Added unicast support to SNTP client
On Thu, Oct 02, 2003 at 03:41:00PM -0700, Dan Jakubiec wrote:
> Hi Gary,
>
> No problem: I've attached the same SNTP patch, but
> this time using 'cvs diff -u'.
Index: packages/net/sntp/current/ChangeLog
===================================================================
RCS file: /cvs/ecos/ecos-opt/net/net/sntp/current/ChangeLog,v
retrieving revision 1.6
diff -u -r1.6 ChangeLog
--- packages/net/sntp/current/ChangeLog 21 May 2003 14:03:36 -0000 1.6
+++ packages/net/sntp/current/ChangeLog 2 Oct 2003 22:36:41 -0000
@@ -1,3 +1,9 @@
+2003-09-29 Dan Jakubiec <firstname.lastname@systech.com>
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
You need to set up emacs to fill in your email address correctly.
This has also been committed to CVS!
+
+ * src/sntp.c: Added support for SNTP unicast mode.
+ * src/sntp.h: Added support for SNTP unicast mode.
+ * src/sntp.cdl: Added support for SNTP unicast mode.
+
2003-05-19 Andrew Lunn <andrew.lunn@ascom.ch>
+/* General Definitions */
+#define NTP_UDP_PORT 123
+
I don't like this. Its not used anywhere in your code and you should
be using getaddrinfo, or getservbyname. Both know about ntp.
Please could you also extend the test program. What i suggest it to
extend the current test program so that after its finished the current
test it sets the time back to the epoc and then sends a query to the
server in the bootp record. After 20 seconds, sanity check the time in
the same way the current test does.
Please could you also update the SGML documentation.
Thanks
Andrew