Closed
Bug 629922
Opened 13 years ago
Closed 13 years ago
None of the modules' documentation loads
Categories
(Add-on SDK Graveyard :: Documentation, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: KWierso, Assigned: wbamberg)
References
()
Details
Attachments
(1 file)
1.42 KB,
patch
|
myk
:
review+
|
Details | Diff | Splinter Review |
When loading the documentation from that URL, I get the following error for every single module: Parse Error in docs/***.md <!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN"> <html><head> <title>404 Not Found</title> </head><body> <h1>Not Found</h1> <p>The requested URL /sdk/1.0b2/docs/packages/api-utils/docs/app-strings.md.div was not found on this server.</p> <hr> <address>Apache/2.2.3 (Red Hat) Server at jetpack.mozillalabs.com Port 80</address> </body></html> (Where "***" is the name of the module) The 1.0b1 docs still load just fine.
Comment 1•13 years ago
|
||
Will, do you know what might be going on here? It appears to cause errors on the docs hosted on the website: https://jetpack.mozillalabs.com/sdk/1.0b2/docs/#module/addon-kit/clipboard
Assignee | ||
Comment 2•13 years ago
|
||
I'm looking at it. It's a bit hard to tell from that error message alone. It looks as if the js in the browser is sending a URL to the server that the server's not able to recognize as a request for a generated piece of a module doc, so it's just treating it as a request for a file. Which fails of course because the file doesn't exist.
Assignee: nobody → wbamberg
Assignee | ||
Comment 3•13 years ago
|
||
Do you know what the process is of taking an SDK release and running the doc server on mozillalabs.org? Could you possibly help me to get a similar environment running here? Without being able to reproduce it here, it's hard to do more than speculate. What should happen: - main.js should look at '#module/addon-kit/clipboard' and end up asking the server for '/packages/addon-kit/docs/clipboard.md.div'. - the server should recognize that it's being asked for a package or module at from the fact that the first part of the path is 'packages' (server.py, lines 305-6) - the server should recognize that it's being asked for a DIV from the extension (line 220) and generate and return that from a Markdown file located at: <root_dir>/docs/clipboard.md, where root_dir is initialized from the place it found the package: for example '/Users/mozilla/addon-sdk-1.0b2/packages/addon-kit'
Comment 5•13 years ago
|
||
Zandr: do you have any thoughts on why this might be happening? Will is having trouble reproducing it (see comment 3).
Comment 6•13 years ago
|
||
The docs directory on the server is created by running 'cfx sdocs' and uploading the resulting tarball. A quick look at the resultant directory shows (to use clipboard as an example): clipboard.md clipboard.md.div.html clipboard.md.html clipboard.md.json So, we don't have clipboard.md.div Is sdocs not building the right things?
Assignee | ||
Comment 7•13 years ago
|
||
Thanks Zandr. cfx sdocs is building the right things, but giving them the wrong names :-(
Assignee | ||
Comment 8•13 years ago
|
||
cfx sdocs called the DIVs it exported '<module_name>.div.html', but main.js was asking for '<module_name>.div'. This patch changes the names of the exported files in line with what main.js expects. If we wanted to fix the 1.0b2 release without changing the code, renaming the files on the server should also work.
Attachment #508616 -
Flags: review?(myk)
Comment 9•13 years ago
|
||
Just for the sake of reproducibility, I'd like to re-run cfx sdocs from a specified tag. After review, could I get such a thing? Once I have that I'll re-upload the docs tarball.
Comment 10•13 years ago
|
||
Comment on attachment 508616 [details] [diff] [review] Fix names of DIVs exported by sdocs Will: after you check this in, I'll cherry-pick the change to the 1.0b2 branch and then give Zandr a reference to pull.
Attachment #508616 -
Flags: review?(myk) → review+
Assignee | ||
Comment 11•13 years ago
|
||
Checked in as: https://github.com/mozilla/addon-sdk/commit/d1c70ea4ea0e22efedac3e01c994c6a8874b84e0
Comment 12•13 years ago
|
||
Resolved by checkin in comment 11.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Comment 13•13 years ago
|
||
Cherry-picked as: https://github.com/mozilla/addon-sdk/commit/961ddbe12bc581f7162ae8b2ea4b3e01e0ee95c9
Comment 14•13 years ago
|
||
sdocs regenerated and uploaded. Looks to be working, so marking Verified.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•