Skip to content

leave it to user to build d-tags.json#265

Merged
dlang-bot merged 1 commit intodlang:masterfrom
MartinNowak:release_docs
Dec 7, 2017
Merged

leave it to user to build d-tags.json#265
dlang-bot merged 1 commit intodlang:masterfrom
MartinNowak:release_docs

Conversation

@MartinNowak
Copy link
Copy Markdown
Member

@MartinNowak MartinNowak commented Oct 12, 2017

  • fixes remaking dlang.org when tool's Makefile is used by build scripts
  • allows to build dman for prereleases, latest, or releases

Win rule untested!

@dlang-bot
Copy link
Copy Markdown
Contributor

Thanks for your pull request, @MartinNowak!

Bugzilla references

Your PR doesn't reference any Bugzilla issue.

If your PR contains non-trivial changes, please reference a Bugzilla issue or create a manual changelog.

@CyberShadow
Copy link
Copy Markdown
Member

A tiny bit sad about the removal of the implicit rule to build d-tags.json, as it did the sufficiently-correct thing in many cases and saved a manual step for fresh clones, but definitely worth it if it makes the release process simpler.

The Jenkins failure looks unrelated. LGTM.

- fixes remaking dlang.org when tool's Makefile is used by build scripts
- allows to build dman for prereleases, latest, or releases
@wilzbach
Copy link
Copy Markdown
Contributor

wilzbach commented Dec 7, 2017

Rebased and force-pushed to retrigger all CIs

@dlang-bot dlang-bot merged commit 20719a4 into dlang:master Dec 7, 2017
@MartinNowak MartinNowak deleted the release_docs branch December 12, 2017 10:34
MartinNowak added a commit to MartinNowak/installer that referenced this pull request Dec 12, 2017
- no longer needed since dlang/tools#265
MartinNowak added a commit to MartinNowak/installer that referenced this pull request Dec 12, 2017
- no longer needed since dlang/tools#265
MartinNowak added a commit to MartinNowak/installer that referenced this pull request Dec 12, 2017
- no longer needed since dlang/tools#265
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants