Bug 59747
| Summary: | Плохо приспособлена для установки из пакета | ||
|---|---|---|---|
| Product: | Sisyphus | Reporter: | Alexei V. Mezin <alexei.mezin> |
| Component: | proxsave | Assignee: | Aleksandr Shamaraev <shad> |
| Status: | REOPENED --- | QA Contact: | qa-sisyphus |
| Severity: | enhancement | ||
| Priority: | P5 | CC: | shad |
| Version: | unstable | ||
| Hardware: | x86_64 | ||
| OS: | Linux | ||
| URL: | https://github.com/tis24dev/proxsave/issues/251 | ||
Добрый день. Да, вы правы, /opt/ "прибито гвоздями", я подумаю что можно сделать, и думаю также можно будет issue апстриму написать по этому поводу. Добрый день. Я исправил отображение версии, также должен идти пропуск проверки обновления, и по идее сейчас не должны отображаться варнинги по пермишн и файлу конфигурации. Не могли бы Вы проверить задание Task #424551 for sisyphus by shad Отключение варнингов это конечно костыль, но учитывая, что на работу это не влияет, то хоть глаза не будет мозолить. Если выставить права как хочет апстрим, то proxsave будет работать только из под рута, а это на мой взгляд не совсем правильно... хотя кто знает какая логика была у разработчика ну а так, дальше только смотреть, что ответит апстим. Доброго дня. Со слов апстим: 1. Permissions поправят в следующем релизе, в настоящий момент это не влияет на работу, можно игнорировать предупреждение 2. на счет BASE_DIR , аптрим предлагает удалить данную строчку из backup.env, так как параметр устаревший и определяется автоматически по местоположению файла В общем учитывая все это, а также исправление отображение версии и игнорирование проверки обновления, багу закрою заданием как исправленную. proxsave-0.29.0-alt2 -> sisyphus: Wed Jul 08 2026 Aleksandr Shamaraev <shad@altlinux> 0.29.0-alt2 - fixed: version number - skipped: update check - disabled some warnings (ALT #59747) Рановато закрывать баг, как мне кажется. Нужно продолжать далее обучать апстрим. Если он, конечно, окажется обучаемым.
> 2. на счет BASE_DIR , аптрим предлагает удалить данную строчку из
> backup.env, так как параметр устаревший и определяется автоматически по
> местоположению файла
Врут. Нагло врут. Можно удалить этот параметр из конфига. Но что там определяется по местоположению файла? Файл у нас (в полном соответствии с FHS), лежит в /usr/bin
Однако, программа при запуске создает /opt/proxsave, причем без вариантов. Из какого места она вычисляет этот каталог? Сдается мне, что из собственных исходников. И в нем складывает кучу барахла. Включая логи и попытки размещения lock-файла. Автором программы все же стоит прочитать FHS.
Вот выхлоп версии alt2:
ISSUES:
Errors: 0
Warnings: 5
Total Issues: 5
Log: /opt/proxsave/log/backup-e-sexy.hostname-20260713-000553.log
Прекрасное место размещения логов!
ISSUE DETAILS:
- [WARNING] Directory /mnt/pve/truenas/pve_backup/e-sexy/ should be owned by root:root (count: 1)
Может админ лучше знает, какие права у него на НФС-шару?
- [WARNING] Security warning: executable hash mismatch for /usr/bin/proxsave (count: 1)
Проверка хэшей в системе с пакетным менаджером должна выполняться пакетным менаджером, если это зачем-то нужно. То есть интернет-проверки оторваны не до конца.
- [WARNING] Security warning: incorrect permissions on /opt/proxsave/identity (current 750, expected 700) (count: 1)
А вот это вообще смешно: /opt/proxsave я перед запуском программы удалил. То есть она сама создала этот каталог и всю его начинку. И сама же ругается на пермишены.
Так что там много чего еще есть, о чем стоит сообщать апстриму. :(
Добрый день. Я напишу апстриму, как будет немного посвободнее со временем... или посмотрю в исходниках.. можно ли что-то сделать... Но как уже говорил, апстрим прибил гвоздями этот путь, так как считает его путем по умолчанию. reopened |
Description
Alexei V. Mezin 2026-07-07 02:49:24 MSKПостоянно лезут warnings. Оно, конечно, не ошибки, почти на работу не влияет. Да и проблема очевидным образом в голове апстрима. Но перечислить надо, хотя бы самые плохие предупреждения: Типичный лог: ISSUES: Errors: 0 Warnings: 6 Total Issues: 6 1. [WARNING] Ignoring deprecated BASE_DIR from backup.env ("/run/proxsave/"); using detected base directory /opt/proxsave Такое впечатление, что было хорошо, но гвоздями прибили использовать /opt/proxsave. Кажется, это там настолько прибито гвоздями, что даже удивительно, зачем так сделано. 2. [WARNING] Executable /usr/bin/proxsave should have permissions 700 (current 755) Какое дело этой программе до пермишенов в /usr/bin? 3. [WARNING] New ProxSave version 0.29.0 (current 0.28.0.0+gdev) Незачем программам лазить в интернет по такому поводу. Надо вылечить эти ошибки на уровне нашего пакета? Или запинать апстрим, чтобы он как-то более грамотно подошел в проблема упаковки программы в пакет.