From kde-bindings Sat Mar 02 14:36:03 2013 From: Arno Rehn Date: Sat, 02 Mar 2013 14:36:03 +0000 To: kde-bindings Subject: Re: [Kde-bindings] [Qyoto] New Build Instructions for Linux, Windows MinGW and Windows Visual Studio Message-Id: <51320E53.8000906 () arnorehn ! de> X-MARC-Message: https://marc.info/?l=kde-bindings&m=136223498227485 On 01/03/13 17:51, David Lechner wrote: > On 3/1/2013 4:54 AM, Arno Rehn wrote: >>> I have one change to smokegen that is needed to get the Visual Studio >>> builds working: >>> https://github.com/dlech/smokegen/commit/de32c886c03b2f8cfe1afc63f8cda188e7b85e19 >>> >>> >> I'm unsure about this. AFAIK the default on windows is: >> .dlls and .exes go into bin/, .lib, .a go into lib/. I'm not entirely >> sure though; I'll look it up. > This is correct. And, we really want the .a or the .lib file and not the > .dll here. It seems that MinGW is smart enough to find the .a in lib/ > even if you point it to the .dll in bin/. You can see that this is what > it is doing if you turn on CMAKE_VERBOSE_MAKEFILE. So, MinGW works > either way (searching for libs in bin/ or lib/). Visual Studio, on the > other hand is not smart enough and you must point it to lib/. Since > searching in lib/ works in all cases, I don't see a reason for searching > in bin/. > > By the way, I should have mentioned in the commit message that this is a > partial revert of aaaa8f289f15a1e0e60d595bb38fc25089ac6e0d Right, then I'll pull it :) >> >>> Also one for smoke qt that makes windows builds (both MinGW and VS) work >>> with less manual configuration: >>> https://github.com/dlech/smokeqt/commit/aee4734d50456b2bcd4893137d2dba1d1ff2e8b1 >>> >>> >> This would however defeat the purpose of CMAKE_INSTALL_PREFIX. Can't you >> just set the same prefix for all smoke related projects? >> > > I see. My goal is to make as little manual configuration needed as > possible. So, how about this instead: > https://github.com/dlech/smokeqt/commit/38f0b45480c67b639ea77b71e94ae20e7811ea5b > > > It sets CMAKE_INSTALL_PREFIX to ${SMOKE_INSTALL_PREFIX} unless the user > specifies their own CMAKE_INSTALL_PREFIX. It has the same effect as the > other patch, but does not break CMAKE_INSTALL_PREFIX. This looks good :) I'll pull it. Thanks for the patches! -- Arno Rehn _______________________________________________ Kde-bindings mailing list Kde-bindings@kde.org https://mail.kde.org/mailman/listinfo/kde-bindings