juergbi commented on code in PR #2113:
URL: https://github.com/apache/buildstream/pull/2113#discussion_r3141794591


##########
src/buildstream/_loader/loader.py:
##########
@@ -128,6 +128,15 @@ def load(self, targets):
 
         for target in targets:
             with PROFILER.profile(Topics.LOAD_PROJECT, target):
+
+                # As we are attempting to load a subproject element via the
+                # command line ensure fully loaded.
+                #
+                # Other callers of `_parse_name` or `_load_file` that reference
+                # files through a project element or otherwise do not need to
+                # ensure fully loaded.
+                self.project.ensure_fully_loaded()

Review Comment:
   I've opened #2114, which fixes this issue in a different way. It includes 
and passes your tests from this branch (adjusted to keep the previous tests 
without changes).



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to