Hi Jean-Pierre,
On Tue, Nov 3, 2015 at 4:41 AM, Jean-Pierre Tosoni jp.tosoni@acksys.fr wrote:
Hi all,
-----Message d'origine----- De : Joe Hershberger [mailto:joe.hershberger@gmail.com] Envoyé : lundi 2 novembre 2015 21:44 À : Chris Packham Cc : u-boot; Joe Hershberger; jp.tosoni@acksys.fr; Hanna Hawa; Angga Objet : Re: [U-Boot] [RFC PATCH 6/8] net: TFTP over IPv6
(...)
One problem with the [hostIpAddr:]fileName syntax is that IPv6 addresses contains colons. So tftp_start() would be confused by 'tftpboot6 $loadaddr 2001:db8::1:zImage'. It is probably possible to change the parsing to separate the host from the filename by parsing from the end (i.e. use strrchr() instead of strchr()) but then there are error cases that may not be handled correctly (e.g. omitting the
filename).
I think we should just change the filename separator for tftp6. How about ','?
How about a '/' to make it look like an URL ? More intuitive maybe.
That might work, but it also could be part of the path to the file itself. It doesn't seem good to be ambiguous about that or to require 2001:db8::1//kernel.tgz
-Joe