Skip to content

Commit

Permalink
assemble: Create set out of the target apps
Browse files Browse the repository at this point in the history
If shortlist is not specified then a set containing target apps should
be created to determine a set of the previously fetched app and the
set of apps to fetch.

Signed-off-by: Mike Sul <[email protected]>
  • Loading branch information
mike-sul committed Mar 5, 2024
1 parent 7f67963 commit 711fa3a
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions assemble.py
Original file line number Diff line number Diff line change
Expand Up @@ -283,9 +283,9 @@ def check_and_get_fetched_apps_uri(target: FactoryClient.Target, shortlist: [str
x.strip() for x in fetched_apps_str.split(',') if x)
else:
# if `shortlist` is not defined or empty then all target apps were fetched
fetched_apps = [x[0] for x in target.apps()]
fetched_apps = set(app[0] for app in target.apps())

apps_to_fetch = set(shortlist) if shortlist else set(target.apps().keys())
apps_to_fetch = set(shortlist) if shortlist else set(app[0] for app in target.apps())

if fetched_apps.issubset(apps_to_fetch):
# if the previously fetched apps is a sub-set of the apps to be fetched then
Expand Down

0 comments on commit 711fa3a

Please sign in to comment.