tree b3aa72475c7d4f90d136d2323483021bf6347b7c
parent 5e64ad0950742a0d894183ec8bd85cd2610b98ed
author Jonas Devlieghere <jonas@devlieghere.com> 1613583007 -0800
committer Copybara-Service <copybara-worker@google.com> 1613726135 -0800

[lldb] Improve error message for modules with dots or dashes

LLDB does not like to import Python files with dashes or dots in their
name. While the former are technically allowed, it is discouraged. Dots
are allowed for subpackages but not in module names. This patch improves
the user experience by printing a useful error.

Before this patch:

  error: module importing failed: SyntaxError('invalid syntax',
  ('<string>', 1, 11, 'import foo-bar\n'))

After this patch:

  error: module importing failed: Python discourages dashes in module
  names: foo-bar

rdar://74263511

[1] https://www.python.org/dev/peps/pep-0008/#package-and-module-names

Differential revision: https://reviews.llvm.org/D96833

GitOrigin-RevId: d6e80578fc5e5199d5783671bd0c8ce1050925f9