provide better feedback when we have exhausted all config_pb2 options
Signed-off-by: Brian S. Stephan <bss@incorporeal.org>
This commit is contained in:
parent
c81f4cd139
commit
7fbccb6cde
@ -68,7 +68,8 @@ def get_config_pb2(with_fallback: bool = args.use_shipped_fallback):
|
|||||||
except (ModuleNotFoundError, TypeError):
|
except (ModuleNotFoundError, TypeError):
|
||||||
# (TypeError could be the windows bug https://github.com/protocolbuffers/protobuf/issues/14345)
|
# (TypeError could be the windows bug https://github.com/protocolbuffers/protobuf/issues/14345)
|
||||||
if not with_fallback:
|
if not with_fallback:
|
||||||
raise
|
logger.exception("no viable set of protobuf modules could be found, please use -P or -S!")
|
||||||
|
raise RuntimeError("no viable set of protobuf modules could be found, please use -P or -S!")
|
||||||
|
|
||||||
# that failed, import the snapshot (may be lagging what's in GP2040-CE)
|
# that failed, import the snapshot (may be lagging what's in GP2040-CE)
|
||||||
logger.warning("using the fallback .proto files! please supply your files with -P if you can!")
|
logger.warning("using the fallback .proto files! please supply your files with -P if you can!")
|
||||||
|
@ -52,7 +52,7 @@ def test_get_config_pb2_precompiled():
|
|||||||
|
|
||||||
def test_get_config_pb2_exception():
|
def test_get_config_pb2_exception():
|
||||||
"""Test that we fail if no config .proto files are available."""
|
"""Test that we fail if no config .proto files are available."""
|
||||||
with pytest.raises(ModuleNotFoundError):
|
with pytest.raises(RuntimeError):
|
||||||
_ = get_config_pb2()
|
_ = get_config_pb2()
|
||||||
|
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user