[LTP] [PATCH v2 02/13] lib: Replace path_exist() with tst_path_exists()

Cyril Hrubis chrubis@suse.cz
Mon Feb 10 14:07:02 CET 2025


Hi!
> > Move and rename the path_exist() function from
> > testcases/kernel/mem/lib/ to the to level library.
> > 
> > This removes mem.h dependency from mem/cpuset/ test.
> > 
> > Signed-off-by: Cyril Hrubis <chrubis@suse.cz>
> > Reviewed-by: Petr Vorel <pvorel@suse.cz>
> > ---
> >  include/tst_fs.h                       | 10 ++++++++++
> >  lib/tst_path_exists.c                  | 23 +++++++++++++++++++++++
> 
> The only thing I don't like is creating file for single functions.
> My long term plan was to have fewer files with more functions.
> I'm sorry I did not point this out in v1, thus feel free to ignore it.

Feel free to move this code to a file you find appropriate and merge the
patchset.

-- 
Cyril Hrubis
chrubis@suse.cz


More information about the ltp mailing list