Bug 42467

Summary:	rpm-macros-qt5 misrequires qt5-base-common
Product:	Sisyphus	Reporter:	viy <viy>
Component:	rpm-macros-qt5	Assignee:	Sergey V Turchin <zerg>
Status:	CLOSED FIXED	QA Contact:	qa-sisyphus
Severity:	normal
Priority:	P5	CC:	zerg
Version:	unstable
Hardware:	x86_64
OS:	Linux
URL:	https://www.altlinux.org/RPM_Macros_Packaging_Policy

Description

viy 2022-04-15 20:56:39 MSK

В rpm-macros-qt5 влезла зависимость на qt5-base-common.
Её надо либо переместить к -devel, как на патче ниже,
либо удалить, если она случайно возникла.

Вылезло на новой сборочной ноде.

Пакет с макросами не должен иметь зависимостей, см.
https://www.altlinux.org/RPM_Macros_Packaging_Policy

diff --git a/qtbase.spec b/qtbase.spec                                          
index c262aa99e5..15a18eed04 100644                                             
--- a/qtbase.spec                                                               
+++ b/qtbase.spec                                                               
@@ -136,6 +136,7 @@ Summary: Development files for %name                        
 Requires: %name-common = %EVR                                                  
 Requires: pkgconfig(gl) pkgconfig(egl)                                         
 Requires: rpm-macros-%gname = %EVR                                             
+Requires: %name-common = %version-%release                                     
 Requires: gcc-c++                                                              
 %description devel                                                             
 %summary.                                                                      
@@ -152,7 +153,6 @@ Requires: %name-devel                                       
 Summary: Set of RPM macros for packaging Qt%major-based applications           
 Group: Development/KDE and QT                                                  
 BuildArch: noarch                                                              
-Requires: %name-common = %version-%release                                     
 %description -n rpm-macros-%gname                                              
 Set of RPM macros for packaging Qt%major-based applications for %distribution  
 Install this package if you want to create RPM packages that use Qt%major

Comment 1

Sergey V Turchin 2022-04-18 09:32:54 MSK

(Ответ для viy на комментарий #0)
> В rpm-macros-qt5 влезла зависимость на qt5-base-common.
> Её надо либо переместить к -devel, как на патче ниже,
Он и так есть в -devel. Прямо в вашем патче видно.
Возможно, rpmbuild её оптимизирует.

> либо удалить, если она случайно возникла.
Нет. Одна из функций подпакета -common -- удаление всего.

> Вылезло на новой сборочной ноде.
Что именно вылезло? В чём проблема?

Я у себя вижу другую ошибку, связанную с этим.
Надо:
-Requires: %name-common = %version-%release
+Requires: %name-common = %EVR

Comment 2

viy 2022-04-18 09:49:00 MSK

(Ответ для Sergey V Turchin на комментарий #1)
> Что именно вылезло? В чём проблема?

Сборчная нода. Виртуальная машина, основная функция - сборка, т.е.
запуск программы hsh.
так же на ней логовед, который чинит пакеты и делает rpmbuild -bs --nodeps.
чтобы rpmbuild -bs --nodeps завершалась успешно, нужно установить
пакеты rpm-macros-*

к сожалению, как оказалось при попытке установки, некоторые пакеты
rpm-macros-* тянут за собой пакеты, в сборочной ноде явно не нужные,
нарушая тем
https://www.altlinux.org/RPM_Macros_Packaging_Policy
К таким пакетам относятся и rpm-macros-qt{5,6}. Они тянут за собой common.

> Одна из функций подпакета -common -- удаление всего.

Стоит добавить исключение: удаление всего "кроме rpm-macros-qt{5,6}"
пакет -common слишком толстый и тяжелый по сравнению с rpm-macros-qt{5,6}.

Comment 3

Sergey V Turchin 2022-04-18 10:00:58 MSK

(Ответ для viy на комментарий #2)
> https://www.altlinux.org/RPM_Macros_Packaging_Policy
> К таким пакетам относятся и rpm-macros-qt{5,6}. Они тянут за собой common.
В принципе, я не против, уберу.

Comment 4

viy 2022-04-18 15:56:47 MSK

(Ответ для Sergey V Turchin на комментарий #3)
> В принципе, я не против, уберу.

Благодарю! Я понимаю, что для вас это мелочь, лишние хлопоты,
но для меня со 100 rpm-macros-* пакетов оптимизация таки дает профит :)

Comment 5

viy 2022-04-19 13:33:31 MSK

fixed in alt12.