Bug 48351

Summary: [4.2] join serjigva@
Product: Team Accounts Reporter: Sergey Gvozdetskiy <gvozdeckysa>
Component: joinAssignee: Gleb F-Malinovskiy <glebfm>
Status: ASSIGNED --- QA Contact: Andrey Cherepanov <cas>
Severity: normal    
Priority: P5 CC: glebfm, grenka, ldv, rider
Version: unspecified   
Hardware: x86_64   
OS: Linux   
URL: https://altlinux.org/Team/Join
Attachments:
Description Flags
public SSH-key
none
public GPG-key none

Description Sergey Gvozdetskiy 2023-11-07 12:07:52 MSK
псевдоним: serjigva
e-mail: sgvozdetskiy@yahoo.com
ментор: Григорий Устинов <grenka@altlinux.org>
цель: собрать пакет(ы), помочь со сборкой чего-либо.
Comment 1 Anton Farygin 2023-11-07 19:51:51 MSK
Нужно дооформить заявку в соответствии с правилами - см. Создание заявки:
https://www.altlinux.org/Team/Join/Candidate
Comment 2 Sergey Gvozdetskiy 2023-11-08 17:58:33 MSK
Created attachment 14991 [details]
public SSH-key

type: ed25519
length: 256
Comment 3 Sergey Gvozdetskiy 2023-11-09 16:49:30 MSK
Created attachment 15005 [details]
public GPG-key

type: RSA & RSA
length: 4096
Comment 4 Grigory Ustinov 2023-11-13 10:26:42 MSK
Менторство подтверждаю.
Comment 5 Grigory Ustinov 2023-11-24 17:24:20 MSK
Прошу перевести на следующий этап.
Comment 6 Andrey Cherepanov 2023-12-06 12:03:32 MSK
(Ответ для Grigory Ustinov на комментарий #5)
> Прошу перевести на следующий этап.

Уважаемый секретарь, ping.
Comment 7 Gleb F-Malinovskiy 2023-12-07 22:58:51 MSK
ssh ключ на gitery.alt зарегистрирован.
Адрес для пересылки создан.

T/J/S -> 2.3.
Comment 8 Grigory Ustinov 2023-12-08 06:43:50 MSK
Прошу перевести на следующий этап.
Comment 9 Gleb F-Malinovskiy 2023-12-09 23:43:28 MSK
ssh ключ на gyle.alt зарегистрирован.
Пакет alt-gpgkeys обновлён.
Адрес подписан на devel@.

T/J/S -> 3.6.
Comment 10 Sergey Gvozdetskiy 2024-01-19 14:54:45 MSK
Просьба переназначить e-mail адрес рассылки с личного (sgvozdetskiy@yahoo.com) на рабочий (gvozdeckysa@basalt.ru).
письма от сборочницы приходят в личный ящик, а там попадают в спам, что неудобно.
спасибо!
Comment 11 Sergey Gvozdetskiy 2024-02-02 11:12:47 MSK
@glebfm, прошу оказать содействие по комментарию выше.
Comment 12 Gleb F-Malinovskiy 2024-02-02 23:50:57 MSK
(In reply to Sergey Gvozdetskiy from comment #10)
> Просьба переназначить e-mail адрес
Готово.
Comment 13 Sergey Gvozdetskiy 2024-02-26 17:16:11 MSK
@glebfm,
тысячу извинений,
неправильно написал адрес, правильно так: gvozdeckysa@basealt.ru
а то совсем перестал получать письма.
Comment 14 Gleb F-Malinovskiy 2024-02-26 23:29:01 MSK
(In reply to Sergey Gvozdetskiy from comment #13)
> @glebfm,
> тысячу извинений,
> неправильно написал адрес
Ой, я тоже не заметил.  Готово.
Comment 15 Sergey Gvozdetskiy 2024-07-10 12:12:47 MSK
на текущий момент были собраны и/или обновлены следующие пакеты:
=Sisyphus initial=
Alaya
albert
ltfs
munge
python3-module-cramjam
xsr

=Update/Backport=
dialog
liblnk
mono
newt52
pdfgrep
php8.2-pdo_sqlsrv
polybar
rabbitvcs
whdd
Comment 16 Grigory Ustinov 2024-07-10 16:05:30 MSK
Сергей, приложите пожалуйста ссылки на таски, чтобы было удобнее просматривать ваши труды.
Comment 17 Sergey Gvozdetskiy 2024-07-12 17:10:32 MSK
Да, пропустил, исправлю.
Задачи без таска в квадратных скобках, с упоминанием заинтересованного лица.

=Sisyphus initial=
Alaya #39054
albert #47237
ltfs #48688
munge #37332
python3-module-cramjam [лабораторная без таска от @grenka]
xsr #49766

=Update/Backport=
dialog [@cas]
liblnk #45855
mono #46650
newt52 [@cas]
pdfgrep [@cas]
php8.2-pdo_sqlsrv [@cas]
polybar #50025
rabbitvcs #30354
whdd [dep для dialog]
Comment 18 Grigory Ustinov 2024-07-12 18:11:13 MSK
Мдямс.. В общем, кандидат освоил базовые навыки сборки пакетов, правильно прикладывать ссылки на таски научится возможно чуть позже. Считаю, что пора призывать рецензента.
Comment 19 Andrey Cherepanov 2024-07-26 11:28:18 MSK
(Ответ для Grigory Ustinov на комментарий #18)
> Мдямс.. В общем, кандидат освоил базовые навыки сборки пакетов, правильно
> прикладывать ссылки на таски научится возможно чуть позже. Считаю, что пора
> призывать рецензента.

Прошу перейти на шаг 4.1.
Comment 20 Sergey Gvozdetskiy 2024-09-23 12:18:56 MSK
Уважаемый секретарь,
пожалуйста окажите содействие, потребность в самостоятельном продвижении коммитов с каждым днём острее, на пустяковые вещи приходится просить аппрувы всё чаще.
Comment 21 Gleb F-Malinovskiy 2024-09-24 19:33:34 MSK
Призван рецензент (rider@) для независимой оценки готовности кандидата.

T/J/S -> 4.2.
Comment 22 Anton Farygin 2024-09-26 08:35:38 MSK
+* Wed Sep 25 2024 Sergey Gvozdetskiy <serjigva@altlinux.org> 1.32.0-alt2
+- WebUI settings by default
+- Add UI pkg Requirement

точки в конце предложений потеряли.
и это во всех пакетах, видимо.
Comment 23 Anton Farygin 2024-09-26 08:46:44 MSK
я расширил и упростил чтение в разделе "лучшие практики" руководства по написанию changelog, рекомендую ещё раз ознакомиться.
https://www.altlinux.org/%D0%A0%D1%83%D0%BA%D0%BE%D0%B2%D0%BE%D0%B4%D1%81%D1%82%D0%B2%D0%BE_%D0%BF%D0%BE_%D0%BD%D0%B0%D0%BF%D0%B8%D1%81%D0%B0%D0%BD%D0%B8%D1%8E_changelog
Comment 24 Anton Farygin 2024-09-26 08:56:00 MSK
Все библиотеки, собранные в задании с обновлением стека aws - собраны с нарушением SharedLibsPolicy (другим сопровождающим).

Пока на них не так много зависящих пакетов следует при обновлении этих библиотек внести изменения в упаковку для полного соответствия именования пакетов с этой политикой.
Comment 25 Anton Farygin 2024-09-26 08:56:56 MSK
Список зависящих пакетов можно посмотреть тут: 
https://packages.altlinux.org/ru/sisyphus/srpms/libaws-c-common/what_depends/by_source/3118015234079199220
Comment 26 Anton Farygin 2024-09-26 08:58:18 MSK
https://bugzilla.altlinux.org/45855 - было бы неплохо закрыть эту ошибку, присылайте результат на review
Comment 27 Anton Farygin 2024-09-26 09:04:16 MSK
https://git.altlinux.org/tasks/archive/done/_348/356416/gears/100/git?p=git;a=blob;f=nfdump.spec;h=6c290a60f009ee0698ae21901f96d9c81f43f36c;hb=80fe969616e5f95a8281c455688b7cc68a7a6b0c

При изменении даже чужих пакетов стоит вычищать комментарии, похожие на мусор.
  22 #Url: http://sourceforge.net/projects/nfdump/
  23 Url: https://github.com/phaag/nfdump/releases

при наличии апстрима в git очень рекомендуется переводить пакет на сборку из апстримного гита - это сильно упростит в дальнейшем аналитику кода и сделанных изменений.
Comment 28 Anton Farygin 2024-09-26 09:07:47 MSK
https://git.altlinux.org/tasks/355604/gears/200/git?p=git;a=blob;f=.gear/wlmaker.spec;h=6cab4d5574ff14670a78923d1a471eb84b634291;hb=d4742149ac7ba38e26ce525f28a96e4ce260b324
Тэг Packager сейчас генериться автоматически и лучше его для пакетов не использовать, а если встречаете - вычищать.

Моё замечание про Changelog'и касается всех ваших пакетов. Их все надо поправить, но изменения отправлять в репозиторий только в случае наличия другой необходимости.

Для ментора - просьба обращать внимание при review пакетов кандидатов на это.
Comment 29 Anton Farygin 2024-09-26 09:10:01 MSK
https://git.altlinux.org/tasks/355604/gears/200/git?p=git;a=tree;f=.gear;h=f4bac1734f6c06431390155a32f38058bf1e488c;hb=d4742149ac7ba38e26ce525f28a96e4ce260b324

https://git.altlinux.org/people/serjigva/packages/?p=wlmaker.git;a=tree;f=.gear;h=f4bac1734f6c06431390155a32f38058bf1e488c;hb=sisyphus

когда каким-то необычным образом упаковываете submodules -то стоит или сделать инструкцию по их обновлению или положить какой-то скриптик в git репозиторий, который облегчит этот процесс другому сопровождающему.
Comment 30 Anton Farygin 2024-09-26 09:16:21 MSK
https://git.altlinux.org/tasks/archive/done/_345/353420/gears/100/git?p=git;a=blob;f=.gear/munge.spec;h=27e07dbb7412cd1a0e658b816de8afb494b39dd7;hb=5d39d2b820a0b74f44996210e3bc7e8915d6c837

Этот пакет (как и все остальные пакеты с библиотеками) нужно переделать для полного соответствия SharedLibsPolicy.

К замечаниям про Packager и Changelog добавляется просьба указывать в комментариях для отключаемых тестов причину отключения.
Comment 31 Anton Farygin 2024-09-26 09:17:14 MSK
Просьба все дальнейшие пакеты присылать на review уже мне (для оперативности можно использовать телеграм).
Comment 32 Grigory Ustinov 2024-09-26 09:56:38 MSK
(Ответ для Anton Farygin на комментарий #28)
> Для ментора - просьба обращать внимание при review пакетов кандидатов на это.

Антон, я пытаюсь. К великому сожалению, подопечные порой делают такую уйму более серьёзных ошибок, что на их фоне требование оформления ченджлога уже выглядит издёвкой. А некоторые вдовесок ещё и не понимают зачем это всё нужно. Пакет ведь собирается? Работает? Ну и что пристал? У меня и так репутация не самого лояльного ментора=)

Честно скажу, иногда гонять подопечного по десятку раз переделывать один и тот же пакет так выматывает, что на ченджлог уже не остаётся никаких сил.