Alexander Neundorf wrote:
John Fine wrote:
It is very simple and not a compiler bug, and you are correct that the
error is in GetRPath.

The code is

  if(this->Valid() &&
     this->Internal->GetFileType() == cmELF::FileTypeExecutable ||
     this->Internal->GetFileType() == cmELF::FileTypeSharedLibrary)

Notice the lack of () around the || lines

If I understand correctly this is fixed in cmake cvs and will be in 2.6.2:
http://www.cmake.org/cgi-bin/viewcvs.cgi/Source/cmELF.cxx?root=CMake&r1=1.12&r2=1.13

Patch looks right to me, thanks Alex! This means Fedora will be able to drop the patch when 2.6.2 is out (always nice) :-).

--
Matthew
This message represents the official view of the voices in my head.
  -- Unknown
(found at http://goldmark.org/jeff/stupid-disclaimers/fun.html)

_______________________________________________
CMake mailing list
[email protected]
http://www.cmake.org/mailman/listinfo/cmake

Reply via email to