Closed GoogleCodeExporter closed 9 years ago
Another example: MinGW with GCC 4.4 (see issue 298).
Original comment by vladlosev
on 7 Jul 2010 at 6:08
Instead of complicating the cmake script, I'd prefer to hard code the disabling
of threading in gtest-port.h for such obscure platforms.
Original comment by w...@google.com
on 27 Sep 2010 at 7:18
The way to control the compiler settings in a CMake setup is through the CMake
script. How else do you plan to pass them to the compiler when using CMake?
And the changes to the script do not have to be too big. All you have to do is
to add an option and condition the detection invocation based on that option.
It will take a total of 4 lines of code; more with comments.
Original comment by vladlosev
on 27 Sep 2010 at 10:48
I wouldn't think MinGW with pthreads is a terribly obscure platform; TDM-GCC
<http://tdm-gcc.tdragon.net/> installs pthreads by default even (you can't use
GCC's OpenMP support without it).
Original comment by ian.cull...@uqconnect.edu.au
on 27 Sep 2010 at 10:56
As said, I'd prefer to *hard-code* the disabling of pthreads in gtest-port.h,
like we did for many other capabilities.
People want gtest to work out-of-the-box. Hard-coding the logic in
gtest-port.h means that most users don't have to specify any cmake flags.
Every option we add to the cmake script increases the cognitive burden on the
users.
It may be that eventually we'll have to add a flag -- I don't know yet. But
before that day comes, I'd like to keep it simple.
Original comment by w...@google.com
on 28 Sep 2010 at 12:08
@cibyr, I was saying that we should make gtest work out-of-the-box on MinGW,
instead of requiring the user to specify any build option. The "won't fix"
refers to "need pthreads detection suppression in cmake build script". I'll
revive 298 to track the fix for mingw, etc.
Original comment by w...@google.com
on 28 Sep 2010 at 12:11
Original issue reported on code.google.com by
vladlosev
on 27 Mar 2010 at 10:32