[CMake] 2.6.0-rc8 install problem on darwin

[Mike Jackson]

On Apr 14, 2008, at 9:44 PM, Bill Hoffman wrote:
> I wonder if it is a permission problem.  I remove all the files  
> before I put a new link in.
>
>   // Remove the old files
>     if(cmSystemTools::FileExists(newName.c_str()))
>       {
>       std::cout << "rm [" << newName << "]\n";
>       cmSystemTools::RemoveFile(newName.c_str());
>       }
>
> You can test it with the menu item from the gui that installs the  
> command line tool.  When I tried it was able to remove all links.


Just an observation but shouldn't that code be something like:

   // Remove the old files
     if(cmSystemTools::FileExists(newName.c_str()))
       {
       std::cout << "rm [" << newName << "]\n";
       	bool success = cmSystemTools::RemoveFile(newName.c_str());
	if ( !success )
	 {
        	    // Handle the error of not being able to remove the  
file....
          }
       }

Just an observation.. and I haven't actually looked deep in the cmake  
code to figure out if this was a copy/paste from the actual code or  
just something as an example..

Peace.
--
Mike Jackson
imikejackson & gmail * come