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

Support for MinGW on Windows #1492

Merged
merged 13 commits into from
Apr 1, 2024
Merged

Support for MinGW on Windows #1492

merged 13 commits into from
Apr 1, 2024

Conversation

justinwsmith
Copy link
Contributor

@justinwsmith justinwsmith commented Mar 13, 2024

Issues

Description of changes:

  • Attempting to build using gcc/mingw surfaced a few minor issues.
  • Adding build/testing of Windows MinGW to our CI tests

Testing

All tests passed on my Windows Server 2022 Datacenter EC2 instance.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and the ISC license.

@justinwsmith justinwsmith requested a review from a team as a code owner March 13, 2024 04:07
@justsmth justsmth marked this pull request as draft March 13, 2024 15:15
@codecov-commenter
Copy link

codecov-commenter commented Mar 13, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 76.98%. Comparing base (1e4601e) to head (e536203).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1492      +/-   ##
==========================================
+ Coverage   76.96%   76.98%   +0.01%     
==========================================
  Files         426      426              
  Lines       71738    71738              
==========================================
+ Hits        55211    55225      +14     
+ Misses      16527    16513      -14     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@justsmth justsmth force-pushed the support-mingw branch 10 times, most recently from 462e490 to fd4a535 Compare March 14, 2024 13:11
@justsmth justsmth changed the title Fix platform macro usage Support for MinGW on Windows Mar 14, 2024
@justsmth justsmth marked this pull request as ready for review March 14, 2024 16:12
@justsmth justsmth requested a review from andrewhop March 14, 2024 16:33
ssl/ssl_test.cc Outdated Show resolved Hide resolved
.github/workflows/mingw.yml Show resolved Hide resolved
.github/workflows/mingw.yml Outdated Show resolved Hide resolved
@justsmth justsmth force-pushed the support-mingw branch 4 times, most recently from 350c47c to b3f1972 Compare March 19, 2024 12:55
@nebeid nebeid requested a review from skmcgrail March 19, 2024 20:01
Copy link
Contributor

@andrewhop andrewhop left a comment

Choose a reason for hiding this comment

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

Everything looks good, just the one question on push or not push.

.github/workflows/mingw.yml Show resolved Hide resolved
skmcgrail
skmcgrail previously approved these changes Mar 21, 2024
@justsmth justsmth enabled auto-merge (squash) March 22, 2024 00:57
auto-merge was automatically disabled March 22, 2024 02:21

Head branch was pushed to by a user without write access

@justinwsmith justinwsmith force-pushed the support-mingw branch 4 times, most recently from 071d349 to 74b4021 Compare March 22, 2024 03:28
@justsmth justsmth force-pushed the support-mingw branch 2 times, most recently from 65b2776 to d44e08a Compare March 22, 2024 17:20
@justsmth justsmth enabled auto-merge (squash) March 22, 2024 19:40
@justsmth justsmth force-pushed the support-mingw branch 2 times, most recently from 17ac61d to c39d658 Compare March 22, 2024 20:54
auto-merge was automatically disabled March 22, 2024 21:41

Head branch was pushed to by a user without write access

@justsmth justsmth enabled auto-merge (squash) March 22, 2024 22:22
@justsmth justsmth merged commit 85ce380 into aws:main Apr 1, 2024
43 of 45 checks passed
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.

5 participants