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

[dmd-cxx] Backport fixes from #6752 and #7196 #7486

Merged
merged 2 commits into from
May 17, 2020

Conversation

ibuclaw
Copy link
Member

@ibuclaw ibuclaw commented May 17, 2020

No description provided.

@dlang-bot
Copy link
Contributor

Thanks for your pull request, @ibuclaw!

Bugzilla references

Auto-close Bugzilla Severity Description
19367 minor std.net.curl does not understand HTTP/2 status lines

Testing this PR locally

If you don't have a local development environment setup, you can use Digger to test this PR:

dub run digger -- build "dmd-cxx + phobos#7486"

@n8sh n8sh changed the title [dmd-cxx] Backport fixes from #6752 and #7126 [dmd-cxx] Backport fixes from #6752 and #7196 May 17, 2020
@ibuclaw
Copy link
Member Author

ibuclaw commented May 17, 2020

Bootstrapped, running regression tests...

Copy link
Member

@Geod24 Geod24 left a comment

Choose a reason for hiding this comment

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

Thanks!

@ibuclaw
Copy link
Member Author

ibuclaw commented May 17, 2020

Regression tests ran, no reported failures.

@ibuclaw ibuclaw merged commit 64ed468 into dlang:dmd-cxx May 17, 2020
@ibuclaw ibuclaw deleted the dmd-cxx-7196 branch May 17, 2020 16:33
kraj pushed a commit to kraj/gcc that referenced this pull request May 17, 2020
- core.cpuid has been fixed to not use i7 detection on AMD processors.
- std.net.curl has been fixed to correctly handle HTTP/2 status lines.
- std.zip has had a test fixed to not rely on unzip being installed.

Fixes: PR d/95166
       PR d/95167
       PR d/95168

Reviewed-on: dlang/druntime#3107
	     dlang/phobos#7486
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants