Bill,
Thanks for responding.
This problem should be fixed in CVS. It no longer adds both /TP and /TC.
Please try cvs before reporting a new bug, or opening an old one.
OK, that's what I did, hence my earlier message (quoted below). Since that
one got no replies, I figured I was being subtly encouraged to check out the
history/status of the bug/resolution via the Bug Tracker.
So, at this point, protocol-wise, I'm confused about the best and proper way
to
a) investigate the status/resolution of a previously reported bug
b) understand where and how a fix was made, both generally and for this
specific bug
c) understand when and why patch releases are prepared, or not
Maybe I've created a hairball of questions by this point, which wasn't my
intent.
I want to use the available resources efficiently and curteously - I'm
asking questions to understand how to do so.
I would welcome guidance on any/all of a) - c), in whole or in part.
Thanks!
- Steve
____
Earlier post:
Thank you for this info. I'm now looking at something that seems
relevant,
but this look has raised more questions in my mind.
I am currently running the binary distro of 2-4-2
Inside:
cmLocalVisualStudio7Generator.cxx
Revision: 1.125.2.3, Sun May 14 19:22:42 2006 UTC (6 weeks, 4 days ago) by
hoffman
Branch: CMake-2-4
CVS Tags: CMake-2-4-2
Changes since 1.125.2.2: +107 -62 lines
I find:
// if the source file does not match the linker language
// then force c or c++
if(linkLanguage && lang && strcmp(lang, linkLanguage) != 0)
{
if(strcmp(lang, "CXX") == 0)
{
// force a C++ file type
compileFlags += " /TP ";
}
else if(strcmp(lang, "C") == 0)
{
// force to c
compileFlags += " /TC ";
}
}
If I'm reading this right (?), this is the CAUSE of the problem - i.e.
this
is where the trailing " /TC" gets added at the end of the build command,
when lang == "C", which is when the file has a .c extension. Is this
right
so far?
then I find, in this later version, ( later than 2-4-2, meaning it's not
in
my CMake executable, yes?) :
cmLocalVisualStudio7Generator.cxx
Revision 1.135 - (view) (download) (as text) (annotate) - [select for
diffs]
Tue Jun 6 16:01:23 2006 UTC (3 weeks, 2 days ago) by hoffman
Branch: MAIN
ENH: fix /TP for c code
Is this the fix you refer to?
I looked at the diff for these two versions, and see that 1.135 adds code
down around line 1167 dealing with flags in some way that I can't quickly
decipher, but which has no immediately recognizable reference to "/TC " or
"/TP ".
If this is the fix, and it works, I'd be interested to understand how it
works.
If this is not the fix, I'm sorry for getting it wrong, and I'd welcome a
pointer to the correct revision.
Finally, it seems that binary distros are prepared relatively
infrequently.
Is getting this:
SET(CMAKE_C_COMPILER ${CMAKE_CXX_COMPILER})
SET_TARGET_PROPERITES( <tgt> PROPERTIES LINKER_LANGUAGE CXX)
to work as intended in these circumstances not judged important enough to
warrant a 2-4-3 build?
It would be helpful, if possible, to be able to refer my development
partner(s) to a binary distro of CMake that handles this situation.
If I have to bite the bullet and start compiling CMake's own source code,
and getting partner(s) to do same, I imagine we can and will, but if
generating a 2-4-3 binary at "home base" is not hard, perhaps it would be
worthwhile and save some people from extra work and possible issues in the
field.
_______________________________________________
CMake mailing list
[email protected]
http://www.cmake.org/mailman/listinfo/cmake
