[Lldb-commits] [PATCH] D69341: [zorg] Port LLDB cmake build factory to git

[Jan Kratochvil via Phabricator via lldb-commits]

jankratochvil added a comment.

After the @sivachandra 's fix I get:

  2019-10-23 21:28:24+0200 [-] LLVMGitPoller: using workdir '/quad/home/buildbot/zorg-git/buildbot/osuosl/master/gitpoller-workdir'
  2019-10-23 21:28:24+0200 [-] LLVMGitPoller: initializing working dir from http://git.lab.llvm.org/llvm/llvm-project.git
  2019-10-23 21:28:24+0200 [-] configuration update complete
  2019-10-23 21:28:25+0200 [-] while initializing LLVMGitPoller repository
  	Traceback (most recent call last):
  	  File "/usr/lib64/python2.7/site-packages/twisted/internet/_baseprocess.py", line 64, in maybeCallProcessEnded
  	    proto.processEnded(Failure(reason))
  	  File "/usr/lib64/python2.7/site-packages/twisted/internet/utils.py", line 163, in processEnded
  	    self.deferred.callback((out, err, code))
  	  File "/usr/lib64/python2.7/site-packages/twisted/internet/defer.py", line 460, in callback
  	    self._startRunCallbacks(result)
  	  File "/usr/lib64/python2.7/site-packages/twisted/internet/defer.py", line 568, in _startRunCallbacks
  	    self._runCallbacks()
  	--- <exception caught here> ---
  	  File "/usr/lib64/python2.7/site-packages/twisted/internet/defer.py", line 654, in _runCallbacks
  	    current.result = callback(current.result, *args, **kw)
  	  File "/quad/home/buildbot/zorg-git/buildbot/osuosl/master/zorg/buildbot/changes/llvmgitpoller.py", line 419, in _convert_nonzero_to_failure
  	    raise EnvironmentError('command failed with exit code %d: %s' % (code, stderr))
  	exceptions.EnvironmentError: command failed with exit code 128: fatal: unable to access 'http://git.lab.llvm.org/llvm/llvm-project.git/': Could not resolve host: git.lab.llvm.org



================
Comment at: zorg/buildbot/builders/LLDBBuilder.py:110
     cmake_cmd = [
-        "cmake", "-G", "Ninja", "../llvm",
+        "cmake", "-G", "Ninja", "../" + self.monorepo_dir,
         "-DCMAKE_BUILD_TYPE=" + config,
----------------
sivachandra wrote:
> Should this be os.path.join(os.pardir, f.monorepo_dir, "llvm") ?
I was curious there is no longer trailing `"llvm"` as I am using `cmake ../llvm-monorepo/llvm/`.  Yes, it does compile with the new expression there:
```
        "cmake", "-G", "Ninja", os.path.join(os.pardir, f.monorepo_dir, "llvm"),
```


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D69341/new/

https://reviews.llvm.org/D69341