Per Einar Ellefsen wrote:
Ok, here's the deal:

download/binaries.pod links to docs::1.0::os::index. However, that link turns up as broken.

I have tracked the problem down the the creation of %src_docs. If you look at the dump, you'll see that docs/1.0/os/win32/index.html is in there, but docs/1.0/os/index.html isn't. This is a bug, because win32/ isn't a docset, but os/ is.

It seems like the problem lies here: (DocSet::RunTime, line 110):
map { m{(.*?/?)[^/]+$} # add autogenerated index.html
? ("$1index.html" => 1, $_ => 1)
: ($_ => 1); # shouldn't happen, but just in case
}
From this, the problem is that it just takes the longest path and appends index.html, without checking if it's a DocSet or not... I'm not sure how to fix this, as I suppose it would require checking up with the other DocSet classes to see the status of that directory...


I can fix this quickly by making win32 a docset and linking to that (which I will probably do anyway); however, that isn't a long-term solution.

See if I've fixed it. Actually while fixing I've greatly simplified the code and found at least one bug (now fixed) :)


I've simply mapped all config.cfg to index.html, that should be the right thing. Obviously the previous solution was just a spaghetti :)

__________________________________________________________________
Stas Bekman            JAm_pH ------> Just Another mod_perl Hacker
http://stason.org/     mod_perl Guide ---> http://perl.apache.org
mailto:[EMAIL PROTECTED] http://use.perl.org http://apacheweek.com
http://modperlbook.org http://apache.org   http://ticketmaster.com


--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]



Reply via email to