kicad-developers team mailing list archive
  
  - 
     kicad-developers team 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