Branch: refs/heads/main
Revision: 4d97dc5
Author: arezaii
Link: Unavailable
Log Message:
Merge pull request #20280 from arezaii/dyno-llvm-filesystem
feat(dyno): migrate some system and file funcs
This PR migrates some file and system functionality
into dyno and the chpl
namespace.
Some motivation for this is the need for dyno-chpldoc
to form and execute a shell command to invoke sphinx
.
Overview of changes:
-
move some functions associated with creating temporary directories
to dyno -
rely on
LLVM
support library for deleting directories, where
previously we conditionally built and used different supporting
functions depending onHAVE_LLVM
, now we assume we using the
support library at least -
migrate
mysystem
functionality to dyno, preserving
old call sites by changing original functions to wrappers around
the new calls in dyno
TESTING:
- paratest
Reviewed by @dlongnecke-cray and @mppf - thanks!
*Note: Old PR feat(dyno): migrate some file and system commands to dyno by arezaii · Pull Request #20264 · chapel-lang/chapel · GitHub is superseded
by this work and will be closed.
Signed-off-by: arezaii ahmad.rezaii@hpe.com
Modified Files:
A compiler/dyno/include/chpl/util/subprocess.h
A compiler/dyno/lib/util/subprocess.cpp
M compiler/dyno/include/chpl/util/filesystem.h
M compiler/dyno/lib/util/CMakeLists.txt
M compiler/dyno/lib/util/Makefile.include
M compiler/dyno/lib/util/filesystem.cpp
M compiler/util/files.cpp
M compiler/util/mysystem.cpp
Compare: https://github.com/chapel-lang/chapel/compare/c7a8e6f29604...4d97dc569281