? 2013/4/20 3:16, Lucas Meneghel Rodrigues ??:
Looks good to me too, pushed to next.
Now Mike, by the way, would you consider in the next patchset to send
them as pull requests? We'd like to move all code review there, so
there's one single point of concentration for reviewers to find work,
OK? If you need any guidance on how to submit github pull requests,
let us know.
Cheers!
OK, I got it, I have saw the mail, and I will send the request as pull
request,
Any problem I will send mail to the mailling list
Thanks
Mike
On Thu, Apr 18, 2013 at 2:30 AM, Feng Yang <[email protected]
<mailto:[email protected]>> wrote:
On 04/17/2013 07:39 PM, Mike Qiu wrote:
Sometimes this will lead an error, So before unlink the file
check if it is real exist.
Signed-off-by: Mike Qiu <[email protected]
<mailto:[email protected]>>
Works for me
Acked-by: Feng Yang <[email protected] <mailto:[email protected]>>
---
client/test.py | 3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/client/test.py b/client/test.py
index 988b502..2d3fd53 100644
--- a/client/test.py
+++ b/client/test.py
@@ -92,7 +92,8 @@ class test(common_test.base_test):
"""
if self.crash_handling_enabled:
# Remove the debugdir info file
- os.unlink(self.debugdir_tmp_file)
+ if os.path.isfile(self.debugdir_tmp_file):
+ os.unlink(self.debugdir_tmp_file)
# Restore the core pattern backup
try:
utils.open_write_close(self.pattern_file,
_______________________________________________
Autotest-kernel mailing list
[email protected] <mailto:[email protected]>
https://www.redhat.com/mailman/listinfo/autotest-kernel
--
Lucas
_______________________________________________
Autotest-kernel mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/autotest-kernel
