composer / satis

Simple static Composer repository generator - For a full private Composer repo use Private Packagist

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Do we need to load all packages each times with providers enabled ?

glickel opened this issue · comments

Hello,
On BuildCommand if we have a package/repository filter, we load all packages to dump them with the PackagesBuilder :

if ($packageSelection->hasFilterForPackages() || $packageSelection->hasRepositoryFilter()) {
    // in case of an active filter we need to load the dumped packages.json and merge the
    // updated packages in
    $oldPackages = $packageSelection->load();
    $packages += $oldPackages;
    ksort($packages);
}

$packagesBuilder = new PackagesBuilder($output, $outputDir, $config, $skipErrors);
$packagesBuilder->dump($packages);

Is it really needed if providers option is enabled ?
If no, I can create a pull request to change the code

Hello,

I'm proposing a change here #650 that could fix your issue