manifest: handle missing top level manifest file

Improve error handling to avoid dumping stack.

Signed-off-by: Martí Bolívar <marti.bolivar@nordicsemi.no>

src/west/manifest.py

@@ -1788,7 +1788,15 @@ def get_option(option, default=None):
 
             current_relpath = manifest_path / manifest_file
             current_abspath = topdir_abspath / current_relpath
-            current_data = current_abspath.read_text(encoding='utf-8')
+            try:
+                current_data = current_abspath.read_text(encoding='utf-8')
+            except FileNotFoundError:
+                raise MalformedConfig(
+                    f'file not found: manifest file {current_abspath} '
+                    '(from configuration options '
+                    f'manifest.path="{manifest_path_option}", '
+                    f'manifest.file="{manifest_file}")')
+
             current_repo_abspath = topdir_abspath / manifest_path
 
             self.abspath = os.fspath(current_abspath)

tests/test_manifest.py

@@ -1015,7 +1015,7 @@ def test_bad_topdir_fails(tmp_workspace):
     # Make sure we get expected failure using Manifest.from_topdir()
     # with the topdir kwarg when no west.yml exists.
 
-    with pytest.raises(FileNotFoundError):
+    with pytest.raises(MalformedConfig):
         MT(topdir=tmp_workspace)
 
 def test_from_bad_topdir(tmpdir):