Bug 34929

Summary:	[FR] %e2k вместо e2k
Product:	Sisyphus	Reporter:	Michael Shigorin <mike>
Component:	php7	Assignee:	Anton Farygin <rider>
Status:	CLOSED FIXED	QA Contact:	qa-sisyphus
Severity:	enhancement
Priority:	P3	CC:	glebfm, grenka, rider
Version:	unstable
Hardware:	e2k
OS:	Linux

Description Michael Shigorin 2018-05-21 15:06:41 MSK

Прошу поправить спек:

-%ifarch e2k
+%ifarch %e2k

...для поддержки e2kv4 и в дальнейшем -- других субархитектур; сизифный rpm этот макрос уже умеет (вот за p8 не уверен, но если вдруг нет -- это баг).

Comment 1 Grigory Ustinov 2018-05-21 15:09:12 MSK

(В ответ на комментарий №0)
> Прошу поправить спек:
> 
> -%ifarch e2k
> +%ifarch %e2k

А почему сразу мне не написать? В acl стоит @everybody же.

Comment 2 Anton Farygin 2018-05-21 15:12:40 MSK

я не хочу править ACL на php, но не надо собирать этот пакет из-за опечатки

Comment 3 Anton Farygin 2018-05-21 15:13:46 MSK

но вообще да, вы правы - не надо кому попало собирать пакет php7

Comment 4 Anton Farygin 2018-05-21 15:16:34 MSK

7.2.5-alt1%ubt-1-gef79280

в пакеты попадёт при выходе следующей версии

Comment 5 Grigory Ustinov 2018-05-21 15:19:04 MSK

(В ответ на комментарий №2)
> я не хочу править ACL на php, но не надо собирать этот пакет из-за опечатки

Это не опечатка, а реальная проблема. Семейство e2k-архитектур объединяется
макросом %e2k, который включает в себя на текущий момент две архитектуры.
Оставляя ifarch e2k мы получаем разную сборку под две похожие архитектуры. Это
совершенно неправильно. Надо исправить. Если ты не хочешь это делать, сделаю я.

А пока проблема есть, пусть лучше бага будет открыта. Закроешь её строчкой (Closes: #34929).

Comment 6 Anton Farygin 2018-05-21 15:25:05 MSK

проблемы уже нет.

Comment 7 Anton Farygin 2018-05-21 15:27:03 MSK

для сборки для e2k берите последний коммит из моего гита. А в репозиторий x86 это попадёт при выходе новой версии.

Я снял ACL с php7, что бы вы не начали делать ошибки при его обновлении.

Comment 8 Michael Shigorin 2018-05-22 10:49:38 MSK

(В ответ на комментарий №1)
> А почему сразу мне не написать? В acl стоит @everybody же.
Потому что у rider@ там довольно хитрая схема для php\d-*.

(В ответ на комментарий №7)
> Я снял ACL с php7, что бы вы не начали делать ошибки при его обновлении.
Резонно.

(В ответ на комментарий №4)
> 7.2.5-alt1%ubt-1-gef79280
> в пакеты попадёт при выходе следующей версии
Спасибо!