Fred, do you know why this wasn’t caught by PR testing?
···
On Jan 31, 2017, at 11:16 AM, Michael Ilseman <milseman@apple.com> wrote:
Thank you!
On Jan 31, 2017, at 11:15 AM, Frédéric Riss <friss@apple.com <mailto:friss@apple.com>> wrote:
Yes, I already pushed a fix. Sorry about that.
Fred
On Jan 31, 2017, at 11:13 AM, Michael Ilseman <milseman@apple.com <mailto:milseman@apple.com>> wrote:
Fred, probably you?
CMake Error at tools/clang/tools/driver/CMakeLists.txt:79 (set):
Syntax error in cmake code at/Users/buildnode/jenkins/workspace/oss-lldb-swift-3.1-incremental-osx/llvm/tools/clang/tools/driver/CMakeLists.txt:79
when parsing string
${LLVM_VERSION_MAJOR}.${LLVM_VERSION_MINOR}.${LLVM_VERSION_PATCH
There is an unterminated variable reference.
-- Configuring incomplete, errors occurred!
See also "/Users/buildnode/jenkins/workspace/oss-lldb-swift-3.1-incremental-osx/Ninja-ReleaseAssert/llvm-macosx-x86_64/CMakeFiles/CMakeOutput.log".
See also "/Users/buildnode/jenkins/workspace/oss-lldb-swift-3.1-incremental-osx/Ninja-ReleaseAssert/llvm-macosx-x86_64/CMakeFiles/CMakeError.log".
FAILED: build.ninja
/Applications/CMake.app/Contents/bin/cmake -H/Users/buildnode/jenkins/workspace/oss-lldb-swift-3.1-incremental-osx/llvm -B/Users/buildnode/jenkins/workspace/oss-lldb-swift-3.1-incremental-osx/Ninja-ReleaseAssert/llvm-macosx-x86_64
<>On Jan 31, 2017, at 10:50 AM, no-reply@swift.org <mailto:no-reply@swift.org> wrote:
[FAILURE] oss-lldb-swift-3.1-incremental-osx [#187]
Build URL: https://ci.swift.org/job/oss-lldb-swift-3.1-incremental-osx/187/
Project: oss-lldb-swift-3.1-incremental-osx
Date of build: Tue, 31 Jan 2017 10:50:27 -0800
Build duration: 26 sec
Identified problems:Compile Error: This build failed because of a compile error. Below is a list of all errors in the build log:
Indication 1 <https://ci.swift.org//job/oss-lldb-swift-3.1-incremental-osx/187/consoleFull#3542627ee1a197b-acac-4b17-83cf-a53b95139a76>
ChangesCommit e4215eb1b07ed69909b001216f83d51f743ef82b by friss:
[CMake] Fixing variable names that were mistypededit: tools/driver/CMakeLists.txt
Commit d82b06d6ee1bebd4ed371fb08efd852c15071d5e by friss:
Add the patch version to the embedded Info.plist.edit: tools/driver/CMakeLists.txt