refactor command as more easily

This commit is contained in:
crazywhalecc
2023-04-22 17:45:43 +08:00
parent 1540af266b
commit 4c0d35c723
10 changed files with 150 additions and 106 deletions

View File

@@ -9,20 +9,17 @@ use SPC\exception\RuntimeException;
use SPC\exception\WrongUsageException;
use SPC\util\DependencyUtil;
use SPC\util\LicenseDumper;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Attribute\AsCommand;
use Symfony\Component\Console\Input\InputOption;
use Symfony\Component\Console\Output\OutputInterface;
/**
* 修改 config 后对其 kv 进行排序的操作
*/
#[AsCommand('dump-license', 'Dump licenses for required libraries')]
class DumpLicenseCommand extends BaseCommand
{
protected static $defaultName = 'dump-license';
public function configure()
{
$this->setDescription('Dump licenses for required libraries');
$this->addOption('by-extensions', null, InputOption::VALUE_REQUIRED, 'Dump by extensions and related libraries', null);
$this->addOption('without-php', null, InputOption::VALUE_NONE, 'Dump without php-src');
$this->addOption('by-libs', null, InputOption::VALUE_REQUIRED, 'Dump by libraries', null);
@@ -35,42 +32,42 @@ class DumpLicenseCommand extends BaseCommand
* @throws FileSystemException
* @throws RuntimeException
*/
public function execute(InputInterface $input, OutputInterface $output): int
public function handle(): int
{
$dumper = new LicenseDumper();
if ($input->getOption('by-extensions') !== null) {
if ($this->getOption('by-extensions') !== null) {
// 从参数中获取要编译的 extensions并转换为数组
$extensions = array_map('trim', array_filter(explode(',', $input->getOption('by-extensions'))));
$extensions = array_map('trim', array_filter(explode(',', $this->getOption('by-extensions'))));
// 根据提供的扩展列表获取依赖库列表并编译
[$extensions, $libraries, $not_included] = DependencyUtil::getExtLibsByDeps($extensions);
$dumper->addExts($extensions);
$dumper->addLibs($libraries);
if (!$input->getOption('without-php')) {
if (!$this->getOption('without-php')) {
$dumper->addSources(['php-src']);
}
$dumper->dump($input->getOption('dump-dir'));
$output->writeln('Dump license with extensions: ' . implode(', ', $extensions));
$output->writeln('Dump license with libraries: ' . implode(', ', $libraries));
$output->writeln('Dump license with' . ($input->getOption('without-php') ? 'out' : '') . ' php-src');
$output->writeln('Dump target dir: ' . $input->getOption('dump-dir'));
$dumper->dump($this->getOption('dump-dir'));
$this->output->writeln('Dump license with extensions: ' . implode(', ', $extensions));
$this->output->writeln('Dump license with libraries: ' . implode(', ', $libraries));
$this->output->writeln('Dump license with' . ($this->getOption('without-php') ? 'out' : '') . ' php-src');
$this->output->writeln('Dump target dir: ' . $this->getOption('dump-dir'));
return 0;
}
if ($input->getOption('by-libs') !== null) {
$libraries = array_map('trim', array_filter(explode(',', $input->getOption('by-libs'))));
if ($this->getOption('by-libs') !== null) {
$libraries = array_map('trim', array_filter(explode(',', $this->getOption('by-libs'))));
$libraries = DependencyUtil::getLibsByDeps($libraries);
$dumper->addLibs($libraries);
$dumper->dump($input->getOption('dump-dir'));
$output->writeln('Dump target dir: ' . $input->getOption('dump-dir'));
$dumper->dump($this->getOption('dump-dir'));
$this->output->writeln('Dump target dir: ' . $this->getOption('dump-dir'));
return 0;
}
if ($input->getOption('by-sources') !== null) {
$sources = array_map('trim', array_filter(explode(',', $input->getOption('by-sources'))));
if ($this->getOption('by-sources') !== null) {
$sources = array_map('trim', array_filter(explode(',', $this->getOption('by-sources'))));
$dumper->addSources($sources);
$dumper->dump($input->getOption('dump-dir'));
$output->writeln('Dump target dir: ' . $input->getOption('dump-dir'));
$dumper->dump($this->getOption('dump-dir'));
$this->output->writeln('Dump target dir: ' . $this->getOption('dump-dir'));
return 0;
}
$output->writeln('You must use one of "--by-extensions=", "--by-libs=", "--by-sources=" to dump');
$this->output->writeln('You must use one of "--by-extensions=", "--by-libs=", "--by-sources=" to dump');
return 1;
}
}