Re: [ITP] tftp-hpa 5.0

[Gernot Hillier <gernot dot hillier at siemens dot com>]

Hi!

On 12.10.2010 20:29, Charles Wilson wrote:
> Here's my revised version of your package:
> 
> http://cygwin.cwilson.fastmail.fm/tftp-5.0-1-src.tar.bz2
> http://cygwin.cwilson.fastmail.fm/tftp-5.0-1.tar.bz2
> http://cygwin.cwilson.fastmail.fm/tftp-server-5.0-1.tar.bz2
[...]

> 7) Massive updating to README file; consolidated information from
>    README.cygwin and README.security changes, plus pulled in relevant
>    data from (current) inetutils.README (*)

Some minor comments about /usr/share/doc/Cygwin/tftp.README:

* confuses config-tftpd with tftpd-config in (at least) two places

* Under 3), it describes the manual way to install the standalone
  service instead of the much more convenient "tftpd-config
  --standalone" feature. This also leaves the user alone with the
  complicated decision whether he has to use cyg_server or not - while
  the script would do all this for him.

  Perhaps you could also add a hint that the user might want to
  change TFTPBOOT and args_value in the script before starting. Or
  perhaps you could even make it interactive?

* In one place, you reference /var/log/messages when syslog is running;
  you could add that the other place would be the Windows Event log when
  syslog is NOT running. I verified that those messages really appear
  there.

* The History misses your name. :-)

Apart from that, the new README looks *very* nice! Thanks for that - it
will save us a lot of user's questions!

> Gernot, if you could take this -src package and rebuild it (just to make
> sure it works for you), and then test the binaries in your environment,
> that'd be a good start.  Then, I'll worry about the IPv6 stuff next
> weekend or so, and we'll be ready to upload.

Build tested successfully.

>From my side: ready to upload after the minor wording changes as
discussed in the other mail and the bits in the README mentioned above.

-- 
Gernot Hillier
Siemens AG, CT T DE IT, GTF System Architecture & Platforms