[Kde-bindings] [Qyoto] New Build Instructions for Linux, Windows MinGW and Windows Visual Studio

[Arno Rehn]

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