.gitignore

@@ -6,5 +6,3 @@
 .fleet
 .idea
 .gigaide
-
-*.out

.golangci.yml

@@ -44,7 +44,3 @@ issues:
     - path: _test\.go
       linters:
         - errcheck
-    # TODO: remove
-    - linters:
-        - staticcheck
-      text: "SA1019:"

.pre-commit-config.yaml

@@ -1,42 +0,0 @@
-# ALR - Any Linux Repository
-# Copyright (C) 2025 Евгений Храмов
-#
-# This program is free software: you can redistribute it and/or modify
-# it under the terms of the GNU General Public License as published by
-# the Free Software Foundation, either version 3 of the License, or
-# (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program.  If not, see <http://www.gnu.org/licenses/>.
-
-repos:
-  - repo: local
-    hooks:
-      - id: test-coverage
-        name: Run test coverage
-        entry: make test-coverage
-        language: system
-        pass_filenames: false
-
-      - id: fmt
-        name: Format code
-        entry: make fmt
-        language: system
-        pass_filenames: false
-
-      - id: update-license
-        name: Update license
-        entry: make update-license
-        language: system
-        pass_filenames: false
-
-      - id: i18n
-        name: Update i18n
-        entry: make i18n
-        language: system
-        pass_filenames: false

Makefile

@@ -12,7 +12,7 @@ INSTALLED_BASH_COMPLETION := $(DESTDIR)$(PREFIX)/share/bash-completion/completio
 INSTALLED_ZSH_COMPLETION := $(DESTDIR)$(PREFIX)/share/zsh/site-functions/_$(NAME)
 
 ADD_LICENSE_BIN := go run github.com/google/addlicense@4caba19b7ed7818bb86bc4cd20411a246aa4a524
-GOLANGCI_LINT_BIN := go run github.com/golangci/golangci-lint/cmd/golangci-lint@v1.63.4
+GOLANGCI_LINT_BIN := go run github.com/golangci/golangci-lint/cmd/golangci-lint@v1.62.2
 XGOTEXT_BIN := go run github.com/Tom5521/xgotext@v1.2.0
 
 .PHONY: build install clean clear uninstall check-no-root
@@ -66,7 +66,3 @@ i18n:
 	$(XGOTEXT_BIN)  --output ./internal/translations/default.pot
 	msguniq --use-first -o ./internal/translations/default.pot ./internal/translations/default.pot
 	msgmerge --backup=off -U ./internal/translations/po/ru/default.po ./internal/translations/default.pot
-
-test-coverage:
-	go test ./... -v -coverpkg=./... -coverprofile=coverage.out
-	bash scripts/coverage-badge.sh

README.md

@@ -3,8 +3,6 @@
 </p>
 <b></b>
 
-[![Go Report Card](https://goreportcard.com/badge/gitea.plemya-x.ru/Plemya-x/ALR)](https://goreportcard.com/report/gitea.plemya-x.ru/Plemya-x/ALR) ![Test coverage](./coverage-badge.svg)
-
 # ALR (Any Linux Repository)
 
 ALR - это независимая от дистрибутива система сборки для Linux, аналогичная [AUR](https://wiki.archlinux.org/title/Arch_User_Repository). В настоящее время она находится в стадии бета-тестирования. Исправлено большинство основных ошибок и добавлено большинство важных функций. alr готов к общему использованию, но все еще может время от времени ломаться или заменяться.

build.go

@@ -63,12 +63,12 @@ func BuildCmd() *cli.Command {
 			var script string
 
 			// Проверяем, установлен ли флаг script (-s)
-
+			if c.IsSet("script") {
-			switch {
-			case c.IsSet("script"):
 				script = c.String("script")
-			case c.IsSet("package"):
+			} else if c.IsSet("package") {
+				// Если флаг script не установлен, проверяем флаг package (-p)
 				packageInput := c.String("package")
+				// Определяем, содержит ли packageInput символ '/'
 				if filepath.Dir(packageInput) == "." {
 					// Не указана директория репозитория, используем 'default' как префикс
 					script = filepath.Join(config.GetPaths(ctx).RepoDir, "default", packageInput, "alr.sh")
@@ -76,7 +76,8 @@ func BuildCmd() *cli.Command {
 					// Используем путь с указанным репозиторием
 					script = filepath.Join(config.GetPaths(ctx).RepoDir, packageInput, "alr.sh")
 				}
-			default:
+			} else {
+				// Ни флаги script, ни package не установлены, используем дефолтный скрипт
 				script = filepath.Join(config.GetPaths(ctx).RepoDir, "alr.sh")
 			}
 

coverage-badge.svg

@@ -1,17 +0,0 @@
-<svg xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" width="109" height="20">
-    <linearGradient id="smooth" x2="0" y2="100%"><stop offset="0" stop-color="#bbb" stop-opacity=".1"/>
-    <stop offset="1" stop-opacity=".1"/></linearGradient>
-    <mask id="round">
-        <rect width="109" height="20" rx="3" fill="#fff"/>
-    </mask>
-    <g mask="url(#round)"><rect width="65" height="20" fill="#555"/>
-        <rect x="65" width="44" height="20" fill="#e05d44"/>
-        <rect width="109" height="20" fill="url(#smooth)"/>
-    </g>
-    <g fill="#fff" text-anchor="middle" font-family="DejaVu Sans,Verdana,Geneva,sans-serif" font-size="11">
-        <text x="33.5" y="15" fill="#010101" fill-opacity=".3">coverage</text>
-        <text x="33.5" y="14">coverage</text>
-        <text x="86" y="15" fill="#010101" fill-opacity=".3">14.4%</text>
-        <text x="86" y="14">14.4%</text>
-    </g>
-</svg>

internal/config/lang.go

@@ -42,19 +42,18 @@ var (
 // Subsequent calls will just return the same value.
 func Language(ctx context.Context) language.Tag {
 	langMtx.Lock()
+	defer langMtx.Unlock()
 	if !langSet {
 		syslang := SystemLang()
 		tag, err := language.Parse(syslang)
 		if err != nil {
 			slog.Error(gotext.Get("Error parsing system language"), "err", err)
-			langMtx.Unlock()
 			os.Exit(1)
 		}
 		base, _ := tag.Base()
 		lang = language.Make(base.String())
 		langSet = true
 	}
-	langMtx.Unlock()
 	return lang
 }
 

internal/config/paths.go

@@ -38,7 +38,7 @@ type Paths struct {
 // using information from the system.
 // Subsequent calls will return the same value.
 //
-// Deprecated: use struct API
+// Depreacted: use struct API
 func GetPaths(ctx context.Context) *Paths {
 	alrConfig := GetInstance(ctx)
 	return alrConfig.GetPaths(ctx)

internal/cpu/cpu.go

@@ -38,12 +38,11 @@ func armVariant() string {
 		return armEnv
 	}
 
-	switch {
+	if cpu.ARM.HasVFPv3 {
-	case cpu.ARM.HasVFPv3:
 		return "arm7"
-	case cpu.ARM.HasVFP:
+	} else if cpu.ARM.HasVFP {
 		return "arm6"
-	default:
+	} else {
 		return "arm5"
 	}
 }

internal/db/db.go

@@ -129,10 +129,7 @@ func (d *Database) initDB(ctx context.Context) error {
 	ver, ok := d.GetVersion(ctx)
 	if ok && ver != CurrentVersion {
 		slog.Warn(gotext.Get("Database version mismatch; resetting"), "version", ver, "expected", CurrentVersion)
-		err = d.reset(ctx)
+		d.reset(ctx)
-		if err != nil {
-			return err
-		}
 		return d.initDB(ctx)
 	} else if !ok {
 		slog.Warn(gotext.Get("Database version does not exist. Run alr fix if something isn't working."), "version", ver, "expected", CurrentVersion)

internal/osutils/move.go

@@ -55,12 +55,12 @@ func copyDirOrFile(sourcePath, destPath string) error {
 		return err
 	}
 
-	switch {
+	if sourceInfo.IsDir() {
-	case sourceInfo.IsDir():
 		return copyDir(sourcePath, destPath, sourceInfo)
-	case sourceInfo.Mode().IsRegular():
+	} else if sourceInfo.Mode().IsRegular() {
 		return copyFile(sourcePath, destPath, sourceInfo)
-	default:
+	} else {
+		// ignore non-regular files
 		return nil
 	}
 }

internal/shutils/helpers/helpers.go

@@ -245,13 +245,10 @@ func gitVersionCmd(hc interp.HandlerContext, cmd string, args []string) error {
 		return fmt.Errorf("git-version: %w", err)
 	}
 
-	err = commits.ForEach(func(*object.Commit) error {
+	commits.ForEach(func(*object.Commit) error {
 		revNum++
 		return nil
 	})
-	if err != nil {
-		return fmt.Errorf("git-version: %w", err)
-	}
 
 	HEAD, err := r.Head()
 	if err != nil {

internal/translations/default.pot

@@ -26,23 +26,23 @@ msgid ""
 "Build package from scratch even if there's an already built package available"
 msgstr ""
 
-#: build.go:87
+#: build.go:80
 msgid "Error pulling repositories"
 msgstr ""
 
-#: build.go:95
+#: build.go:87
 msgid "Unable to detect a supported package manager on the system"
 msgstr ""
 
-#: build.go:107
+#: build.go:98
 msgid "Error building package"
 msgstr ""
 
-#: build.go:114
+#: build.go:104
 msgid "Error getting working directory"
 msgstr ""
 
-#: build.go:123
+#: build.go:112
 msgid "Error moving the package"
 msgstr ""
 
@@ -218,7 +218,7 @@ msgstr ""
 msgid "Unable to create package cache directory"
 msgstr ""
 
-#: internal/config/lang.go:49
+#: internal/config/lang.go:50
 msgid "Error parsing system language"
 msgstr ""
 
@@ -226,7 +226,7 @@ msgstr ""
 msgid "Database version mismatch; resetting"
 msgstr ""
 
-#: internal/db/db.go:138
+#: internal/db/db.go:135
 msgid ""
 "Database version does not exist. Run alr fix if something isn't working."
 msgstr ""
@@ -347,27 +347,27 @@ msgstr ""
 msgid "Executing build()"
 msgstr ""
 
-#: pkg/build/build.go:487
+#: pkg/build/build.go:489
 msgid "Executing package()"
 msgstr ""
 
-#: pkg/build/build.go:509
+#: pkg/build/build.go:527
 msgid "Executing files()"
 msgstr ""
 
-#: pkg/build/build.go:587
+#: pkg/build/build.go:605
 msgid "AutoProv is not implemented for this package format, so it's skipped"
 msgstr ""
 
-#: pkg/build/build.go:598
+#: pkg/build/build.go:616
 msgid "AutoReq is not implemented for this package format, so it's skipped"
 msgstr ""
 
-#: pkg/build/build.go:705
+#: pkg/build/build.go:723
 msgid "Would you like to remove the build dependencies?"
 msgstr ""
 
-#: pkg/build/build.go:811
+#: pkg/build/build.go:829
 msgid "The checksums array must be the same length as sources"
 msgstr ""
 

internal/translations/po/ru/default.po

@@ -33,23 +33,23 @@ msgid ""
 "Build package from scratch even if there's an already built package available"
 msgstr "Создайте пакет с нуля, даже если уже имеется готовый пакет"
 
-#: build.go:87
+#: build.go:80
 msgid "Error pulling repositories"
 msgstr "Ошибка при извлечении репозиториев"
 
-#: build.go:95
+#: build.go:87
 msgid "Unable to detect a supported package manager on the system"
 msgstr "Не удалось обнаружить поддерживаемый менеджер пакетов в системе"
 
-#: build.go:107
+#: build.go:98
 msgid "Error building package"
 msgstr "Ошибка при сборке пакета"
 
-#: build.go:114
+#: build.go:104
 msgid "Error getting working directory"
 msgstr "Ошибка при получении рабочего каталога"
 
-#: build.go:123
+#: build.go:112
 msgid "Error moving the package"
 msgstr "Ошибка при перемещении пакета"
 
@@ -229,7 +229,7 @@ msgstr "Не удалось создать каталог кэша репози
 msgid "Unable to create package cache directory"
 msgstr "Не удалось создать каталог кэша пакетов"
 
-#: internal/config/lang.go:49
+#: internal/config/lang.go:50
 msgid "Error parsing system language"
 msgstr "Ошибка при парсинге языка системы"
 
@@ -237,7 +237,7 @@ msgstr "Ошибка при парсинге языка системы"
 msgid "Database version mismatch; resetting"
 msgstr "Несоответствие версий базы данных; сброс настроек"
 
-#: internal/db/db.go:138
+#: internal/db/db.go:135
 msgid ""
 "Database version does not exist. Run alr fix if something isn't working."
 msgstr ""
@@ -363,29 +363,29 @@ msgstr "Исполнение prepare()"
 msgid "Executing build()"
 msgstr "Исполнение build()"
 
-#: pkg/build/build.go:487
+#: pkg/build/build.go:489
 msgid "Executing package()"
 msgstr "Исполнение package()"
 
-#: pkg/build/build.go:509
+#: pkg/build/build.go:527
 msgid "Executing files()"
 msgstr "Исполнение files()"
 
-#: pkg/build/build.go:587
+#: pkg/build/build.go:605
 msgid "AutoProv is not implemented for this package format, so it's skipped"
 msgstr ""
 "AutoProv не реализовано для этого формата пакета, поэтому будет пропущено"
 
-#: pkg/build/build.go:598
+#: pkg/build/build.go:616
 msgid "AutoReq is not implemented for this package format, so it's skipped"
 msgstr ""
 "AutoReq не реализовано для этого формата пакета, поэтому будет пропущено"
 
-#: pkg/build/build.go:705
+#: pkg/build/build.go:723
 msgid "Would you like to remove the build dependencies?"
 msgstr "Хотели бы вы удалить зависимости сборки?"
 
-#: pkg/build/build.go:811
+#: pkg/build/build.go:829
 msgid "The checksums array must be the same length as sources"
 msgstr "Массив контрольных сумм должен быть той же длины, что и источники"
 

pkg/build/build.go

@@ -200,14 +200,14 @@ func BuildPackage(ctx context.Context, opts types.BuildOpts) ([]string, []string
 
 	// Добавляем путь и имя только что собранного пакета в
 	// соответствующие срезы
-	builtPaths = append(builtPaths, pkgPath)
+	pkgPaths := append(builtPaths, pkgPath)
-	builtNames = append(builtNames, vars.Name)
+	pkgNames := append(builtNames, vars.Name)
 
 	// Удаляем дубликаты из pkgPaths и pkgNames.
 	// Дубликаты могут появиться, если несколько зависимостей
 	// зависят от одних и тех же пакетов.
-	pkgPaths := removeDuplicates(builtPaths)
+	pkgPaths = removeDuplicates(pkgPaths)
-	pkgNames := removeDuplicates(builtNames)
+	pkgNames = removeDuplicates(pkgNames)
 
 	return pkgPaths, pkgNames, nil // Возвращаем пути и имена пакетов
 }
@@ -482,13 +482,31 @@ func executeFunctions(ctx context.Context, dec *decoder.Decoder, dirs types.Dire
 		}
 	}
 
-	packageFn, ok := dec.GetFunc("package")
+	// Выполнение всех функций, начинающихся с package_
-	if ok {
+	for {
-		slog.Info(gotext.Get("Executing package()"))
+		packageFn, ok := dec.GetFunc("package")
-		err := packageFn(ctx, interp.Dir(dirs.SrcDir))
+		if ok {
-		if err != nil {
+			slog.Info(gotext.Get("Executing package()"))
-			return nil, err
+			err := packageFn(ctx, interp.Dir(dirs.SrcDir))
+			if err != nil {
+				return nil, err
+			}
 		}
+
+		/*
+			// Проверка на наличие дополнительных функций package_*
+			packageFuncName := "package_"
+			if packageFunc, ok := dec.GetFunc(packageFuncName); ok {
+				slog.Info("Executing " + packageFuncName)
+				err = packageFunc(ctx, interp.Dir(dirs.SrcDir))
+				if err != nil {
+					return err
+				}
+			} else {
+				break // Если больше нет функций package_*, выходим из цикла
+			}
+		*/
+		break
 	}
 
 	output := &FunctionsOutput{}

pkg/build/build_internal_test.go

@@ -20,6 +20,8 @@ import (
 	"context"
 	"testing"
 
+	"github.com/stretchr/testify/assert"
+
 	"gitea.plemya-x.ru/Plemya-x/ALR/internal/db"
 	"gitea.plemya-x.ru/Plemya-x/ALR/internal/types"
 	"gitea.plemya-x.ru/Plemya-x/ALR/pkg/manager"
@@ -134,7 +136,6 @@ func (m *TestManager) IsInstalled(pkg string) (bool, error) {
 	return true, nil
 }
 
-// TODO: fix test
 func TestInstallBuildDeps(t *testing.T) {
 	type testEnv struct {
 		pf   PackageFinder
@@ -142,7 +143,7 @@ func TestInstallBuildDeps(t *testing.T) {
 		opts types.BuildOpts
 
 		// Contains pkgs captured by FindPkgsFunc
-		// capturedPkgs []string
+		capturedPkgs []string
 	}
 
 	type testCase struct {
@@ -152,62 +153,60 @@ func TestInstallBuildDeps(t *testing.T) {
 	}
 
 	for _, tc := range []testCase{
-		/*
+		{
-			{
+			Name: "install only needed deps",
-				Name: "install only needed deps",
+			Prepare: func() *testEnv {
-				Prepare: func() *testEnv {
+				pf := TestPackageFinder{}
-					pf := TestPackageFinder{}
+				vars := types.BuildVars{}
-					vars := types.BuildVars{}
+				m := TestManager{}
-					m := TestManager{}
+				opts := types.BuildOpts{
-					opts := types.BuildOpts{
+					Manager:     &m,
-						Manager:     &m,
+					Interactive: false,
-						Interactive: false,
+				}
+
+				env := &testEnv{
+					pf:           &pf,
+					vars:         &vars,
+					opts:         opts,
+					capturedPkgs: []string{},
+				}
+
+				pf.FindPkgsFunc = func(ctx context.Context, pkgs []string) (map[string][]db.Package, []string, error) {
+					env.capturedPkgs = append(env.capturedPkgs, pkgs...)
+					result := make(map[string][]db.Package)
+					result["bar"] = []db.Package{{
+						Name: "bar-pkg",
+					}}
+					result["buz"] = []db.Package{{
+						Name: "buz-pkg",
+					}}
+
+					return result, []string{}, nil
+				}
+
+				vars.BuildDepends = []string{
+					"foo",
+					"bar",
+					"buz",
+				}
+				m.IsInstalledFunc = func(pkg string) (bool, error) {
+					if pkg == "foo" {
+						return true, nil
+					} else {
+						return false, nil
 					}
+				}
 
-					env := &testEnv{
+				return env
-						pf:           &pf,
-						vars:         &vars,
-						opts:         opts,
-						capturedPkgs: []string{},
-					}
-
-					pf.FindPkgsFunc = func(ctx context.Context, pkgs []string) (map[string][]db.Package, []string, error) {
-						env.capturedPkgs = append(env.capturedPkgs, pkgs...)
-						result := make(map[string][]db.Package)
-						result["bar"] = []db.Package{{
-							Name: "bar-pkg",
-						}}
-						result["buz"] = []db.Package{{
-							Name: "buz-pkg",
-						}}
-
-						return result, []string{}, nil
-					}
-
-					vars.BuildDepends = []string{
-						"foo",
-						"bar",
-						"buz",
-					}
-					m.IsInstalledFunc = func(pkg string) (bool, error) {
-						if pkg == "foo" {
-							return true, nil
-						} else {
-							return false, nil
-						}
-					}
-
-					return env
-				},
-				Expected: func(t *testing.T, e *testEnv, res []string, err error) {
-					assert.NoError(t, err)
-					assert.Len(t, res, 2)
-					assert.ElementsMatch(t, res, []string{"bar-pkg", "buz-pkg"})
-
-					assert.ElementsMatch(t, e.capturedPkgs, []string{"bar", "buz"})
-				},
 			},
-		*/
+			Expected: func(t *testing.T, e *testEnv, res []string, err error) {
+				assert.NoError(t, err)
+				assert.Len(t, res, 2)
+				assert.ElementsMatch(t, res, []string{"bar-pkg", "buz-pkg"})
+
+				assert.ElementsMatch(t, e.capturedPkgs, []string{"bar", "buz"})
+			},
+		},
 	} {
 		t.Run(tc.Name, func(tt *testing.T) {
 			ctx := context.Background()

pkg/repos/pull.go

@@ -201,33 +201,34 @@ func (rs *Repos) processRepoChanges(ctx context.Context, repo types.Repo, r *git
 			continue
 		}
 
-		switch {
+		if to == nil {
-		case to == nil:
 			actions = append(actions, action{
 				Type: actionDelete,
 				File: from.Path(),
 			})
-		case from == nil:
+		} else if from == nil {
 			actions = append(actions, action{
 				Type: actionUpdate,
 				File: to.Path(),
 			})
-		case from.Path() != to.Path():
+		} else {
-			actions = append(actions,
+			if from.Path() != to.Path() {
-				action{
+				actions = append(actions,
-					Type: actionDelete,
+					action{
-					File: from.Path(),
+						Type: actionDelete,
-				},
+						File: from.Path(),
-				action{
+					},
+					action{
+						Type: actionUpdate,
+						File: to.Path(),
+					},
+				)
+			} else {
+				actions = append(actions, action{
 					Type: actionUpdate,
 					File: to.Path(),
-				},
+				})
-			)
+			}
-		default:
-			actions = append(actions, action{
-				Type: actionUpdate,
-				File: to.Path(),
-			})
 		}
 	}
 

scripts/coverage-badge.sh

@@ -1,46 +0,0 @@
-# ALR - Any Linux Repository
-# Copyright (C) 2025 Евгений Храмов
-#
-# This program is free software: you can redistribute it and/or modify
-# it under the terms of the GNU General Public License as published by
-# the Free Software Foundation, either version 3 of the License, or
-# (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program.  If not, see <http://www.gnu.org/licenses/>.
-
-
-#!/bin/bash
-COVERAGE=$(go tool cover -func=coverage.out | grep total | awk '{print $3}' | sed 's/%//')
-
-COLOR="#4c1"
-if (( $(echo "$COVERAGE < 50" | bc -l) )); then
-    COLOR="#e05d44"
-elif (( $(echo "$COVERAGE < 80" | bc -l) )); then
-    COLOR="#dfb317"
-fi
-
-cat <<EOF > coverage-badge.svg
-<svg xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" width="109" height="20">
-    <linearGradient id="smooth" x2="0" y2="100%"><stop offset="0" stop-color="#bbb" stop-opacity=".1"/>
-    <stop offset="1" stop-opacity=".1"/></linearGradient>
-    <mask id="round">
-        <rect width="109" height="20" rx="3" fill="#fff"/>
-    </mask>
-    <g mask="url(#round)"><rect width="65" height="20" fill="#555"/>
-        <rect x="65" width="44" height="20" fill="${COLOR}"/>
-        <rect width="109" height="20" fill="url(#smooth)"/>
-    </g>
-    <g fill="#fff" text-anchor="middle" font-family="DejaVu Sans,Verdana,Geneva,sans-serif" font-size="11">
-        <text x="33.5" y="15" fill="#010101" fill-opacity=".3">coverage</text>
-        <text x="33.5" y="14">coverage</text>
-        <text x="86" y="15" fill="#010101" fill-opacity=".3">${COVERAGE}%</text>
-        <text x="86" y="14">${COVERAGE}%</text>
-    </g>
-</svg>
-EOF