On 11/18/19 4:21 PM, Sergei Trofimovich wrote: > diff --git a/repoman/lib/repoman/modules/scan/depend/profile.py > b/repoman/lib/repoman/modules/scan/depend/profile.py > index d980f4eca..0b1d74483 100644 > --- a/repoman/lib/repoman/modules/scan/depend/profile.py > +++ b/repoman/lib/repoman/modules/scan/depend/profile.py > @@ -33,6 +33,7 @@ class ProfileDependsChecks(ScanBase): > @param options: cli options > @param repo_settings: repository settings instance > @param include_arches: set > + @param include_profiles: set > @param caches: dictionary of our caches > @param repoman_incrementals: tuple > @param env: the environment > @@ -46,6 +47,7 @@ class ProfileDependsChecks(ScanBase): > self.options = kwargs.get('options') > self.repo_settings = kwargs.get('repo_settings') > self.include_arches = kwargs.get('include_arches') > + self.include_profiles = kwargs.get('include_profiles') > self.caches = kwargs.get('caches') > self.repoman_incrementals = kwargs.get('repoman_incrementals') > self.env = kwargs.get('env') > @@ -81,8 +83,11 @@ class ProfileDependsChecks(ScanBase): > if arch not in self.include_arches: > continue > > - relevant_profiles.extend( > - (keyword, groups, prof) for prof in > self.profiles[arch]) > + for prof in self.profiles[arch]: > + if self.include_profiles is not None: > + if prof not in self.include_profiles:
Since prof is an instance of repoman.profile.ProfileDesc, you actually have to use prof.sub_path for this containment check. That means your benchmark skipped the profile that you intended to include. > + continue > + relevant_profiles.append((keyword, groups, > prof)) -- Thanks, Zac
signature.asc
Description: OpenPGP digital signature