You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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.
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.
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.
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.
Activity
SergeyRyabinin commentedon Feb 27, 2025
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 commentedon Feb 28, 2025
Thanks for sharing the details of the past reject. I understand your judgement.
Sure. I'll provide a GitHub Actions workflow for it. It'll explain what you need for your internal CI build system.
kou commentedon Feb 28, 2025
Could you check #3319?
SergeyRyabinin commentedon Mar 3, 2025
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 commentedon Mar 3, 2025
Thanks!
If you have any question, please let me know.
kou commentedon Apr 1, 2025
Is there any progress?
SergeyRyabinin commentedon Apr 4, 2025
Hi @kou ,
Please sorry for a delayed response.
I will try to work on this item next week.
Best regards,
S
kou commentedon Apr 5, 2025
Thanks!!!
Fixes for building with mingw #3319 #3315
Fixes for building with mingw #3319 #3315
Fixes for building with mingw #3319 #3315
Fixes for building with mingw #3319 #3315
Fixes for building with mingw #3319 #3315 (#3385)
kou commentedon May 10, 2025
Can we close this (and #3319) because #3385 was merged?
(Thanks!)
kai-ion commentedon Sep 23, 2026
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.