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.
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
d.tag #900
d.tag #900
Conversation
a44dbe4
to
4a8fc17
Compare
Thanks, needs a rebase though. |
4a8fc17
to
13229c1
Compare
Rebased. |
chmgen : chmgen.d $(DMD) | ||
$(DMD) -I${PHOBOS_DIR} -g chmgen.d | ||
|
||
d.tag : chmgen $(ALL_FILES) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm having a race condition because this target depends on the druntime/phobos documentation without stating so.
Would be fine to just add druntime-release and phobos-release as dependencies, or does it also depend on the prerelease stuff?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well, strictly speaking it doesn't care, as it will follow links recursively. But the CHM probably shouldn't contain prerelease documentation, hmm.
dlang/tools#156 (comment)