[CMake] breakage in recent cmake cvs

[Andy Cedilnik]

Hi Alex,

I just commited this regex and suggest it. Before it was [A-Za-z0-9_.-]
or something similar.

       Andy

Alexander Neundorf wrote:

>>Von: Andy Cedilnik <andy.cedilnik at kitware.com> 
>> 
>>Hi Alex, 
>> 
>>Partially my fault. The problem is that we have not really make any 
>>strict rule about what valid variable name is. You are trying to use 
>>/home/alex/src/cmakebug/main.cpp_deps as a variable name. I guess that 
>>should be ok, but I would suggest the following rule: 
>> 
>>Variables names have to match the following regular expression: 
>>[A-Za-z0-9/._-]+ 
>>    
>>
> 
>But the name  "/home/alex/src/cmakebug/main.cpp_deps" would match this 
>regexp, wouldn't it ? 
> 
>  
>
>>>MACRO(ADD_FILE_DEPENDANCY file) 
>>>  SET(${file}_deps ${${file}_deps} ${ARGN}) 
>>>  SET_SOURCE_FILES_PROPERTIES( 
>>>     ${file} 
>>>     PROPERTIES 
>>>     OBJECT_DEPENDS 
>>>     "${${file}_deps}" 
>>>  ) 
>>>ENDMACRO(ADD_FILE_DEPENDANCY) 
>>>      
>>>
> 
>This macro is intended to create a unique variable name for each file. 
>The file name contains '/'. Do you see another way to add more than one 
>file dependencies to a source file ? 
>I really need this. 
> 
>Bye 
>Alex 
> 
>
>  
>


-- 
Andy Cedilnik
Kitware Inc.