Linking errors with _MCF things post v1.10
Closed this issue · 3 comments
Thanks for this action, we use it on our project quiche. FYI something seems to have broken lately in brechtsanders/winlibs_mingw#147. We're working around that for now by pinning at v1.10, which doesn't show the problem.
Thanks for the heads up.
I don't want to be deciding which versions people should or shouldn't use so I'd rather not do a revert. I guess one possible policy would be to update the default winlibs tag a week or so after the corresponding winlibs release and update only if there are no regressions reported on winlibs's issue tracker rather than immediately (or as close to as I can get).
That said, is this really is all it takes to reproduce the error? If it is, I think I'll revert anyway. Edit: I forgot the part about requiring -static
. That makes me a lot less interested in a revert.
We're working around that for now by pinning at v1.10, which doesn't show the problem.
You can just pin the previous WinLibs tag instead (i.e. tag: 12.2.0-16.0.0-10.0.0-msvcrt-r5
) although in this case it doesn't make any difference since all setup-winlibs-action@v1.11 changed was that default WinLibs tag.
I don't want to be deciding which versions people should or shouldn't use so I'd rather not do a revert. I guess one possible policy would be to update the default winlibs tag a week or so after the corresponding winlibs release and update only if there are no regressions reported on winlibs's issue tracker rather than immediately (or as close to as I can get).
This is totally up to you. We have a way to workaround new things breaking stuff and that's good enough for my needs.
You can just pin the previous WinLibs tag instead (i.e. tag: 12.2.0-16.0.0-10.0.0-msvcrt-r5) although in this case it doesn't make any difference since all setup-winlibs-action@v1.11 changed was that default WinLibs tag.
Ah thanks for the tip. I was just following the instructions at https://github.com/bwoodsend/setup-winlibs-action#versioning, which implied we should do both and I didn't look any further into it.
I think we can call this sufficiently historical to be closable...