Re: [PATCH] rtemstoolkit/path : Add listdir function

2019-05-19 Thread Chris Johns
Hi, Looks good, however in review I wonder if we should print the os error in each case? Something like: except IOError as e: raise error.general('Could not list files: %s: %s' % (path, str(e))) If we do this would it also make sense to update `mkdir`? Something like: except OSError as

[PATCH] rtemstoolkit/path : Add listdir function

2019-05-19 Thread Vijay Kumar Banerjee
--- rtemstoolkit/path.py | 29 - tester/rt/coverage.py | 4 ++-- 2 files changed, 30 insertions(+), 3 deletions(-) diff --git a/rtemstoolkit/path.py b/rtemstoolkit/path.py index 2b569ae..9051bb5 100644 --- a/rtemstoolkit/path.py +++ b/rtemstoolkit/path.py @@ -157,6 +