<div dir="ltr">you mean like this "exe = os.path.join('./newdir/','proc_attach')". I can't use "os.getcwd" as it will become the absolute path which we should not use.<div><br></div><div>thanks,</div><div>vbalu</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, May 3, 2017 at 11:51 AM, Zachary Turner <span dir="ltr"><<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Can you rewrite the exe path computation to use os.path.join so we're not assuming posix syntax?<br><div class="gmail_quote"><div><div class="h5"><div dir="ltr">On Tue, May 2, 2017 at 11:11 PM vignesh balu via Phabricator via lldb-commits <<a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a>> wrote:<br></div></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5">vbalu updated this revision to Diff 97550.<br>
vbalu added a comment.<br>
<br>
Corrected the code. Now this test will fail without <a href="https://reviews.llvm.org/D32271" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D32271</a> patch.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D32522" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D32522</a><br>
<br>
Files:<br>
  packages/Python/lldbsuite/<wbr>test/functionalities/process_<wbr>attach/TestProcessAttach.py<br>
<br>
<br>
Index: packages/Python/lldbsuite/<wbr>test/functionalities/process_<wbr>attach/TestProcessAttach.py<br>
==============================<wbr>==============================<wbr>=======<br>
--- packages/Python/lldbsuite/<wbr>test/functionalities/process_<wbr>attach/TestProcessAttach.py<br>
+++ packages/Python/lldbsuite/<wbr>test/functionalities/process_<wbr>attach/TestProcessAttach.py<br>
@@ -8,6 +8,7 @@<br>
 import os<br>
 import time<br>
 import lldb<br>
+import shutil<br>
 from lldbsuite.test.decorators import *<br>
 from lldbsuite.test.lldbtest import *<br>
 from lldbsuite.test import lldbutil<br>
@@ -38,6 +39,25 @@<br>
         process = target.GetProcess()<br>
         self.assertTrue(process, PROCESS_IS_VALID)<br>
<br>
+    def test_attach_to_process_frm_<wbr>different_dir_by_id(self):<br>
+        """Test attach by process id"""<br>
+        os.mkdir(os.path.join(os.<wbr>getcwd(),'newdir'))<br>
+        self.buildProgram('main.cpp','<wbr>newdir/proc_attach')<br>
+        exe = './newdir/proc_attach'<br>
+        self.addTearDownHook(lambda: shutil.rmtree(os.path.join(os.<wbr>getcwd())))<br>
+<br>
+        # Spawn a new process<br>
+        popen = self.spawnSubprocess(exe)<br>
+        self.addTearDownHook(self.<wbr>cleanupSubprocesses)<br>
+<br>
+        os.chdir('newdir')<br>
+        self.runCmd("process attach -p " + str(popen.pid))<br>
+<br>
+        target = self.dbg.GetSelectedTarget()<br>
+<br>
+        process = target.GetProcess()<br>
+        self.assertTrue(process, PROCESS_IS_VALID)<br>
+<br>
     def test_attach_to_process_by_<wbr>name(self):<br>
         """Test attach by process name"""<br>
         self.build()<br>
<br>
<br></div></div>
______________________________<wbr>_________________<br>
lldb-commits mailing list<br>
<a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/lldb-commits</a><br>
</blockquote></div>
</blockquote></div><br></div>