mingw-w64 support

[kou]

Describe the feature

#1836 was raised and rejected in 2022. Is this policy still valid?

MSYS2 has patches for this:

• https://github.com/msys2/MINGW-packages/blob/master/mingw-w64-aws-sdk-cpp/0001-fix-dll-import-libary-folder.patch
• https://github.com/msys2/MINGW-packages/blob/master/mingw-w64-aws-sdk-cpp/aws-sdk-cpp-pr-1333.patch

Use Case

I'm an Apache Arrow https://arrow.apache.org/ developer.

Apache Arrow uses aws-sdk-cpp for S3 support. Apache Arrow uses system aws-sdk-cpp or bundled aws-sdk-cpp by ExternalProject_Add()/FetchContent.

Apache Arrow supports mingw-w64 based environments. If aws-sdk-cpp supports mingw-w64, Apache Arrow don't need to apply patches with ExternalProject_Add()/FetchContent.

Proposed Solution

How about merging the patches in MSYS2?

Other Information

No response

Acknowledgements

• I may be able to implement this feature request
• This feature might incur a breaking change

[SergeyRyabinin]

Hi @kou,
Thank you for submitting this issue and thank you for providing this patches.
We are not against adding mingw support. The main reason the previous PRs were not merged is because we don't have this platform on our internal CI build system, so we were not able to ensure we would not break it, so we could not claim this target as supported.

Would it be possible for you to provide an example how you build the SDK on mingw-w64? I.e. the set of commands executed, and the prerequisites required so we could reproduce the build.

Best regards,
Sergey

[kou]

Thanks for sharing the details of the past reject. I understand your judgement.

Would it be possible for you to provide an example how you build the SDK on mingw-w64? I.e. the set of commands executed, and the prerequisites required so we could reproduce the build.

Sure. I'll provide a GitHub Actions workflow for it. It'll explain what you need for your internal CI build system.

[kou]

Could you check #3319?

[SergeyRyabinin]

Hi!
Thank you a lot for submitting this sample PR. I'll use it as an example while adding the mingw support.

Best regards,
Sergey

[kou]

Thanks!

If you have any question, please let me know.

[kou]

Is there any progress?

[SergeyRyabinin]

Hi @kou ,

Please sorry for a delayed response.
I will try to work on this item next week.

Best regards,
S

[kou]

Thanks!!!

[kou]

Can we close this (and #3319) because #3385 was merged?
(Thanks!)

[kai-ion]

Converting this to a Discussion. It's an improvements we plan to pursue over time rather than one actionable change with a near-term milestone, so a Discussion better fits collecting the specifics and design input.