← Back to team overview

kicad-developers team mailing list archive

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