[llvm] r223321 - Add missing dependency on llvm-ar. Should hopefully fix the bots.

[Rafael Espíndola]

Looks like the same issue is being discussed on the thread for
r223309. I am trying to reproduce the issue. Can you reply to that
thread with the cmake and msbuild commands that you used?

On 8 December 2014 at 06:25, Dmitry Babokin <babokin at gmail.com> wrote:
> Hi,
>
> This commit seems to break Windows build (fail happens on INSTALL step). I
> also noticed that "llvm-ranlib" appears in VS solution on top level, but not
> in Tools folder (where I would expect him to be). Here's the error message:
>
> 233>  -- Up-to-date: C:/llvm/bin-trunk/bin/llvm-ar.exe
> 233>  CMake Error at tools/llvm-ar/cmake_install.cmake:43 (FILE):
> 233>    file INSTALL cannot find
> 233>    "C:/llvm/build-trunk/$(Configuration)/bin/llvm-ranlib.exe".
> 233>  Call Stack (most recent call first):
> 233>    tools/cmake_install.cmake:38 (INCLUDE)
> 233>    cmake_install.cmake:56 (INCLUDE)
> 233>
> 233>
> 233>C:\Program Files
> (x86)\MSBuild\Microsoft.Cpp\v4.0\V110\Microsoft.CppCommon.targets(134,5):
> error MSB3073: The command "setlocal
> 233>C:\Program Files
> (x86)\MSBuild\Microsoft.Cpp\v4.0\V110\Microsoft.CppCommon.targets(134,5):
> error MSB3073: "C:\Program Files (x86)\CMake 2.8\bin\cmake.exe"
> -DBUILD_TYPE=Release -P cmake_install.cmake
> 233>C:\Program Files
> (x86)\MSBuild\Microsoft.Cpp\v4.0\V110\Microsoft.CppCommon.targets(134,5):
> error MSB3073: if %errorlevel% neq 0 goto :cmEnd
> 233>C:\Program Files
> (x86)\MSBuild\Microsoft.Cpp\v4.0\V110\Microsoft.CppCommon.targets(134,5):
> error MSB3073: :cmEnd
> 233>C:\Program Files
> (x86)\MSBuild\Microsoft.Cpp\v4.0\V110\Microsoft.CppCommon.targets(134,5):
> error MSB3073: endlocal & call :cmErrorLevel %errorlevel% & goto :cmDone
> 233>C:\Program Files
> (x86)\MSBuild\Microsoft.Cpp\v4.0\V110\Microsoft.CppCommon.targets(134,5):
> error MSB3073: :cmErrorLevel
> 233>C:\Program Files
> (x86)\MSBuild\Microsoft.Cpp\v4.0\V110\Microsoft.CppCommon.targets(134,5):
> error MSB3073: exit /b %1
> 233>C:\Program Files
> (x86)\MSBuild\Microsoft.Cpp\v4.0\V110\Microsoft.CppCommon.targets(134,5):
> error MSB3073: :cmDone
> 233>C:\Program Files
> (x86)\MSBuild\Microsoft.Cpp\v4.0\V110\Microsoft.CppCommon.targets(134,5):
> error MSB3073: if %errorlevel% neq 0 goto :VCEnd
> 233>C:\Program Files
> (x86)\MSBuild\Microsoft.Cpp\v4.0\V110\Microsoft.CppCommon.targets(134,5):
> error MSB3073: :VCEnd" exited with code 1.
> ========== Build: 232 succeeded, 1 failed, 0 up-to-date, 0 skipped
> ==========
>
>
> Dmitry.
>
> On Thu, Dec 4, 2014 at 3:00 AM, Rafael Espindola
> <rafael.espindola at gmail.com> wrote:
>>
>> Author: rafael
>> Date: Wed Dec  3 18:00:36 2014
>> New Revision: 223321
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=223321&view=rev
>> Log:
>> Add missing dependency on llvm-ar. Should hopefully fix the bots.
>>
>> Modified:
>>     llvm/trunk/tools/llvm-ar/CMakeLists.txt
>>
>> Modified: llvm/trunk/tools/llvm-ar/CMakeLists.txt
>> URL:
>> http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-ar/CMakeLists.txt?rev=223321&r1=223320&r2=223321&view=diff
>>
>> ==============================================================================
>> --- llvm/trunk/tools/llvm-ar/CMakeLists.txt (original)
>> +++ llvm/trunk/tools/llvm-ar/CMakeLists.txt Wed Dec  3 18:00:36 2014
>> @@ -19,7 +19,8 @@ set(llvm_ar_binary "llvm-ar${CMAKE_EXECU
>>  set(llvm_ranlib
>> "${LLVM_RUNTIME_OUTPUT_INTDIR}/llvm-ranlib${CMAKE_EXECUTABLE_SUFFIX}")
>>
>>  add_custom_command(OUTPUT ${llvm_ranlib}
>> -                   COMMAND ${CMAKE_COMMAND} -E ${LLVM_LINK_OR_COPY}
>> "${llvm_ar_binary}" "${llvm_ranlib}")
>> +                   COMMAND ${CMAKE_COMMAND} -E ${LLVM_LINK_OR_COPY}
>> "${llvm_ar_binary}" "${llvm_ranlib}"
>> +                   DEPENDS llvm-ar)
>>
>>  add_custom_target(llvm_ranlib_build ALL DEPENDS ${llvm_ranlib})
>>  install(FILES "${llvm_ranlib}" DESTINATION bin)
>>
>>
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
>