Skip to content
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

posix.mak: Update MOD_EXCLUDES lists #1265

Merged
merged 1 commit into from
May 18, 2016

Conversation

CyberShadow
Copy link
Member

@CyberShadow
Copy link
Member Author

_PRERELEASE / _RELEASE macros seem like an additional point of failure for the release process. Perhaps they should be named according to the versions they imply (e.g. MOD_EXCLUDES_2071) so that the build will fail if they are not updated.

@@ -36,11 +36,11 @@ STABLE_RDMD=$(STABLE_DMD_ROOT)/dmd2/$(OS)/$(if $(filter $(OS),osx),bin,bin$(MODE

# exclude lists
MOD_EXCLUDES_PRERELEASE=$(addprefix --ex=, gc. rt. core.internal. core.stdc.config core.sys. \
std.c. std.algorithm.internal std.internal. std.regex.internal. std.typelist \
std.c. std.algorithm.internal std.internal. std.regex.internal. \
std.windows.iunknown std.windows.registry etc.linux.memoryerror std.stream std.cstream \
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Somewhat off topic, but why is std.windows.registry excluded? It doesn't have a deprecation note. (The other excluded modules are correct).

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not at all off-topic. It's excluded because it's a Windows-only module, and unlike the other Windows-only modules not excluded here, it doesn't have a version(StdDdoc) block which only has the documentation.

@schuetzm schuetzm merged commit 9edc1a2 into dlang:master May 18, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants