Skip to content

html doc build for dmd #1196

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 4 commits into from
Apr 2, 2016
Merged

html doc build for dmd #1196

merged 4 commits into from
Apr 2, 2016

Conversation

andralex
Copy link
Member

Works in conjunction with dlang/dmd#5352.

@andralex andralex mentioned this pull request Jan 15, 2016
Merged
@andralex
Copy link
Member Author

@CyberShadow do you know why this might have happened? http://dtest.thecybershadow.net/results/57c4402ecd7285277ea0da611679fc83f4e490e0/dd10bb6137ed229bcb872c6611942ae183a0a031/build.log

@CyberShadow
Copy link
Member

I'm guessing you didn't specify whichever dependency generates .generated/2.069.2.ddoc for the dmd documentation target (${GENERATED}/${LATEST}.ddoc or one of the macros that contain it, ${DDOC} / ${STD_DDOC} / ${STD_DDOC_PRE}).

@andralex
Copy link
Member Author

Ah, it depends on the entire $(STD_DDOC_PRE). Fixed.

@andralex
Copy link
Member Author

@CyberShadow thx!!

@andralex
Copy link
Member Author

"All checks have passed"

@CyberShadow
Copy link
Member

Looks like the tools section is generating broken HTML

@adamdruppe
Copy link
Contributor

@andralex @CyberShadow so what's the status here?

@CyberShadow
Copy link
Member

The tools section is still generating broken HTML. Look at the top of any new generated file.

@CyberShadow
Copy link
Member

Seeing as Andrei seems to be busy with more important things, does anyone want to adopt this PR?

@andralex
Copy link
Member Author

andralex commented Apr 1, 2016

OK, fixed. This + dlang/dmd#5617 = win

@JackStouffer
Copy link
Member

Looking good 👍

This is missing one thing however, there needs to be a link somewhere on the site, perferably in the navbar, to these docs.

@WalterBright WalterBright merged commit 60378b9 into dlang:master Apr 2, 2016
@WalterBright
Copy link
Member

Here it is in all its glory: http://dlang.org/dmd-prerelease/

@JackStouffer
Copy link
Member

Better than nothing I guess

@CyberShadow CyberShadow mentioned this pull request Jul 26, 2017
Merged
@andralex andralex deleted the dmddoc branch December 11, 2017 02:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@andralex @CyberShadow @adamdruppe @JackStouffer @WalterBright