[CMake] Implicit toolchain file usage

[Laszlo Papp]

On Thu, Jan 17, 2013 at 8:43 PM, Alexander Neundorf <a.neundorf-work at gmx.net
> wrote:

> On Thursday 17 January 2013, Laszlo Papp wrote:
> > On Thu, Jan 17, 2013 at 8:31 PM, Alexander Neundorf
> > <a.neundorf-work at gmx.net
> >
> > > wrote:
> > >
> > > do you mean using -DCMAKE_SYSTEM_NAME=qnx should trigger something ?
> > > Don't know, my experience is that cross compiling environments usually
> > > vary a
> > > lot in their setup.
> >
> > *Current*
> >
> > cmake
> >
> -DCMAKE_TOOLCHAIN_FILE="../frontends/blackberry/cmake/Toolchain-QNX-8.0.0.c
> > make" -DBUILD_WIKIREADER_BLACKBERRY=ON ..
> >
> > ==========
> >
> > *Proposed*
> >
> > cmake -DBUILD_WIKIREADER_BLACKBERRY=ON ..
> >
> > and
> >
> > ../CMakeListst.txt:
> >
> set_toolchain_file("../frontends/blackberry/cmake/Toolchain-QNX-8.0.0.cmake
> > ")
> >
> > This is pseudo code, but you get my point, don't you? It would be shorter
> > for the developer, packager or user to use cmake.
> >
> > > Can you attach your toolchain file here so we can have a look ?
> >
> >
> http://quickgit.kde.org/?p=scratch%2Flpapp%2Fwikireader.git&a=blob&h=2e3e19
> >
> 7590bfdce77ae282be840d7b167afafb94&hb=7bf46fccfa4c087500a12116588e071b1183b
> > 4e7&f=frontends%2Fblackberry%2Fcmake%2FToolchain-QNX-8.0.0.cmake
>
>
> oh, this is definitely more than what should be necessary.
>
> Why do you have to set all the suffixes and prefixes ?
>

I will clean it up later once the project works for an initial release.


> This should go into Platforms/QNX.cmake.
>
> Manually setting CMAKE_AR etc. should also not be necessary, this should
> also
> go into QNX.cmake or QNX-qcc.cmake or somewhere like this.
>

Feel free to get anything upstreamed from my file.


> The specific build flags are your choice.
>

I see. It means I can avoid the -DCMAKE_TOOLCHAIN_FILE in the end of the
day.

Thank you for your reply.

Laszlo
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.cmake.org/pipermail/cmake/attachments/20130118/61ed6e93/attachment.htm>