Skip to content

Commit 5d4431e

Browse files
authored
Merge pull request #609 from pbalcer/urtrace-ext-py
[urtrace] small cli fixes
2 parents c41bff5 + ada39b5 commit 5d4431e

File tree

2 files changed

+10
-6
lines changed

2 files changed

+10
-6
lines changed

tools/urtrace/CMakeLists.txt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,8 +21,8 @@ if(MSVC)
2121
XPTI_STATIC_LIBRARY XPTI_CALLBACK_API_EXPORTS)
2222
endif()
2323

24-
set(UR_TRACE_CLI ${CMAKE_RUNTIME_OUTPUT_DIRECTORY}/urtrace)
24+
set(UR_TRACE_CLI_BIN ${CMAKE_RUNTIME_OUTPUT_DIRECTORY}/urtrace)
2525

2626
add_custom_target(ur_trace_cli)
27-
add_custom_command(TARGET ur_trace_cli PRE_BUILD COMMAND ${CMAKE_COMMAND} -E copy ${CMAKE_CURRENT_SOURCE_DIR}/urtrace ${UR_TRACE_CLI})
27+
add_custom_command(TARGET ur_trace_cli PRE_BUILD COMMAND ${CMAKE_COMMAND} -E copy ${CMAKE_CURRENT_SOURCE_DIR}/urtrace.py ${UR_TRACE_CLI_BIN})
2828
add_dependencies(ur_collector ur_trace_cli)

tools/urtrace/urtrace renamed to tools/urtrace/urtrace.py

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -133,8 +133,12 @@ def get_dynamic_library_name(name):
133133
if args.debug:
134134
print(env)
135135

136-
result = subprocess.run(config['command'], env=env)
137-
if args.debug:
138-
print(result)
136+
if config['command']:
137+
result = subprocess.run(config['command'], env=env)
138+
if args.debug:
139+
print(result)
140+
exit(result.returncode)
141+
else:
142+
parser.print_help()
143+
sys.exit("\n Error: Missing command to run.")
139144

140-
exit(result.returncode)

0 commit comments

Comments
 (0)