Hi,
On 06/24/2015 01:50 AM, Martin Quinson wrote:
> [..]
>> +--- simgrid-3.11.1.orig/buildtools/Cmake/GenerateDoc.cmake
>> ++++ simgrid-3.11.1/buildtools/Cmake/GenerateDoc.cmake
>> +@@ -48,7 +48,7 @@ if(DOXYGEN_FOUND)
>> + ADD_CUSTOM_COMMAND(TARGET simgrid_documentation
>> + COMMAND ${FIG2DEV_PATH}/fig2dev -Lmap
>> ${CMAKE_HOME_DIRECTORY}/doc/shared/fig/simgrid_modules.fig | perl -pe
>> 's/imagemap/simgrid_modules/g'| perl -pe 's/<IMG/<IMG style=border:0px/g' |
>> ${CMAKE_HOME_DIRECTORY}/tools/doxygen/fig2dev_postprocessor.pl >
>> ${CMAKE_HOME_DIRECTORY}/doc/simgrid_modules.map
>> + COMMAND pwd
>> +- COMMAND ${CMAKE_COMMAND} -E tar czf html/msg-tuto-src.tgz msg-tuto-src/
>> ++ COMMAND ${CMAKE_COMMAND} -E env GZIP=-9n tar czf html/msg-tuto-src.tgz
>> msg-tuto-src/
>> + COMMAND ${CMAKE_COMMAND} -E echo "XX Run doxygen"
>> + COMMAND ${DOXYGEN_EXECUTABLE} Doxyfile
>> + COMMAND ${CMAKE_COMMAND} -E echo "XX Generate the index files"
>
> The trick is that we are not using tar here, but the tar
> command of cmake. So the patch gives me:
> CMake Error: cmake version 3.0.2
> Usage: /usr/bin/cmake -E [command] [arguments ...]
> Available commands:
> ...
>
I am aware that not tar is used being used but instead cmake tar. This
is why I also used cmake env to set the environment variable instead of
using other means. But it seams that the env command is not available in
cmake version 3.0.2 in jessie but in cmake version 3.2.2-2 in stretch
and sid. There does not seem to be a way to supply environment variables
to cmake 3.0.2. So in addition to my patch the cmake minimum version
should be bumped to 3.2.2.
> As an upstream author, I'm not willing to switch from "cmake -E tar"
> to "tar" because it will not work on windows or such.
>
> We could try to set the environment variable to "cmake -E tar", but
> I'm not sure that cmake will do what is expected here. I miss the time
> to test right now unfortunately. Your help would be welcome.
Using `GZIP=-9n cmake -E tar` does not work. It seems that cmake does
not pass environment variables to the command it runs. This explains the
introduction of the env cmake command.
If you do not see any other way then it seems that a bump of the minimum
cmake version is necessary.
>
> The other part of the patch wrt GENERATE_TIMESTAMP is already commited
> upstream, thanks.
>
Thanks!
Cheers,
akira
--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
