Skip to content

Commit 152d2f0

Browse files
Merge pull request #1068 from linsword13/vtune
Fix up directory removal
2 parents c80d69e + 7064a07 commit 152d2f0

File tree

1 file changed

+6
-2
lines changed
  • var/ramble/repos/builtin/modifiers/intel-vtune

1 file changed

+6
-2
lines changed

var/ramble/repos/builtin/modifiers/intel-vtune/modifier.py

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,11 @@ class IntelVtune(BasicModifier):
5252
mode="mpi",
5353
)
5454

55-
register_builtin("setup_vtune_results_dir", injection_method="prepend")
55+
register_builtin(
56+
"setup_vtune_results_dir", required=True, injection_method="prepend"
57+
)
5658

5759
def setup_vtune_results_dir(self):
58-
return ["rm -rf {vtune_results_dir}", "mkdir -p {vtune_results_dir}"]
60+
# For some analysis types (like per-node), the results_dir only acts as a prefix,
61+
# and a series of directories will be created, such as vtune_dir.node1, vtune_dir.node2
62+
return ["rm -rf {vtune_results_dir}*"]

0 commit comments

Comments
 (0)