Conversation
|
Hm I still have build errors with that in comparison to #18175. build-x64-mingw-static-dbg-out.logbuild-x64-mingw-static-dbg-err.log |
|
I also have found https://github.com/msys2/MINGW-packages/tree/master/mingw-w64-icu which can be a useful reference for patches |
Basically, it is using unprotected backslash as a path separator in the failing bash command line. (Of course, it doesn't happen on my Linux box, but I will retry on Windows once more.) |
|
@autoantwort Both x64-mingw-static and x64-mingw-dynamic build for me also on Windows, host triplet == target triplet. |
|
Yeah I have not explicitly set the host triplet, so it is x64-windows for me |
So this is the difference... |
|
Yeah seems that this is the problem. Would be good if it works even when you don't change the host triplet, but imho this is not so important. Imho we need a better host triplet selection. |
|
#18175 has changes around
Could this change be the solution? I cannot test this, due to absence of a windows triplet toolchain. |
Probably, ist schon etwas länger her. I can test it |
|
Yeah that fixes the issue |
|
When this is merged, I can finally submit changes for building Qt5 with mingw, on Windows... |
|
Cross-compiling is a bit muddy right now, due to |
Okay. But this is unrelated to this PR. This is for host == target == "mingw". |
BTW, I'm handling the issue about building
|
|
I'm guessing we have to replace |
(Ba)sh doesn't mess with |
@JackBoosY Double quotes within an argument call for trouble, in particular in a Windows + MSYS environment. It means different things to Windows and POSIX. Whenever possible, quote full parameters. |
|
@longnguyen2004 @dg0yt Also, it doesn't work: |
@JackBoosY Is this related to this PR, #20253? |
|
@dg0yt Nope, I just found that in my PR #20165. |
|
@JackBoosY I would prefer to discuss #20165 issues in #20165. |
|
Thanks for MinGW-ing :) |
|
is there anything here that should be upstreamed to ICU? unicode-org/icu#1733 is there but the author is not responding. |
|
Some filesystem layout patterns might be specifc for vcpkg. |
Describe the pull request
What does your PR fix?
Fixes building icu with mingw, for dynamic and static linkage. A more lightweight alternative to [icu] fix mingw builds #18175 by @autoantwort.
bindirs with dlls).sinto statc library names. This fixes mingw-static ([icu] mingw build failure #16663), and allows to simplify the portfile.Which triplets are supported/not supported? Have you updated the CI baseline?
all, no.
Tested locally: x64-mingw-dynamic and -static on Linux and on Windows (with mingw host triplet).
Does your PR follow the maintainer guide?
yes
If you have added/updated a port: Have you run
./vcpkg x-add-version --alland committed the result?yes