From 16dd798f107f621b89d3a4621bd60095b5379f79 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=D0=95=D0=B2=D0=B3=D0=B5=D0=BD=D0=B8=D0=B9=20=D0=A5=D1=80?= =?UTF-8?q?=D0=B0=D0=BC=D1=8B=D1=87=D0=AA=20=D0=A5=D1=80=D0=B0=D0=BC=D0=BE?= =?UTF-8?q?=D0=B2?= Date: Mon, 23 Feb 2026 16:11:48 +0300 Subject: [PATCH] =?UTF-8?q?=D0=98=D1=81=D0=BF=D1=80=D0=B0=D0=B2=D0=BB?= =?UTF-8?q?=D0=B5=D0=BD=D0=BE=20=D0=BB=D0=BE=D0=B3=D0=B8=D1=80=D0=BE=D0=B2?= =?UTF-8?q?=D0=B0=D0=BD=D0=B8=D0=B5?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Исправлено применение logLevel из конфига (значение в lowercase не распознавалось) - Добавлено предупреждение при некорректном значении logLevel - Исправлен уровень отладочных сообщений с Info на Debug (12 мест) --- internal/build/build.go | 18 +++++++++--------- main.go | 6 +++++- pkg/alrsh/alrsh.go | 7 +++---- 3 files changed, 17 insertions(+), 14 deletions(-) diff --git a/internal/build/build.go b/internal/build/build.go index 9caa8a0..b2519f6 100644 --- a/internal/build/build.go +++ b/internal/build/build.go @@ -614,9 +614,9 @@ func (b *Builder) BuildALRDeps( allPkgs = append(allPkgs, p.pkg) } - slog.Info("DEBUG: allPkgs count", "count", len(allPkgs)) + slog.Debug("allPkgs count", "count", len(allPkgs)) for _, p := range allPkgsWithKeys { - slog.Info("DEBUG: package in depTree", "key", p.key, "name", p.pkg.Name, "repo", p.pkg.Repository) + slog.Debug("package in depTree", "key", p.key, "name", p.pkg.Name, "repo", p.pkg.Repository) } needBuildPkgs, err := b.installerExecutor.FilterPackagesByVersion(ctx, allPkgs, input.OSRelease()) @@ -630,9 +630,9 @@ func (b *Builder) BuildALRDeps( needBuildNames[pkg.Name] = true } - slog.Info("DEBUG: needBuildPkgs count", "count", len(needBuildPkgs)) + slog.Debug("needBuildPkgs count", "count", len(needBuildPkgs)) for _, pkg := range needBuildPkgs { - slog.Info("DEBUG: package needs build", "name", pkg.Name) + slog.Debug("package needs build", "name", pkg.Name) } // Строим needBuildSet по КЛЮЧАМ depTree, а не по pkg.Name @@ -647,13 +647,13 @@ func (b *Builder) BuildALRDeps( // Шаг 3: Группируем подпакеты по basePkgName для оптимизации сборки // Если несколько подпакетов из одного мультипакета, собираем их вместе - slog.Info("DEBUG: sortedPkgs", "pkgs", sortedPkgs) + slog.Debug("sortedPkgs", "pkgs", sortedPkgs) // Шаг 4: Собираем пакеты в правильном порядке, проверяя кеш for _, pkgName := range sortedPkgs { node := depTree[pkgName] if node == nil { - slog.Info("DEBUG: node is nil", "pkgName", pkgName) + slog.Debug("node is nil", "pkgName", pkgName) continue } @@ -662,7 +662,7 @@ func (b *Builder) BuildALRDeps( // Пропускаем уже установленные пакеты if !needBuildSet[pkgName] { - slog.Info("DEBUG: skipping (not in needBuildSet)", "pkgName", pkgName) + slog.Debug("skipping (not in needBuildSet)", "pkgName", pkgName) continue } @@ -687,12 +687,12 @@ func (b *Builder) BuildALRDeps( if allInCache { // Подпакет в кеше, используем его - slog.Info("DEBUG: using cached package", "pkgName", pkgName) + slog.Debug("using cached package", "pkgName", pkgName) buildDeps = append(buildDeps, cachedDeps...) continue } - slog.Info("DEBUG: building package", "pkgName", pkgName) + slog.Debug("building package", "pkgName", pkgName) // Собираем только запрошенный подпакет // SkipDepsBuilding: true предотвращает рекурсивный вызов BuildALRDeps diff --git a/main.go b/main.go index 48c15ce..cbe1749 100644 --- a/main.go +++ b/main.go @@ -122,7 +122,7 @@ func GetApp() *cli.App { func setLogLevel(newLevel string) { level := slog.LevelInfo - switch newLevel { + switch strings.ToUpper(newLevel) { case "DEBUG": level = slog.LevelDebug case "INFO": @@ -131,6 +131,10 @@ func setLogLevel(newLevel string) { level = slog.LevelWarn case "ERROR": level = slog.LevelError + default: + if newLevel != "" { + slog.Warn("unknown logLevel value, falling back to INFO", "value", newLevel) + } } logger, ok := slog.Default().Handler().(*logger.Logger) if !ok { diff --git a/pkg/alrsh/alrsh.go b/pkg/alrsh/alrsh.go index e0dfeb8..a68f307 100644 --- a/pkg/alrsh/alrsh.go +++ b/pkg/alrsh/alrsh.go @@ -172,14 +172,13 @@ func (s *ScriptFile) createPackageFromMeta( return nil, err } - // DEBUG: Выводим что в metaRunner.Vars и dec.Runner.Vars для deps_debian if depsDebianMeta, ok := metaRunner.Vars["deps_debian"]; ok { - slog.Info("DEBUG createPackageFromMeta: metaRunner.Vars[deps_debian]", "value", depsDebianMeta.String(), "list", depsDebianMeta.List) + slog.Debug("createPackageFromMeta: metaRunner.Vars[deps_debian]", "value", depsDebianMeta.String(), "list", depsDebianMeta.List) } else { - slog.Info("DEBUG createPackageFromMeta: metaRunner.Vars[deps_debian] NOT FOUND") + slog.Debug("createPackageFromMeta: metaRunner.Vars[deps_debian] NOT FOUND") } if depsDebianParent, ok := dec.Runner.Vars["deps_debian"]; ok { - slog.Info("DEBUG createPackageFromMeta: parent Vars[deps_debian]", "value", depsDebianParent.String(), "list", depsDebianParent.List) + slog.Debug("createPackageFromMeta: parent Vars[deps_debian]", "value", depsDebianParent.String(), "list", depsDebianParent.List) } // Сливаем переменные родительского runner'а с переменными мета-функции.