Bug 46983 - [done] join krf10@
Summary: [done] join krf10@
Status: CLOSED FIXED
Alias: None
Product: Team Accounts
Classification: Development
Component: join (show other bugs)
Version: unspecified
Hardware: x86_64 Linux
: P5 normal
Assignee: Gleb F-Malinovskiy
QA Contact: Andrey Cherepanov
URL: https://altlinux.org/Team/Join
Keywords:
Depends on:
Blocks:
 
Reported: 2023-07-19 14:35 MSK by ratkin
Modified: 2024-11-02 11:01 MSK (History)
6 users (show)

See Also:


Attachments
ssh public key (93 bytes, application/vnd.ms-publisher)
2023-07-19 14:35 MSK, ratkin
no flags Details
gpg public key (2.98 KB, text/plain)
2023-07-19 14:36 MSK, ratkin
no flags Details
gpg public key (3.06 KB, text/plain)
2023-08-04 13:48 MSK, ratkin
no flags Details
new ssh public key (90 bytes, application/vnd.ms-publisher)
2023-10-31 10:03 MSK, ratkin
no flags Details
new gpg public key (3.06 KB, text/plain)
2023-10-31 10:06 MSK, ratkin
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description ratkin 2023-07-19 14:35:01 MSK
Created attachment 13881 [details]
ssh public key

Псевдоним : krf10
адрес пересылки почты : ratkinda@basealt.ru
имя ментора : Sergey Turchin zerg@
Цель : участие в разработке ПО
Comment 1 ratkin 2023-07-19 14:36:13 MSK
Created attachment 13882 [details]
gpg public key
Comment 2 Sergey V Turchin 2023-07-20 11:42:41 MSK
Подтверждаю. Согласен быть ментором.
Comment 3 Gleb F-Malinovskiy 2023-08-04 12:30:00 MSK
(In reply to ratkin from comment #1)
> Created attachment 13882 [details]
> gpg public key

Позвольте поинтересоваться, а как был сформирован этот файл?
Вы сами убрали -----BEGIN PGP PUBLIC KEY BLOCK----- и -----END PGP PUBLIC KEY BLOCK----- или воспользовались каким-то инструментом, который их куда-то дел?

В любом случае, ключ в таком виде не подойдёт.
Comment 4 ratkin 2023-08-04 13:38:13 MSK
Comment on attachment 13882 [details]
gpg public key

-----BEGIN PGP PUBLIC KEY BLOCK-----

mQINBGS3rssBEACwGjI3RZG9t2hMcpd79j4hvg/Wv3Q2Eul5S/QTmApD3Q02yc6U
sHoBco9LOwA4PxYTeR4RGGyiwMtu/BuB+X6rp5x8xcbmEyf3UeBEQpJg9cIUbS2Q
o+EfcWRK3PC7AUOAow0AqeFFQzFvOz9oOqd7bxFBtLaq6miTHwVZy0i63csjJok1
iKiNDIaYnm2Pay2Pvguo9kL3+XhgdD9dTli9h2qKDs33lo9bK+fBR6D2ey7S0wCd
bL5Dqd36lhD4tVx7BiLvElk8BdoQFG1kKGgmcFGfQWmL+T0GRj+tjxPl7GT/cQOW
0Indb4acX2NLloWiGNLpkAFCyTNFWYMTwF7hx9U2i7QkNCu7Gpe9phlaHsd8lr4N
eBftrJ9sniMh5Iw93G2c8Ppc1Jl5VVTqnhLSx1pqVI4GBYVvG/F10GbHM1waLMPG
oJeDb3gOFpMtKzr95fR8fAAbuZgv28LA32pBcNaGuqgvVcb+ELzY0i64F9gbLm87
YkeVpAotulglCGW4A5BiV21D1hOvzA02rtI2Ah7iUl51hacKy02jBQXAjeEBR43S
55R/PSBjyVC2q+VG7e/JarHEJHJyJjVbEvzcsmzh1nJVjJ4pZCu8GmCOznXPsnOD
Aqju+jVYbWmMnJeOshEJaJf4IPdAIxA02Rsn70YKi0rtQHZwu5QhjnNHsQARAQAB
tClEYW5paWwtVmlrdG9yIFJhdGtpbiA8a3JmMTBAYWx0bGludXgub3JnPokCOAQT
AQgAIgUCZLeuywIbAwYLCQgHAwIGFQgCCQoLBBYCAwECHgECF4AACgkQP6EFp0bW
FpEKeg/+PX0k/JUBXbboZnAubB2L7jwXqCIfKP7InxLd9esMaoCMPm0CbSANpIFa
pBySHkLBsIfgqZOzqdSvbI1tC5Qcjt+iqpAkqNsrVmb0Bi5tzCcJDDJjbOlBbZLY
Q65Y5deWmkOc2K222IHfK67VLdyJ6xibJwBUCTH6PzzqxcsbBsTqwtH9gbUnM33x
Tyq/onmRmEvFDuCyz9tacoSLyvs+CZj2577V3ls+2JYgZ5LXhpJTI2cJz3CX+ycM
L8mVfI2/XNYrea4YonVuCYqDxWhZTrdcRkXrb0oVLy7CYwQjnQDr5dAy5uDLaWcM
TZJ0JoRA0ZMu1/a3bH8bsgg4cWzpXnZ86+ryDNPChkxjmO6T+WRdScf06yykCqAg
pgZ9+XZE04EtApJLiGD0Mptn2teNe3EXjVN8jO50LcF64812ujLl4lp56CEb9AgJ
ntjuoE8NE8aEgcyw8fl28VAzUp8WjGvRgiwsP2fvpu6hlk9Va/8dM6n5OTyflp6A
lug5KN6ZiXlrQpICBJ/eNzQqJZUsDxMq4FQYzfGQT0GZhKanWcN5Phc2Rr8RFJ2U
HNlVMttqfQd6WVf4jMCL4j3jWS7N7phf77sEme7IeDyuhl7zIVKIlTEE/PAWkQoT
/TG6DrfjjJId8A90FyLPyk8Mwm+Ymhf36o3pV1+w7/q+M9ADniC5Ag0EZLeuywEQ
ANAJurkCqX//vsUgboClr/+1H2+bjmw8RL4Ccbqs8EDcQht0pjytNEYmjFLtPCDq
Gyvj7W1py4Xwi2ew8ev29Poj9z6Xmk0qfdlkSpSqWSRRXEOv23p3UcbKFSVngq4P
hRZeGH9OI+xD59TIdhn4+6AqkT2bmqp3A5nmVCE0IFbEp7WjwGWbbmzJgOhWjyHl
NsgaEdfg/zmL/d4rb09GRFZvkJ1V3RU+8Ime9ln9Qe4t/MBmlFtU+KKORIuc8H/G
HcE2CGXFJ8wPQH1jwpst27WuIafyhtt+90tXZw7XCerKOP6JiaQYiMZ8wtRT7g0c
9dcW+G37rbMfugxO0ospTSWeinlA41rLY+4MmVkitcmDp9TV0od/HvfUhs4Ewrup
bm80us1o4dKnS7+h3kLzjCCTPk9BlHW9sPSjhJnfpFGgcFojwrRjGDHD6EHkc6mD
pKnzV2vG3K55kofJ7bPb1FLq33w/TvLmBF4oy+/CF3hlIQw3nTzbiXW8krBT9/Xy
sgdbinWV9kduRZ9L2GRXac3xXiFr9iWBoxmJoB/tih/PfncU6T9yjr0l51Tx1PFe
6Qq5HEdcVnP0U1Rt+/hOg2sSDUXqMIPhILwDmft1RB4bQUYIKtCq4iDY9fvqW86e
0m+oJFgkLUHWRcba0oXA0ohb4e0kUoRh/68TLI13wSHxABEBAAGJAh8EGAEIAAkF
AmS3rssCGwwACgkQP6EFp0bWFpGqJQ//YGpg9EEX7w+xto9N00/tkBqfUzqwXVfo
F5BaiT8r1Itg+Yoz1aD11nKojZMQf3tvcTk0UkpnvggjhQ5ck7UB0PVsaSK23nrg
5tKGbc8R4sBwmoq3pdT3cgQjinTs+wuJwSixnR3zPKAB2KLTjILOm48KURuepZDv
MFld8sZT/WJE4xtcaWJNYzJrFHx/7dFiJud0DQLoYF5lmffMs7h2AjVJJNoNpTUO
eAevYTPpFqPKfdyUqwSPkggr8p+37uf/4SzLp+k9dmTsDsY0Gx7CUMT1xQ/XziCZ
TiunbMn3CZFmIXrfAf48fRBFJSVM1/QMUk9cARuWkV3Z3ssciOeinm5oLHZg+Rf4
byTROHR7r9tU15KnU9KJEr2RE6otmnR/HFaat+DifCDPGxnFoYTHX85qMFpiLYx8
+6CpyPjUZhoxUXVgS8f3rlM1213Db7pytuinbMlpqHYzONojkTx9ukG+bENC6oFI
WzNRnSUl/8kU8wSvZJ9COQkzEhXrvNZn1oLqkeHUfzgAgLF+5+FvHjvburkqfPp/
3JbdElGfKDRNATYj2BTW2fdjXh8Cuud2p+oHrl6j0snUo8l6kShrJJeZ1jmcqZ4R
HVK3DIwAYcNKSQgzf9JbGwwbKYWfsUuGYgQPqxbomk/YtYhUE69Jd+K68jBgSB0x
Cw3SbgazqBQ=
=YVUy
-----END PGP PUBLIC KEY BLOCK-----
Comment 5 ratkin 2023-08-04 13:42:12 MSK
Comment on attachment 13882 [details]
gpg public key

-----BEGIN PGP PUBLIC KEY BLOCK-----

mQINBGS3rssBEACwGjI3RZG9t2hMcpd79j4hvg/Wv3Q2Eul5S/QTmApD3Q02yc6U
sHoBco9LOwA4PxYTeR4RGGyiwMtu/BuB+X6rp5x8xcbmEyf3UeBEQpJg9cIUbS2Q
o+EfcWRK3PC7AUOAow0AqeFFQzFvOz9oOqd7bxFBtLaq6miTHwVZy0i63csjJok1
iKiNDIaYnm2Pay2Pvguo9kL3+XhgdD9dTli9h2qKDs33lo9bK+fBR6D2ey7S0wCd
bL5Dqd36lhD4tVx7BiLvElk8BdoQFG1kKGgmcFGfQWmL+T0GRj+tjxPl7GT/cQOW
0Indb4acX2NLloWiGNLpkAFCyTNFWYMTwF7hx9U2i7QkNCu7Gpe9phlaHsd8lr4N
eBftrJ9sniMh5Iw93G2c8Ppc1Jl5VVTqnhLSx1pqVI4GBYVvG/F10GbHM1waLMPG
oJeDb3gOFpMtKzr95fR8fAAbuZgv28LA32pBcNaGuqgvVcb+ELzY0i64F9gbLm87
YkeVpAotulglCGW4A5BiV21D1hOvzA02rtI2Ah7iUl51hacKy02jBQXAjeEBR43S
55R/PSBjyVC2q+VG7e/JarHEJHJyJjVbEvzcsmzh1nJVjJ4pZCu8GmCOznXPsnOD
Aqju+jVYbWmMnJeOshEJaJf4IPdAIxA02Rsn70YKi0rtQHZwu5QhjnNHsQARAQAB
tClEYW5paWwtVmlrdG9yIFJhdGtpbiA8a3JmMTBAYWx0bGludXgub3JnPokCOAQT
AQgAIgUCZLeuywIbAwYLCQgHAwIGFQgCCQoLBBYCAwECHgECF4AACgkQP6EFp0bW
FpEKeg/+PX0k/JUBXbboZnAubB2L7jwXqCIfKP7InxLd9esMaoCMPm0CbSANpIFa
pBySHkLBsIfgqZOzqdSvbI1tC5Qcjt+iqpAkqNsrVmb0Bi5tzCcJDDJjbOlBbZLY
Q65Y5deWmkOc2K222IHfK67VLdyJ6xibJwBUCTH6PzzqxcsbBsTqwtH9gbUnM33x
Tyq/onmRmEvFDuCyz9tacoSLyvs+CZj2577V3ls+2JYgZ5LXhpJTI2cJz3CX+ycM
L8mVfI2/XNYrea4YonVuCYqDxWhZTrdcRkXrb0oVLy7CYwQjnQDr5dAy5uDLaWcM
TZJ0JoRA0ZMu1/a3bH8bsgg4cWzpXnZ86+ryDNPChkxjmO6T+WRdScf06yykCqAg
pgZ9+XZE04EtApJLiGD0Mptn2teNe3EXjVN8jO50LcF64812ujLl4lp56CEb9AgJ
ntjuoE8NE8aEgcyw8fl28VAzUp8WjGvRgiwsP2fvpu6hlk9Va/8dM6n5OTyflp6A
lug5KN6ZiXlrQpICBJ/eNzQqJZUsDxMq4FQYzfGQT0GZhKanWcN5Phc2Rr8RFJ2U
HNlVMttqfQd6WVf4jMCL4j3jWS7N7phf77sEme7IeDyuhl7zIVKIlTEE/PAWkQoT
/TG6DrfjjJId8A90FyLPyk8Mwm+Ymhf36o3pV1+w7/q+M9ADniC5Ag0EZLeuywEQ
ANAJurkCqX//vsUgboClr/+1H2+bjmw8RL4Ccbqs8EDcQht0pjytNEYmjFLtPCDq
Gyvj7W1py4Xwi2ew8ev29Poj9z6Xmk0qfdlkSpSqWSRRXEOv23p3UcbKFSVngq4P
hRZeGH9OI+xD59TIdhn4+6AqkT2bmqp3A5nmVCE0IFbEp7WjwGWbbmzJgOhWjyHl
NsgaEdfg/zmL/d4rb09GRFZvkJ1V3RU+8Ime9ln9Qe4t/MBmlFtU+KKORIuc8H/G
HcE2CGXFJ8wPQH1jwpst27WuIafyhtt+90tXZw7XCerKOP6JiaQYiMZ8wtRT7g0c
9dcW+G37rbMfugxO0ospTSWeinlA41rLY+4MmVkitcmDp9TV0od/HvfUhs4Ewrup
bm80us1o4dKnS7+h3kLzjCCTPk9BlHW9sPSjhJnfpFGgcFojwrRjGDHD6EHkc6mD
pKnzV2vG3K55kofJ7bPb1FLq33w/TvLmBF4oy+/CF3hlIQw3nTzbiXW8krBT9/Xy
sgdbinWV9kduRZ9L2GRXac3xXiFr9iWBoxmJoB/tih/PfncU6T9yjr0l51Tx1PFe
6Qq5HEdcVnP0U1Rt+/hOg2sSDUXqMIPhILwDmft1RB4bQUYIKtCq4iDY9fvqW86e
0m+oJFgkLUHWRcba0oXA0ohb4e0kUoRh/68TLI13wSHxABEBAAGJAh8EGAEIAAkF
AmS3rssCGwwACgkQP6EFp0bWFpGqJQ//YGpg9EEX7w+xto9N00/tkBqfUzqwXVfo
F5BaiT8r1Itg+Yoz1aD11nKojZMQf3tvcTk0UkpnvggjhQ5ck7UB0PVsaSK23nrg
5tKGbc8R4sBwmoq3pdT3cgQjinTs+wuJwSixnR3zPKAB2KLTjILOm48KURuepZDv
MFld8sZT/WJE4xtcaWJNYzJrFHx/7dFiJud0DQLoYF5lmffMs7h2AjVJJNoNpTUO
eAevYTPpFqPKfdyUqwSPkggr8p+37uf/4SzLp+k9dmTsDsY0Gx7CUMT1xQ/XziCZ
TiunbMn3CZFmIXrfAf48fRBFJSVM1/QMUk9cARuWkV3Z3ssciOeinm5oLHZg+Rf4
byTROHR7r9tU15KnU9KJEr2RE6otmnR/HFaat+DifCDPGxnFoYTHX85qMFpiLYx8
+6CpyPjUZhoxUXVgS8f3rlM1213Db7pytuinbMlpqHYzONojkTx9ukG+bENC6oFI
WzNRnSUl/8kU8wSvZJ9COQkzEhXrvNZn1oLqkeHUfzgAgLF+5+FvHjvburkqfPp/
3JbdElGfKDRNATYj2BTW2fdjXh8Cuud2p+oHrl6j0snUo8l6kShrJJeZ1jmcqZ4R
HVK3DIwAYcNKSQgzf9JbGwwbKYWfsUuGYgQPqxbomk/YtYhUE69Jd+K68jBgSB0x
Cw3SbgazqBQ=
=YVUy
-----END PGP PUBLIC KEY BLOCK-----
Comment 6 Gleb F-Malinovskiy 2023-08-04 13:47:03 MSK
Приложите, пожалуйста, в виде нового attachment.
Comment 7 ratkin 2023-08-04 13:48:30 MSK
Created attachment 13981 [details]
gpg public key
Comment 8 Gleb F-Malinovskiy 2023-08-04 13:52:53 MSK
(In reply to ratkin from comment #7)
> Created attachment 13981 [details]
> gpg public key
Ok.

Но мне всё равно очень интересно, что случилось с предыдущим вариантом, я не просто так же спросил. :)
Comment 9 ratkin 2023-08-04 13:57:56 MSK
(Ответ для Gleb F-Malinovskiy на комментарий #8)
> (In reply to ratkin from comment #7)
> > Created attachment 13981 [details] [подробности] [details]
> > gpg public key
> Ok.
> 
> Но мне всё равно очень интересно, что случилось с предыдущим вариантом, я не
> просто так же спросил. :)
Просто не выделил те строки в ключе когда копировал)
Comment 10 Gleb F-Malinovskiy 2023-08-04 14:00:06 MSK
(In reply to ratkin from comment #9)
> (Ответ для Gleb F-Malinovskiy на комментарий #8)
> > Но мне всё равно очень интересно, что случилось с предыдущим вариантом, я не
> > просто так же спросил. :)
> Просто не выделил те строки в ключе когда копировал)

Понятно, я уж испугался, что какой-то инструмент стал так делать.
Comment 11 Sergey V Turchin 2023-08-14 13:44:24 MSK
Кандидат готов начать вступление.
Comment 12 Gleb F-Malinovskiy 2023-08-30 12:54:42 MSK
ssh ключ на gitery.alt зарегистрирован.
Адрес для пересылки создан.

T/J/S -> 2.3.
Comment 13 ratkin 2023-10-31 10:03:25 MSK
Created attachment 14935 [details]
new ssh public key
Comment 14 ratkin 2023-10-31 10:06:09 MSK
Created attachment 14936 [details]
new gpg public key
Comment 15 ratkin 2023-10-31 10:08:12 MSK
я положил систему и потерял прошлые ключи,можно их заменить на те , которые прикрепил выше?
Comment 16 Gleb F-Malinovskiy 2023-11-08 12:15:46 MSK
(In reply to ratkin from comment #15)
> я положил систему и потерял прошлые ключи,можно их заменить на те , которые
> прикрепил выше?

Постарайтесь, пожалуйста, хранить ключи более бережно.
Comment 17 Gleb F-Malinovskiy 2023-11-08 12:17:57 MSK
ssh ключ на gitery.alt зарегистрирован.

T/J/S -> 2.3.
Comment 18 ratkin 2023-11-16 09:15:58 MSK
Закончилась квота на gitery , прошу увеличить
Comment 19 Anton Farygin 2023-11-16 10:54:14 MSK
https://bugzilla.altlinux.org/enter_bug.cgi?product=Team%20Accounts компонент quota
Comment 21 Sergey V Turchin 2023-12-12 12:17:49 MSK
(Ответ для ratkin на комментарий #20)
> https://git.altlinux.org/people/krf10/packages/?p=cinelerra-gg.git
У нас есть макросы %autoreconf и %configure .
Comment 22 Sergey V Turchin 2023-12-12 12:20:38 MSK
(Ответ для ratkin на комментарий #20)
> https://git.altlinux.org/people/krf10/packages/?p=python3-module-scrapy.git
Вместо python3-dev надо python3-devel. Это нюанс.
Бинарь неверняка не являются частью пакета модуля.
Comment 23 ratkin 2023-12-13 09:57:36 MSK
(Ответ для Sergey V Turchin на комментарий #22)
> (Ответ для ratkin на комментарий #20)
> > https://git.altlinux.org/people/krf10/packages/?p=python3-module-scrapy.git
> Вместо python3-dev надо python3-devel. Это нюанс.
> Бинарь неверняка не являются частью пакета модуля.

через бинарник запускается кравлер, без него не будет работать
Comment 24 Sergey V Turchin 2023-12-13 11:23:23 MSK
(Ответ для ratkin на комментарий #23)
> через бинарник запускается кравлер, без него не будет работать
Если бинарник может работать, как отдельная утилита и не зависит от питоньего модуля, то его надо в отдельный пакет.
Comment 25 Sergey V Turchin 2024-02-26 11:26:24 MSK
Подопечный готов собирать пакеты.
Comment 26 Gleb F-Malinovskiy 2024-03-27 18:49:41 MSK
ssh ключ на gyle.alt зарегистрирован.
Пакет alt-gpgkeys обновлён.
Адрес подписан на devel@.

T/J/S -> 3.6.
Comment 27 Sergey V Turchin 2024-06-21 10:09:31 MSK
Подопечный готов отправлять пакеты в Сизиф.
Comment 28 Sergey V Turchin 2024-06-21 10:11:15 MSK
Это, что уже было сделано в рамках исправлений. https://packages.altlinux.org/ru/sisyphus/maintainers/krf10/srpms
Comment 30 Sergey V Turchin 2024-09-03 14:59:15 MSK
up
Comment 31 Gleb F-Malinovskiy 2024-09-24 19:27:48 MSK
Призван рецензент (arseny@) для независимой оценки готовности кандидата.

T/J/S -> 4.2.
Comment 32 Arseny Maslennikov 2024-09-27 12:46:22 MSK
(In reply to Gleb F-Malinovskiy from comment #31)
> Призван рецензент (arseny@) для независимой оценки готовности кандидата.
> 
> T/J/S -> 4.2.

ACK.

Позавчера начал читать dre-ace-tao и немного завяз :(, там по лидеру апстримного проекта плачет не рецензент, а кто-то ещё. Поэтому пока не комментирую работу кандидата. Надеюсь в течение недели дочитать всё из comment 29 и тогда уже комментировать.
Comment 33 Arseny Maslennikov 2024-09-30 21:07:39 MSK
(In reply to ratkin from comment #29)
> пакеты собранные для Join:
> https://packages.altlinux.org/ru/tasks/345179/

dre-ace-tao:

1) Спек dre-ace-tao новый для Sisyphus, но в нём нет:
  %define _unpackaged_files_terminate_build 1
Эта директива должна была бы подразумеваться по умолчанию, но мы решили так не делать по просьбе вашего ментора, которому пришлось бы мегасрочно чинить сотни пакетов — иначе их было бы нельзя обновить или пересобрать. В новых же пакетах нет ни одной причины не включать эту директиву. Реально ненужный файл из buildroot можно удалить (или заставить сборочный код его туда не ставить, если тот легко править, как в случае https://mesonbuild.com).
Соответственно, в логе сборки видим:
  [00:15:24] warning: Installed (but unpackaged) file(s) found:
  [00:15:24]     /etc/rc.d/init.d/init.info
  [00:15:24]     /etc/rc.d/init.d/start_d.ins
  [00:15:24]     /etc/rc.d/init.d/stop_d.ins
К ним, кстати, ещё вернёмся далее.

2) Вижу по истории спека, что почищен всякий мёртвый код, но не до конца: в спеке всё ещё остались:
  %%if 0%{?fedora} > 8
  %%endif
  %%if 0%{?suse_version}
  %%endif
Предлагаю развернуть эти блоки (либо оставить содержимое, либо нет)

Часто авторы апстримов не умеют и не хотят писать спеки пакетов, делают это только от большой нужды.
Здесь видно, что авторы апстрима решили натянуть один RPM-спек на вообще любые системы, куда пакет мог бы быть установлен.
Это значительно усложняет чтение спека, ALT-only спек был бы короче.
Ну и далее в том же духе: стоило бы поменять местами наполнение platform_macros.GNU и условные макроподстановки, "вывернуть наизнанку":
  %if %{?_with_inline:1}%{!?_with_inline:0}
  %define inline -D__ACE_INLINE__ -U__ACE_NO_INLINE__
  %else
  %define inline -D__ACE_NO_INLINE__ -U__ACE_INLINE__
  %endif
  
  cat >> $ACE_ROOT/include/makeinclude/platform_macros.GNU <<EOF
  ssl = 1
  
  %if %{?_with_inline:1}%{!?_with_inline:0}
  inline = 1
  %else
  inline = 0
  %endif
  
  %if %{?_with_xt:1}%{!?_with_xt:0}
  xt = 1
  ace_xtreactor = 1
  x11 = 1
  tao_xtresource = 1
  %endif
  
  %if %{?_with_tk:1}%{!?_with_tk:0}
  ace_tkreactor = 1
  tao_tkresource = 1
  tk = 1
  %endif
  
  %if %{?_with_fl:1}%{!?_with_fl:0}
  fl = 1
  tao_flresource = 1
  ace_flreactor = 1
  %endif
  
  %if %{?_with_qt:1}%{!?_with_qt:0}
  qt4 = 1
  gl = 1
  ace_qt4reactor = 1
  tao_qt4resource = 1
  %endif
  EOF
Меньше fork+exec, короче паста.
Чуть ниже, где buildbits = 64, они забыли aarch64 (или, если проект родом из нулевых и ранее, не думали, что этим кончится). (В этом, кстати, прошу разобраться, здесь может быть зарыта серьёзная ошибка упаковки под aarch64) И т. п.

3) Видно, что приходится очень много раз ручками звать команду install(1). У неё есть опция `-v`. Пожалуйста, включайте её; это сильно повысит полезность лога сборки, куда попадают стандартные потоки rpmbuild. Ну, и аналогично с mv -v, cp -v, ln -v, chmod -c, ..., хотя там могут быть и исключения.

Мелкие (мелочные) замечания, они же nitpicks, nits:
— скриптлеты у tao-* однотипны, а иногда просто совпадают; возможно, они напрашиваются на макрофикацию — по крайней мере, пока мы не ввели дистрибутивный механизм резервации UID для сервисов.
Comment 34 Arseny Maslennikov 2024-09-30 21:09:13 MSK
(In reply to ratkin from comment #29)
> пакеты собранные для Join:
> https://packages.altlinux.org/ru/tasks/345179/
В спеке есть вот этот фрагмент:
https://git.altlinux.org/tasks/345179/gears/100/git?p=git;a=blob;f=ace-tao.spec;h=eb1ff505e253943f96e91c211ddbf97b629b1eb4;hb=be051e8f04ce4279a84279528a4313a81532bc00#l1378
Вопросы _к кандидату_:
— зачем там под %buildroot создаются симлинки в никуда? Тем более, что они потом unpackaged, что мы с вами увидели выше. Тем более, что потом их приходится выфильтровывать из цикла.
— обоснуйте, пожалуйста, выбор выражений для sed в строках 1386-1391. Уверены ли вы, что номера строк сохранятся в будущих релизах?

Ещё вопросы к кандидату:
— чего призваны добиться следующие строки:
  85 Requires(post):   /sbin/install-info
  86 Requires(preun):  /sbin/install-info
— Нужна ли вообще поддержка следующего случая? Я не эксперт по ace и tao, интересуюсь. Понятно, что главным экспертом по ace и tao в ALT Linux Team будет наш кандидат. :)
  160 %if 0%{?make_nosrc}
  161 # Leave out the distro for now
  162 NoSource: 0
  163 %endif
— Зачем здесь прямые зависимости на файлы? Неужели autoreq не справляется?
  466 Requires: %_libdir/libX11.so
  467 Requires: %_libdir/libXt.so

В зависимости от ответа станет понятно, есть ли тут грубые ошибки упаковки или нет.
Comment 35 Arseny Maslennikov 2024-09-30 21:15:03 MSK
(In reply to ratkin from comment #29)
> пакеты собранные для Join:
> https://packages.altlinux.org/ru/tasks/344757/

fwbackups:

Не вижу define _unpackaged_files_terminate_build 1.
Соответственно:
  [00:00:06] warning: Installed (but unpackaged) file(s) found:
  [00:00:06]     /usr/share/metainfo/com.diffingo.fwbackups.metainfo.xml

В коммите https://git.altlinux.org/tasks/344757/gears/200/git?p=git;a=commitdiff;h=39f3ef45e254febb327ee7840166b6cd51af1806
пропали runtime-зависимости на:
  /usr/bin/crontab
  tar
  rsync
Нужны ли они, сейчас и в прошлом?

На заметку: gear поддерживает вариант директивы `copy?:`.
`man 5 gear-rules`:
  ... unmatched patterns are ignored instead of causing errors.
Удобно, если патчи стали не нужны, и *.patch не раскрывается ни во что.
Но решать вам.
Ещё на заметку: у нас есть провайды вида python3(x) для чего-либо, доступного по import x; на мой взгляд, логичнее указывать в спеке в BR эту информацию, чем имя пакета, в общем случае. Точно так же: решать вам.

> https://packages.altlinux.org/tasks/344215/
> https://packages.altlinux.org/tasks/344185/
cargo и питоньи модули посмотрю чуть позже, а пока жду ответов на вопросы. :)
Comment 36 Sergey V Turchin 2024-10-01 09:16:01 MSK
(Ответ для Arseny Maslennikov на комментарий #33)
> 1) Спек dre-ace-tao новый для Sisyphus, но в нём нет:
>   %define _unpackaged_files_terminate_build 1
> Эта директива должна была бы подразумеваться по умолчанию, но мы решили так
> не делать по просьбе вашего ментора, которому пришлось бы мегасрочно чинить
> сотни пакетов — иначе их было бы нельзя обновить или пересобрать.
Нет. Я вообще против и сейчас. Для меня это бесполезная опция, ведущая только к дополнительному усложнению сопровождения пакетов.

> В новых же пакетах нет ни одной причины не включать эту директиву.
Нет ни одной причины её включать для меня.
Comment 37 Sergey V Turchin 2024-10-01 09:20:05 MSK
(Ответ для Sergey V Turchin на комментарий #36)
> >   %define _unpackaged_files_terminate_build 1
> Для меня это бесполезная опция, ведущая
> только к дополнительному усложнению сопровождения пакетов.
Если хотите включить -- сделайте голосование среди тех, кто собирает _больше_ пакетов, чем я и я соглашусь с результатами.
Comment 38 Anton Farygin 2024-10-01 09:44:25 MSK
(Ответ для Sergey V Turchin на комментарий #36)
> (Ответ для Arseny Maslennikov на комментарий #33)
> > 1) Спек dre-ace-tao новый для Sisyphus, но в нём нет:
> >   %define _unpackaged_files_terminate_build 1
> > Эта директива должна была бы подразумеваться по умолчанию, но мы решили так
> > не делать по просьбе вашего ментора, которому пришлось бы мегасрочно чинить
> > сотни пакетов — иначе их было бы нельзя обновить или пересобрать.
> Нет. Я вообще против и сейчас. Для меня это бесполезная опция, ведущая
> только к дополнительному усложнению сопровождения пакетов.
> 
> > В новых же пакетах нет ни одной причины не включать эту директиву.
> Нет ни одной причины её включать для меня.

Я поддерживаю.
Мне в моих пакетах очень удобно видеть в логах список файлов, которые я не упаковал.

И это поведение ломается любым удалением неупакованных файлов.

Опция была бы полезна только в том случае, когда для неё будет добавлена возможность игнорирования файлов по маске.
Comment 39 Grigory Ustinov 2024-10-01 10:39:34 MSK
(Ответ для Anton Farygin на комментарий #38)
> (Ответ для Sergey V Turchin на комментарий #36)
> > (Ответ для Arseny Maslennikov на комментарий #33)
> > > 1) Спек dre-ace-tao новый для Sisyphus, но в нём нет:
> > >   %define _unpackaged_files_terminate_build 1
> > > Эта директива должна была бы подразумеваться по умолчанию, но мы решили так
> > > не делать по просьбе вашего ментора, которому пришлось бы мегасрочно чинить
> > > сотни пакетов — иначе их было бы нельзя обновить или пересобрать.
> > Нет. Я вообще против и сейчас. Для меня это бесполезная опция, ведущая
> > только к дополнительному усложнению сопровождения пакетов.
> > 
> > > В новых же пакетах нет ни одной причины не включать эту директиву.
> > Нет ни одной причины её включать для меня.
> 
> Я поддерживаю.
> Мне в моих пакетах очень удобно видеть в логах список файлов, которые я не
> упаковал.
> 
> И это поведение ломается любым удалением неупакованных файлов.
> 
> Опция была бы полезна только в том случае, когда для неё будет добавлена
> возможность игнорирования файлов по маске.

Я буквально вчера лично писал Арсению "гневное письмо" по поводу обязаловки использования этого макроса. Не думал, что моя нелюбовь к этому макросу найдёт отклик в сердцах других людей. Я тоже собираю много пакетов и тоже категорически против использования этого макроса, так как он больше мешает, чем помогает. Неупакованные файлы довольно ярко отображаются в логах и нет ничего плохого в том, чтобы лишний раз хотя бы бегло глянуть лог. Мало ли, может бросится в глаза ещё что-то.

Плюсую против использования этого макроса.
Comment 40 Arseny Maslennikov 2024-10-01 15:30:20 MSK
(In reply to Anton Farygin from comment #38)
> (Ответ для Sergey V Turchin на комментарий #36)
> > (Ответ для Arseny Maslennikov на комментарий #33)
> > > 1) Спек dre-ace-tao новый для Sisyphus, но в нём нет:
> > >   %define _unpackaged_files_terminate_build 1
> > > Эта директива должна была бы подразумеваться по умолчанию, но мы решили так
> > > не делать по просьбе вашего ментора, которому пришлось бы мегасрочно чинить
> > > сотни пакетов — иначе их было бы нельзя обновить или пересобрать.
> > Нет. Я вообще против и сейчас. Для меня это бесполезная опция, ведущая
> > только к дополнительному усложнению сопровождения пакетов.
> > 
> > > В новых же пакетах нет ни одной причины не включать эту директиву.
> > Нет ни одной причины её включать для меня.
> 
> Я поддерживаю.
> Мне в моих пакетах очень удобно видеть в логах список файлов, которые я не
> упаковал.
> 
> И это поведение ломается любым удалением неупакованных файлов.

Это вполне разумное желание. У мейнтейнера есть (и никто не отнимает) право явно решить, что определённые файлы (возможно, по маске/глобу) не нужны, а у всей Team есть право увидеть в логе, что произошло.

Но это решение, на мой взгляд, должно присутствовать в спеке (т. е. попасть в СКВ, а ещё лучше быть там задокументированным) как явное: мол, строчки в спеке "мейнтейнер явно проигнорировал список файловых путей", они появились в коммите (т. е. sign-off деяние), строчки в логе с перечислением проигнорированных файлов.
Проще должно быть не только мейнтейнеру, но и всем остальным.

Явный rm -vf %buildroot/pattern1 %buildroot/pattern2 %buildroot/pattern3 в конце секции %install — пойдёт:
+ в спеке присутствует (явно, командой)
+ в логе присутствует (опция --verbose)
­+ можно по glob
- недостаток: длинно писать (общий префикс %buildroot у всех путей).
* сводка о неупакованных файлах будет не в конце, а после %install, но, с другой стороны, она и сейчас не в конце, а иногда на пару-тройку экранов раньше (перед строчками Wrote: на каждый rpm-артефакт).

Иначе говоря, я бы предложил _u_f_t_b 1 по умолчанию + какое-то устраивающее всех решение для явного игнора файлов, удовлетворяющее предыдущим абзацам. Например, стоило бы обсудить, в какой части лога мейнтейнерам удобно видеть список неупакованных путей (в конце? в секции %install?).

--
Вообще критики правы в том, что этот макрос, как и многие наши меры автоматизированного поддержания порядка в репозитории, вместо помощи мейнтейнеру устроен как кувалда, бьющая его по голове, что плохо. Но лучше что-то, чем совсем ничего.
Comment 41 Arseny Maslennikov 2024-10-01 15:32:50 MSK
(In reply to Arseny Maslennikov from comment #40)
> (In reply to Anton Farygin from comment #38)
> > (Ответ для Sergey V Turchin на комментарий #36)
> > > (Ответ для Arseny Maslennikov на комментарий #33)
> > > > 1) Спек dre-ace-tao новый для Sisyphus, но в нём нет:
> > > >   %define _unpackaged_files_terminate_build 1
> > > > Эта директива должна была бы подразумеваться по умолчанию, но мы решили так
> > > > не делать по просьбе вашего ментора, которому пришлось бы мегасрочно чинить
> > > > сотни пакетов — иначе их было бы нельзя обновить или пересобрать.
> > > Нет. Я вообще против и сейчас. Для меня это бесполезная опция, ведущая
> > > только к дополнительному усложнению сопровождения пакетов.
> > > 
> > > > В новых же пакетах нет ни одной причины не включать эту директиву.
> > > Нет ни одной причины её включать для меня.
> > 
> > Я поддерживаю.
> > Мне в моих пакетах очень удобно видеть в логах список файлов, которые я не
> > упаковал.
> > 
> > И это поведение ломается любым удалением неупакованных файлов.
> 
> Это вполне разумное желание. У мейнтейнера есть (и никто не отнимает) право
> явно решить, что определённые файлы (возможно, по маске/глобу) не нужны, а у
> всей Team есть право увидеть в логе, что произошло.
> 
> Но это решение, на мой взгляд, должно присутствовать в спеке (т. е. попасть
> в СКВ, а ещё лучше быть там задокументированным) как явное: мол, строчки в
> спеке "мейнтейнер явно проигнорировал список файловых путей", они появились
> в коммите (т. е. sign-off деяние), строчки в логе с перечислением
> проигнорированных файлов.
> Проще должно быть не только мейнтейнеру, но и всем остальным.
> 
> Явный rm -vf %buildroot/pattern1 %buildroot/pattern2 %buildroot/pattern3 в
> конце секции %install — пойдёт:
> + в спеке присутствует (явно, командой)
> + в логе присутствует (опция --verbose)
> ­+ можно по glob
> - недостаток: длинно писать (общий префикс %buildroot у всех путей).
> * сводка о неупакованных файлах будет не в конце, а после %install, но, с
> другой стороны, она и сейчас не в конце, а иногда на пару-тройку экранов
> раньше (перед строчками Wrote: на каждый rpm-артефакт).
> 
> Иначе говоря, я бы предложил _u_f_t_b 1 по умолчанию + какое-то устраивающее
> всех решение для явного игнора файлов, удовлетворяющее предыдущим абзацам.
> Например, стоило бы обсудить, в какой части лога мейнтейнерам удобно видеть
> список неупакованных путей (в конце? в секции %install?).
> 
> --
> Вообще критики правы в том, что этот макрос, как и многие наши меры
> автоматизированного поддержания порядка в репозитории, вместо помощи
> мейнтейнеру устроен как кувалда, бьющая его по голове, что плохо. Но лучше
> что-то, чем совсем ничего.

Но это всё касается возобновившейся дискуссии о _u_f_t_b 1 по умолчанию. А у нас здесь joinится кандидат; в контексте его пакетов от отсутствия _u_f_t_b 1 в спеке произошли грубые ошибки упаковки, которые бы этот макрос предотвратил, а файлов не так много, чтобы этот макрос мешал. Если _u_f_t_b 1 мешает итеративной работе над спеком, его можно на ранних стадиях этой работы задать в 0, а на поздних (до сборочницы) задать в 1.
Я категорически против того, чтобы приучать новых участников Team к неряшливости, хоть никто из людей ей и не чужд; она и без помощи ментора/рецензента заводится, незачем помогать этому процессу. (Конечно, явный игнор файловых путей мейнтейнером из предыдущего коммента ничего общего с неряшливостью не имеет, а, напротив, ей противоположен)

Есть ещё _stripped_files_terminate_build, но:
— чтобы он помогал, нужно разбираться в отладочном инфо для бинарников, в причинах, почему оно может не генерироваться, etc.,
— если пакет noarch, там нет arch-specific бинарников, т. е. семантика макроса неважна;
— для очень толстых программ и библиотек генерацию отладочного инфо вообще приходится выключать из-за ограниченных ресурсов;
поэтому на его применении не настаиваю.
Comment 42 Anton Farygin 2024-10-01 15:49:47 MSK
Да, с тем что ментейнер должен уметь пользоваться данным макросом, когда это нужно - согласен.
Comment 43 Sergey V Turchin 2024-10-01 17:07:41 MSK
(Ответ для Arseny Maslennikov на комментарий #41)
> у нас здесь joinится кандидат; в контексте его пакетов
Вот и не надо было выходить за этот контекст. :-)
Comment 44 ratkin 2024-10-01 19:00:22 MSK
(Ответ для Arseny Maslennikov на комментарий #33)
> Ну и далее в том же духе: стоило бы поменять местами наполнение
> platform_macros.GNU и условные макроподстановки, "вывернуть наизнанку":
>   %if %{?_with_inline:1}%{!?_with_inline:0}
>   %define inline -D__ACE_INLINE__ -U__ACE_NO_INLINE__
>   %else
>   %define inline -D__ACE_NO_INLINE__ -U__ACE_INLINE__
>   %endif
>   
>   cat >> $ACE_ROOT/include/makeinclude/platform_macros.GNU <<EOF
>   ssl = 1
>   
>   %if %{?_with_inline:1}%{!?_with_inline:0}
>   inline = 1
>   %else
>   inline = 0
>   %endif
>   
>   %if %{?_with_xt:1}%{!?_with_xt:0}
>   xt = 1
>   ace_xtreactor = 1
>   x11 = 1
>   tao_xtresource = 1
>   %endif
>   
>   %if %{?_with_tk:1}%{!?_with_tk:0}
>   ace_tkreactor = 1
>   tao_tkresource = 1
>   tk = 1
>   %endif
>   
>   %if %{?_with_fl:1}%{!?_with_fl:0}
>   fl = 1
>   tao_flresource = 1
>   ace_flreactor = 1
>   %endif
>   
>   %if %{?_with_qt:1}%{!?_with_qt:0}
>   qt4 = 1
>   gl = 1
>   ace_qt4reactor = 1
>   tao_qt4resource = 1
>   %endif
>   EOF
> Меньше fork+exec, короче паста.
Не очень понял что имеется в виду под вывернуть наизнанку

(Ответ для Arseny Maslennikov на комментарий #34)
> — зачем там под %buildroot создаются симлинки в никуда? Тем более, что они
> потом unpackaged, что мы с вами увидели выше. Тем более, что потом их
> приходится выфильтровывать из цикла.
> — обоснуйте, пожалуйста, выбор выражений для sed в строках 1386-1391.
> 
Эту часть я посмотрел в https://packages.altlinux.org/ru/p5/srpms/ace-tao-ciao/specfiles/

про aarch64 они и правда забыли в спеке
Ответы на вопросы и исправления скоро подготовлю
Comment 45 Arseny Maslennikov 2024-10-01 19:15:37 MSK
(In reply to ratkin from comment #44)
> (Ответ для Arseny Maslennikov на комментарий #33)
> > Ну и далее в том же духе: стоило бы поменять местами наполнение
> > platform_macros.GNU и условные макроподстановки, "вывернуть наизнанку":
> >   %if %{?_with_inline:1}%{!?_with_inline:0}
> >   %define inline -D__ACE_INLINE__ -U__ACE_NO_INLINE__
> >   %else
> >   %define inline -D__ACE_NO_INLINE__ -U__ACE_INLINE__
> >   %endif
> >   
> >   cat >> $ACE_ROOT/include/makeinclude/platform_macros.GNU <<EOF
> >   ssl = 1
> >   
> >   %if %{?_with_inline:1}%{!?_with_inline:0}
> >   inline = 1
> >   %else
> >   inline = 0
> >   %endif
> >   
> >   %if %{?_with_xt:1}%{!?_with_xt:0}
> >   xt = 1
> >   ace_xtreactor = 1
> >   x11 = 1
> >   tao_xtresource = 1
> >   %endif
> >   
> >   %if %{?_with_tk:1}%{!?_with_tk:0}
> >   ace_tkreactor = 1
> >   tao_tkresource = 1
> >   tk = 1
> >   %endif
> >   
> >   %if %{?_with_fl:1}%{!?_with_fl:0}
> >   fl = 1
> >   tao_flresource = 1
> >   ace_flreactor = 1
> >   %endif
> >   
> >   %if %{?_with_qt:1}%{!?_with_qt:0}
> >   qt4 = 1
> >   gl = 1
> >   ace_qt4reactor = 1
> >   tao_qt4resource = 1
> >   %endif
> >   EOF
> > Меньше fork+exec, короче паста.
> Не очень понял что имеется в виду под вывернуть наизнанку
Сейчас в спеке вызовы cat >> f вложены в макроветвления — и тогда эти вызовы должны быть отдельными (в зависимости от значений _with_* они либо присутствуют, либо отсутствуют).
Я имел в виду следующую трансформацию: вложить макроветвления в вызов cat >> f; в частности, тогда достаточно только одного обращения к cat, а содержимое общего heredoc будет зависеть от _with_*.
Меньше вызовов программ, меньше самоповторов в коде, код короче. :)
Comment 46 ratkin 2024-10-07 18:59:11 MSK
(Ответ для Arseny Maslennikov на комментарий #34)
> Вопросы _к кандидату_:
> — зачем там под %buildroot создаются симлинки в никуда? Тем более, что они
> потом unpackaged, что мы с вами увидели выше. Тем более, что потом их
> приходится выфильтровывать из цикла.
Убрал симлинки

> — обоснуйте, пожалуйста, выбор выражений для sed в строках 1386-1391.
> Уверены ли вы, что номера строк сохранятся в будущих релизах?
> 
выражения для sed я подсмотред(написал выше), номера строк не менялись с 2009
> Ещё вопросы к кандидату:
> — чего призваны добиться следующие строки:
>   85 Requires(post):   /sbin/install-info
>   86 Requires(preun):  /sbin/install-info
> — Нужна ли вообще поддержка следующего случая? Я не эксперт по ace и tao,
> интересуюсь. Понятно, что главным экспертом по ace и tao в ALT Linux Team
> будет наш кандидат. :)
>   160 %if 0%{?make_nosrc}
>   161 # Leave out the distro for now
>   162 NoSource: 0
>   163 %endif
Данные строки не нужны, удалил их
> — Зачем здесь прямые зависимости на файлы? Неужели autoreq не справляется?
>   466 Requires: %_libdir/libX11.so
>   467 Requires: %_libdir/libXt.so
там чуть выше было написана причина 
# The xorg package naming scheme changed, use specific files for now.
# old -> Requires: xorg-x11-devel
# new -> Requires: libX11-devel
Заменил на пакеты

Так же дочистил спек и добавил verbose опции.
buildbits=64 для aarch поставить нельзя т.к. это просто добаляет опцию -m64 которой там нет, хотя вот здесь aarch64 учитывается https://git.altlinux.org/tasks/345179/gears/300/git?p=git;a=blob;f=ACE_wrappers/ace/config-linux-common.h;h=d71151c02d2fb4fe67a8a33eba484b55d2e91bca;hb=HEAD#l181
Comment 47 ratkin 2024-10-07 19:03:02 MSK
(Ответ для Arseny Maslennikov на комментарий #35)
> (In reply to ratkin from comment #29)
> > пакеты собранные для Join:
> > https://packages.altlinux.org/ru/tasks/344757/
> 
> fwbackups:
> 
> Не вижу define _unpackaged_files_terminate_build 1.
> Соответственно:
>   [00:00:06] warning: Installed (but unpackaged) file(s) found:
>   [00:00:06]     /usr/share/metainfo/com.diffingo.fwbackups.metainfo.xml

Добавил файл в пакет
> 
> В коммите
> https://git.altlinux.org/tasks/344757/gears/200/git?p=git;a=commitdiff;
> h=39f3ef45e254febb327ee7840166b6cd51af1806
> пропали runtime-зависимости на:
>   /usr/bin/crontab
>   tar
>   rsync
> Нужны ли они, сейчас и в прошлом?
> 
они скорее Recommended так что убрал
> На заметку: gear поддерживает вариант директивы `copy?:`.
> `man 5 gear-rules`:
>   ... unmatched patterns are ignored instead of causing errors.
> Удобно, если патчи стали не нужны, и *.patch не раскрывается ни во что.
Поправил на этот вариант
> Но решать вам.
> Ещё на заметку: у нас есть провайды вида python3(x) для чего-либо,
> доступного по import x; на мой взгляд, логичнее указывать в спеке в BR эту
> информацию, чем имя пакета, в общем случае. Точно так же: решать вам.
Мне удобнее имена пакетов, так что предпочёл бы их оставить
Comment 48 ratkin 2024-10-07 19:04:21 MSK
Для aarch64 пока отключил сборку, попробую разобраться что там
Comment 49 Arseny Maslennikov 2024-10-07 22:33:57 MSK
(In reply to ratkin from comment #46)
> (Ответ для Arseny Maslennikov на комментарий #34)
> > Вопросы _к кандидату_:
> > — зачем там под %buildroot создаются симлинки в никуда? Тем более, что они
> > потом unpackaged, что мы с вами увидели выше. Тем более, что потом их
> > приходится выфильтровывать из цикла.
> Убрал симлинки
OK.
(In reply to ratkin from comment #46)
> (Ответ для Arseny Maslennikov на комментарий #34)
> > — обоснуйте, пожалуйста, выбор выражений для sed в строках 1386-1391.
> > Уверены ли вы, что номера строк сохранятся в будущих релизах?
> > 
> выражения для sed я подсмотред(написал выше)
Ага. Ну вот не надо бездумно повторять за людьми.
> номера строк не менялись с 2009
Это если и успокаивает, то совсем чуть-чуть. От этого код не прекращает быть неоправданно трудным в ручном чтении.
(In reply to ratkin from comment #46)
> (Ответ для Arseny Maslennikov на комментарий #34)
> > Ещё вопросы к кандидату:
> > — чего призваны добиться следующие строки:
> >   85 Requires(post):   /sbin/install-info
> >   86 Requires(preun):  /sbin/install-info
> > — Нужна ли вообще поддержка следующего случая? Я не эксперт по ace и tao,
> > интересуюсь. Понятно, что главным экспертом по ace и tao в ALT Linux Team
> > будет наш кандидат. :)
> >   160 %if 0%{?make_nosrc}
> >   161 # Leave out the distro for now
> >   162 NoSource: 0
> >   163 %endif
> Данные строки не нужны, удалил их
Хорошо.
> > — Зачем здесь прямые зависимости на файлы? Неужели autoreq не справляется?
> >   466 Requires: %_libdir/libX11.so
> >   467 Requires: %_libdir/libXt.so
> там чуть выше было написана причина 
> # The xorg package naming scheme changed, use specific files for now.
> # old -> Requires: xorg-x11-devel
> # new -> Requires: libX11-devel
> Заменил на пакеты
OK, хорошо.
Правильно, что написали комментарий: я поленился тогда убедиться, что эту строчку вы вписали, а не real@ и те, у кого он взял ранние версии спека.
> Так же дочистил спек и добавил verbose опции.
Вижу, поменяли местами вложенность. Любо!

> buildbits=64 для aarch поставить нельзя т.к. это просто добаляет опцию -m64
> которой там нет, хотя вот здесь aarch64 учитывается
> https://git.altlinux.org/tasks/345179/gears/300/git?p=git;a=blob;
> f=ACE_wrappers/ace/config-linux-common.h;
> h=d71151c02d2fb4fe67a8a33eba484b55d2e91bca;hb=HEAD#l181
Очень хорошо. Тогда там стоит оставить коммент, что это `-m64`, а не что-то иное.
Comment 50 Arseny Maslennikov 2024-10-07 23:10:52 MSK
https://git.altlinux.org/tasks/345179/gears/300/git?p=git;a=commitdiff;h=f8570569477a7802b222a0dc1820821fff1d93a9
> @@ -2227,6 +2168,9 @@ fi
>  %endif
>  
>  %changelog
> +* Fri Oct 04 2024 Daniil-Viktor Ratkin <krf10@altlinux.org> 8.0.1-alt2
> +- speak clean up
> +
>  * Tue Aug 06 2024 Daniil-Viktor Ratkin <krf10@altlinux.org> 8.0.1-alt1
>  - new version
>  

Формально это будет минорная претензия, ничем не отличающаяся от других возражений по наполнению rpm changelog, но это предложение меня ввергло в смятение, как когда-то Глеба повреждённое представление GPG-ключа.
Любопытно, это <strong><u>чем</u></strong> же вы умудрились так написать. :) Какой инструмент автора спека способен "spec" превратить... в "speak"?
Это надиктовано голосовому помощнику, что ли, или просто составлено LLM? Если да, это в принципе отбрасывает тень на вклад того, кто так делает; не надо так. Мы, в отличие от других проектов, если я что-нибудь не прозевал, пока не сталкивались с тем, чтобы мейнтейнеры от своего имени вносили в ALT непроверенный/неисправленный LLM-генерат, но вряд ли хотели бы попасть в такую ситуацию (как и в ситуацию, где мейнтейнер приносит зловредный код на сборочницу и в репозиторий). Мы мейнтейнерам доверяем.
The process of joining ALT Linux Team включает не только обучение кандидата в альтотиму собирать пакеты и нажимать правильные кнопки, но и установление взаимного доверия и опыт погружения в командную (team) работу, поэтому я обязан об этом упомянуть.

Короче, просто напишите нормально. :)
Comment 51 ratkin 2024-10-07 23:21:59 MSK
> Какой инструмент автора спека способен "spec" превратить... в "speak"?
это я руками написал, по-ходу в голове разговор был в этот момент)
Comment 52 Sergey V Turchin 2024-10-08 09:43:04 MSK
(Ответ для Arseny Maslennikov на комментарий #50)
> Какой инструмент автора спека способен "spec" превратить... в "speak"?
Тот же, который у меня периодически превращает rpm в rm, legion в leguin и kernel в kdernel. ;-)
Comment 53 ratkin 2024-10-08 23:18:22 MSK
(Ответ для Arseny Maslennikov на комментарий #49)
> > h=d71151c02d2fb4fe67a8a33eba484b55d2e91bca;hb=HEAD#l181
> Очень хорошо. Тогда там стоит оставить коммент, что это `-m64`, а не что-то
> иное.
Оставил коммент, вернул сборку aarch64, настроек из конфигов должно хватать.
Поправил changelog.
Comment 54 Arseny Maslennikov 2024-10-09 15:12:59 MSK
(In reply to Sergey V Turchin from comment #52)
> (Ответ для Arseny Maslennikov на комментарий #50)
> > Какой инструмент автора спека способен "spec" превратить... в "speak"?
> Тот же, который у меня периодически превращает rpm в rm, legion в leguin и
> kernel в kdernel. ;-)
Кстати, телефонные автокорректы при написании спека тоже лучше выключать. :)

(In reply to ratkin from comment #51)
> > Какой инструмент автора спека способен "spec" превратить... в "speak"?
> это я руками написал, по-ходу в голове разговор был в этот момент)
Ну ничего, бывает. Но про инструменты имейте в виду. Не нужно, чтобы инструменты были умнее мейнтейнера. :)
Comment 55 Arseny Maslennikov 2024-10-09 15:13:14 MSK
(In reply to ratkin from comment #29)
> пакеты собранные для Join:
> https://packages.altlinux.org/ru/tasks/344215/
> https://packages.altlinux.org/ru/tasks/344185/

czkawka:

Посмотрел я на чкавку.

Два вопроса: оба из них точно касаются и dre-ace-tao тоже, кстати.

1) У вас в gear-правилах:
     tar.gz: czkawka
И в спеке:
   8 Source0: %name-%version.tar.gz
То есть в дереве под git-коммитом каталог (который, как известно[1], тоже git-дерево), из него программе gear предписано собрать сжатый tarball, а потом RPM в сборочном окружении (вызвав tar, конечно) распакует его на место. А зачем сжимать? Чем это лучше следующего:
     tar: czkawka

[1] https://maryrosecook.com/blog/post/git-from-the-inside-out

2) В списках файлов подпакетов присутствует директива %doc и какие-то файлы там у неё. С какой целью вы её там указали и чего хотели бы добиться?

Эти вопросы чаще всего не имеют целью указать на ошибку; для большинства правил упаковки, не обеспеченных автопроверкой, бывают исключения, и важно, чтобы мейнтейнер/кандидат дал на них обоснованный ответ сам, и:
* либо понимал, что тут исключение, и почему,
* либо признал ошибку,
* либо иным образом обосновал свою позицию (возможно, глядящую в будущее) и был убедителен.
Вот на вопрос про %doc есть ответ третьего типа ;), но мне интересен ответ Даниила-Виктора.

Я же пока пойду читать питон (task 344185)
Comment 56 Sergey V Turchin 2024-10-11 15:44:17 MSK
(Ответ для Arseny Maslennikov на комментарий #54)
> > у меня периодически превращает rpm в rm, legion в leguin и
> > kernel в kdernel. ;-)
> Кстати, телефонные автокорректы при написании спека тоже лучше выключать. :)
Да, они таких слов не знают. ;-)

P.S.
Это одна из 1-х настроек, отключаемых мной сразу же на смартфоне.
Comment 57 Sergey V Turchin 2024-10-11 15:46:49 MSK
(Ответ для Arseny Maslennikov на комментарий #55)
> А зачем сжимать? Чем это лучше следующего:
>      tar: czkawka
Да. rpm сверху сожмёт xz, т.е. будет только хуже.
Ещё я, напрмер, постоянно пользуюсь gear-rpm --short-circuit, а со сжатием оно прилично медленнее.
Comment 58 ratkin 2024-10-11 19:09:16 MSK
(Ответ для Arseny Maslennikov на комментарий #55)

>    8 Source0: %name-%version.tar.gz
> То есть в дереве под git-коммитом каталог (который, как известно[1], тоже
> git-дерево), из него программе gear предписано собрать сжатый tarball, а
> потом RPM в сборочном окружении (вызвав tar, конечно) распакует его на
> место. А зачем сжимать? Чем это лучше следующего:
>      tar: czkawka
как я понял причин сжимать нету, уберу сжатие
> 
> 2) В списках файлов подпакетов присутствует директива %doc и какие-то файлы
> там у неё. С какой целью вы её там указали и чего хотели бы добиться?
> 
В czkawka в каждом пакете разные readme, в которых есть подсказки про использование и описано в чём разница в пакетах, changelog там указан для самой программы и при обновлении версии там можно узнать что поменялось,а вот лицензии в каждом пакете вероятно излишни(то же касается ace-tao).

В ace-tao %doc есть в каждом пакете, это вероятно излишне т.к. файлы там одинаковые, их думаю достаточно было указать в основном пакете libace, а в целом не вижу причин не указать авторов, лицензию, и доп информацию
Comment 59 ratkin 2024-10-21 18:48:15 MSK
@arseny вы там не забыли про join)?
Comment 60 Arseny Maslennikov 2024-10-23 18:40:55 MSK
(In reply to ratkin from comment #59)
> @arseny вы там не забыли про join)?

Пришлось выпасть из контекста, поэтому чуть не забыл. На этой неделе прокомментирую. Если нет — смело пингуйте :)
Comment 61 ratkin 2024-10-29 12:04:11 MSK
(Ответ для Arseny Maslennikov на комментарий #60)
> прокомментирую. Если нет — смело пингуйте :)
@arseny ping

Ещё немного поменял ace-tao, убрал sed, добавил init скрипты для альта(до этого sed просто менял скрипты федоры под альт).
Comment 62 Arseny Maslennikov 2024-10-29 13:35:21 MSK
(In reply to ratkin from comment #58)
> (Ответ для Arseny Maslennikov на комментарий #55)
> 
> >    8 Source0: %name-%version.tar.gz
> > То есть в дереве под git-коммитом каталог (который, как известно[1], тоже
> > git-дерево), из него программе gear предписано собрать сжатый tarball, а
> > потом RPM в сборочном окружении (вызвав tar, конечно) распакует его на
> > место. А зачем сжимать? Чем это лучше следующего:
> >      tar: czkawka
> как я понял причин сжимать нету,
Ну, в общем, да.

> уберу сжатие
OK, увидел. :)

> > 2) В списках файлов подпакетов присутствует директива %doc и какие-то файлы
> > там у неё. С какой целью вы её там указали и чего хотели бы добиться?
> > 
> В czkawka в каждом пакете разные readme, в которых есть подсказки про
> использование и описано в чём разница в пакетах, changelog там указан для
> самой программы и при обновлении версии там можно узнать что поменялось,а
> вот лицензии в каждом пакете вероятно излишни(то же касается ace-tao).
> 
> В ace-tao %doc есть в каждом пакете, это вероятно излишне т.к. файлы там
> одинаковые, их думаю достаточно было указать в основном пакете libace, а в
> целом не вижу причин не указать авторов, лицензию, и доп информацию

Ладно. Я думал, что у вас %doc не сработает в подпакетах, но, если взглянуть на пакеты из задания, то всё там правильно упаковалось согласно спеку.

На самом деле класть лицензии в пакет необязательно, хоть они и лежат во многих пакетах по старой памяти.
Лицензии, строго говоря, у нас лежат в пакете common-licenses; это не только сокращает к-во копий текстов лицензий, но и полезно с точки зрения маркировки пакета лицензиями на его содержимое (для машинного разбора удобнее взять тег License:, чем догадываться, где там в пакете текст лицензии и что он означает). В существующих спеках в альте наблюдается некоторый разнобой с этим, но давно планируется навести порядок.
Если у вас в пакете необычная лицензия, то надо в теге License: своего спека указать либо её SPDX-идентификатор, либо `ALT-$ID` и под этим ID положить в common-licenses текст лицензии; см, например, ALT-Zsh.
Если же типичная лицензия, то достаточно просто её указать в License:.
Comment 63 Arseny Maslennikov 2024-10-29 13:38:57 MSK
(In reply to ratkin from comment #61)
> (Ответ для Arseny Maslennikov на комментарий #60)
> > прокомментирую. Если нет — смело пингуйте :)
> @arseny ping
> 
> Ещё немного поменял ace-tao, убрал sed, добавил init скрипты для альта(до
> этого sed просто менял скрипты федоры под альт).
Здо́рово!
Comment 64 Arseny Maslennikov 2024-10-29 14:04:42 MSK
(In reply to Arseny Maslennikov from comment #55)
> (In reply to ratkin from comment #29)
> > пакеты собранные для Join:
> Я же пока пойду читать питон (task 344185)

К питону вопросов особых нет; разве только в качестве URL пакетов с модулями, наверное, стоит указывать URL проекта на pypi.org.
Comment 65 Arseny Maslennikov 2024-10-29 16:02:15 MSK
(In reply to ratkin from comment #29)
> пакеты собранные для Join:
> https://packages.altlinux.org/ru/tasks/345179/
> https://packages.altlinux.org/ru/tasks/344215/
> https://packages.altlinux.org/ru/tasks/344757/
> https://packages.altlinux.org/ru/tasks/344185/

В общем, тут меня всё более-менее устраивает.

В цели вступления написано:
(In reply to ratkin from comment #0)
> Цель : участие в разработке ПО
Т. е. в ELF надо шарить хотя бы на среднем (intermediate) уровне.

Можно было бы ещё попросить ув. кандидата правильно обновить/упаковать какую-нибудь бинарную разделяемую библиотеку, но я сходу не подберу годный пакет. Можно какую-нибудь уже упакованную и заброшенную — например, по данным repology — взять.

Ну или хотя бы посоветую ознакомиться вот с этой для начала страничкой:
https://www.altlinux.org/Shared_Libs_Policy_Comments
Смысл в том, чтобы:
— отделять пакет с lib$name.so.* от пакета с lib$name.so и остального;
— обеспечить соустанавливаемость двух таких пакетов с библиотеками разных версий ABI в одну систему.
В debian, например, так делают пару десятков лет или более, а в arch не делают до сих пор. Пока мы это не автоматизировали, надо явно за таким следить, иначе в ряде ситуаций наш apt не справляется.
Comment 66 Arseny Maslennikov 2024-10-29 16:06:36 MSK
С моей точки зрения, кандидат научился собирать пакеты разного рода, не допуская грубых ошибок, с точностью до удобной ему манеры работы над спеком, и продемонстрировал, что понимает, что делает. Думаю, можно переходить на следующую стадию T/J/S.
Comment 67 Gleb F-Malinovskiy 2024-11-01 16:45:24 MSK
Пользователь добавлен в группу мейнтейнеров.

Желаю удачного мейнтейнерства!
Comment 68 Sergey V Turchin 2024-11-02 11:01:09 MSK
Уря!