On 13 Apr 2010 18:19:00 +0100, Chris Young wrote: > On Tue, 13 Apr 2010 11:31:31 +0100, Daniel Silverstone wrote: > > > On Mon, Apr 12, 2010 at 11:48:30PM +0100, John-Mark Bell wrote: > > > In theory, this patch fixes handling of file: URIs. > > > It behaves correctly with the RISC OS, GTK, and framebuffer frontends. > > > It needs testing with Amiga, BeOS, and Windows frontends. > > Just checking the Amiga frontend now, will report back.
Well, it works, sort of. The core CURL file fetcher still has the
same problem as before (tries to open all paths with a leading slash),
and file://localhost/ is no longer working either (unfortunately a lot
of local HTML documentation expects this, I'd happily ditch it
otherwise)
The localhost problem can be solved by modifying amiga/misc.c
url_to_path as follows:
if (strncmp(url, "file://", SLEN("file://")) != 0)
return NULL;
url += SLEN("file://");
if (strncmp(url, "localhost", SLEN("localhost")) == 0)
url += SLEN("localhost");
url += SLEN("/");
(this could be further improved by skipping anything between the
second and third slash I guess, as that can't be a path now as there
is no translation incorrectly changing it)
Not sure on the CURL file fetcher though, no matter how many
characters I skip in url_to_path it still tries to open /PROGDIR: -
looks like it's not even calling url_to_path?
Chris
ns.log.gz
Description: Binary data
