From ff15973a255736d59ec893c5043fc462be5010cd Mon Sep 17 00:00:00 2001 From: henderkes Date: Fri, 31 Oct 2025 09:08:15 +0100 Subject: [PATCH] suggestions --- src/SPC/builder/unix/library/postgresql.php | 2 +- src/SPC/util/DependencyUtil.php | 19 +------------------ src/SPC/util/SPCConfigUtil.php | 3 +-- 3 files changed, 3 insertions(+), 21 deletions(-) diff --git a/src/SPC/builder/unix/library/postgresql.php b/src/SPC/builder/unix/library/postgresql.php index a81f93de..e55b6e93 100644 --- a/src/SPC/builder/unix/library/postgresql.php +++ b/src/SPC/builder/unix/library/postgresql.php @@ -45,7 +45,7 @@ trait postgresql protected function build(): void { - $libs = array_map(fn ($x) => $x->getName(), $this->getDependencies()); + $libs = array_map(fn ($x) => $x->getName(), $this->getDependencies(true)); $spc = new SPCConfigUtil($this->builder, ['no_php' => true, 'libs_only_deps' => true]); $config = $spc->config(libraries: $libs, include_suggest_lib: $this->builder->getOption('with-suggested-libs')); diff --git a/src/SPC/util/DependencyUtil.php b/src/SPC/util/DependencyUtil.php index c300832d..83123f65 100644 --- a/src/SPC/util/DependencyUtil.php +++ b/src/SPC/util/DependencyUtil.php @@ -105,7 +105,7 @@ class DependencyUtil * @param array $additional_libs Array of additional libraries * @return array Ordered array of extension names */ - public static function getExtsAndLibs(array $exts, array $additional_libs = [], bool $include_suggested_exts = false, bool $include_suggested_libs = false, array $extra_libraries_from_builder = []): array + public static function getExtsAndLibs(array $exts, array $additional_libs = [], bool $include_suggested_exts = false, bool $include_suggested_libs = false): array { $dep_list = self::platExtToLibs(); @@ -144,23 +144,6 @@ class DependencyUtil $dep_list[$name]['suggests'] = array_values($dep_list[$name]['suggests']); } } - // include suggested libraries - if ($extra_libraries_from_builder) { - // check every deps suggests - foreach ($dep_list as $name => $obj) { - $del_list = []; - foreach ($obj['suggests'] as $id => $suggest) { - if (!str_starts_with($suggest, 'ext@') && in_array($suggest, $extra_libraries_from_builder)) { - $dep_list[$name]['depends'][] = $suggest; - $del_list[] = $id; - } - } - foreach ($del_list as $id) { - unset($dep_list[$name]['suggests'][$id]); - } - $dep_list[$name]['suggests'] = array_values($dep_list[$name]['suggests']); - } - } // convert ext_name to ext@ext_name $origin_exts = $exts; diff --git a/src/SPC/util/SPCConfigUtil.php b/src/SPC/util/SPCConfigUtil.php index d7f35efb..f321ca10 100644 --- a/src/SPC/util/SPCConfigUtil.php +++ b/src/SPC/util/SPCConfigUtil.php @@ -62,8 +62,7 @@ class SPCConfigUtil $extensions[] = $ext; } } - $extra_builder_libs = $this->builder?->getLibs() ?? []; - [$extensions, $libraries] = DependencyUtil::getExtsAndLibs($extensions, $libraries, $include_suggest_ext, $include_suggest_lib, array_map(fn ($l) => $l->getName(), $extra_builder_libs)); + [$extensions, $libraries] = DependencyUtil::getExtsAndLibs($extensions, $libraries, $include_suggest_ext, $include_suggest_lib); ob_start(); if ($this->builder === null) {