Skip to content

Commit

Permalink
Fix hang in config file generation (#101)
Browse files Browse the repository at this point in the history
- Updated Timemory module.
- Fixes a crash when running rocprof-sys-avail -G without explicitly providing -F <format>. The default value of "txt" was not being used.
- Define "choices" before "default" when defining the "--config-format" argument in the parser.
  • Loading branch information
dgaliffiAMD authored Feb 11, 2025
1 parent 697d1ac commit 3833c8d
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion external/timemory
2 changes: 1 addition & 1 deletion source/bin/rocprof-sys-avail/avail.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -454,8 +454,8 @@ main(int argc, char** argv)
parser.add_argument({ "-F", "--config-format" }, "Configuration file format")
.min_count(1)
.max_count(3)
.set_default(std::set<std::string>{ "txt" })
.choices({ "txt", "json", "xml" })
.set_default(std::set<std::string>{ "txt" })
.dtype("filename")
.action([&_config_fmts](parser_t& _p) {
_config_fmts = _p.get<std::set<std::string>>("config-format");
Expand Down

0 comments on commit 3833c8d

Please sign in to comment.