CHM and related fixes #577
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Regression introduced in commit 24e526e
|
Excellent, it builds now. But I do get some warnings and errors: |
|
Btw, the makefile's clean target includes: I think this is unnecessary, it makes it hard to do a clean build of chm without having to copy all of those phobos html files all over again. Actually it would be nice if chmgen looked for the Phobos HTML files in a path specified by an environment variable so we didn't have to copy the files around. |
|
They sort of have to be there for the site structure to actually work when on your hard drive. I suggest using a symlink / directory junction, or overriding the DOC variable when building Phobos/Druntime docs. |
|
I'm not sure what is the cause for those error messages, but I suspect stale .html files in your checkout (probably which no longer correspond to a .dd file). Try using The warnings are more or less expected. I think they all indicate valid (albeit minor) issues with the documentation, whether it's orphaned pages which are no longer reachable via navigation, redirect pages that really should be redone as a HTTP 30x redirect, or other such things. |
ghost
merged commit 
This pull request was closed.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
https://issues.dlang.org/show_bug.cgi?id=12717