Mathieu Malaterre wrote:
I checked in a version of your fix, but changed it to use ExpandListArguement. Can you please test?



Index: cmCPackDebGenerator.cxx
===================================================================
RCS file: /cvsroot/CMake/CMake/Source/CPack/cmCPackDebGenerator.cxx,v
retrieving revision 1.21
diff -r1.21 cmCPackDebGenerator.cxx
202a203,226
>   const char* controlExtra =
>     this->GetOption("CPACK_DEBIAN_PACKAGE_CONTROL_EXTRA");
>   if( controlExtra )
>     {
>     std::vector<std::string> controlExtraList;
>     cmSystemTools::ExpandListArgument(controlExtra, controlExtraList);
>     for(std::vector<std::string>::iterator i =
>           controlExtraList.begin(); i != controlExtraList.end(); ++i)
>       {
>       std::string filenamename =
>         cmsys::SystemTools::GetFilenameName(i->c_str());
>       std::string localcopy = toplevel;
>       localcopy += "/";
>       localcopy += filenamename;
>       // if we can copy the file, it means it does exist, let's add it:
>       if( cmsys::SystemTools::CopyFileIfDifferent(
>             i->c_str(), localcopy.c_str()) )
>         {
>         // debian is picky and need relative to ./ path in the tar.gz
>         cmd += " ./";
>         cmd += filenamename;
>         }
>       }
>     }

[EMAIL PROTECTED] ~/My Builds/CMake/Source/CPack
$ cvs commit -m "ENH: add CPACK_DEBIAN_PACKAGE_CONTROL_EXTRA variable" -l
cvs commit: Examining .
/cvsroot/CMake/CMake/Source/CPack/cmCPackDebGenerator.cxx,v <-- cmCPackDebGenerator.cxx
new revision: 1.22; previous revision: 1.21

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

Reply via email to