kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #23127
Re: [PATCH v2 0/3] Correctly split patch
Hi Wayne,
Am 15.02.2016 um 17:17 schrieb Wayne Stambaugh:
> Have you tested this patch on MSYS and/or MSYS2?
Yes, I've tested that with MSYS2. The difference to the previous patch
are the parts that went to the cmake developers list:
- fix the case where multiple CXXFLAGS that are neither -D nor -I are
returned from wx-config
- use cygpath to translate from weird hybrid path (C:/msys64/...) to
Windows path (C:\msys64\...).
This means we still have a few changes that aren't in upstream, and the
CMake people seem unsure whose fault it is -- the MSYS people
deliberately changed wx-config to return hybrid paths instead of Unix
style (which probably explains why the MSYS support in the upstream
FindwxWidgets.cmake does not work -- it used to before).
> The last time we went through this exercise, FindwxWidgets.cmake was broken on msys builds.
Indeed, but we can now ask Jenkins to test a patch.
> Also, do I apply the first
> 9 patches than the 3 patches from this set and should I apply them all
> and just have single commit which seems to make the most sense.
The "v2" set replaces the incorrectly split [2/19] patch. I can prepare
a branch if that is easier.
Simon
Attachment:
signature.asc
Description: OpenPGP digital signature
Follow ups
References