This is an archive of the discontinued LLVM Phabricator instance.

[lldb] Print message after loading 'crashlog' command
ClosedPublic

Authored by kastiglione on Feb 7 2022, 9:35 AM.

Details

Summary

Previously, importing crashlog resulted in a message being printed. The
message was about other commands (those in heap.py), not crashlog. The
changes in D117237 made it so that the heap.py messages were printed only when
importing lldb.macosx.heap, not when importing lldb.macosx.crashlog. Some
users may see no output and think crashlog wasn't successfully loaded. This
ensures users see that crashlog is loaded.

rdar://88283132

Diff Detail

Event Timeline

kastiglione requested review of this revision.Feb 7 2022, 9:35 AM
kastiglione created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptFeb 7 2022, 9:35 AM

We have (shell) tests that import the crashlog command. Can you add a check line to make sure the message gets printed?

Update createPythonInit.py

We have (shell) tests that import the crashlog command. Can you add a check line to make sure the message gets printed?

The message isn't printed for a Python import, only for an lldb command script import.

We have (shell) tests that import the crashlog command. Can you add a check line to make sure the message gets printed?

The message isn't printed for a Python import, only for an lldb command script import.

Both ScriptInterpreter/Python/Crashlog/text.test and ScriptInterpreter/Python/Crashlog/json.test are using command script import.

Update tests

remove changes to createPythonInit.py

This revision is now accepted and ready to land.Feb 7 2022, 12:24 PM
kastiglione edited the summary of this revision. (Show Details)Feb 7 2022, 12:33 PM
This revision was automatically updated to reflect the committed changes.