Bug 36929

Summary: Генерируются лишние зависимости
Product: Sisyphus Reporter: Владимир Диденко <vladimir.didenko>
Component: rpm-build-girAssignee: Yuri N. Sedunov <aris>
Status: CLOSED FIXED QA Contact: qa-sisyphus
Severity: normal    
Priority: P3 CC: aris, mike, shaba
Version: unstable   
Hardware: all   
OS: Linux   

Description Владимир Диденко 2019-06-20 16:13:54 MSK 
Текущая версия rpm-build-gir генерирует лишние зависимости, если импортируется подмодуль, вместо всего модуля. Например, строчка

var giImportNames = imports.gi.GIRepository.Repository

сгенерирует зависимость как на typelib(GIRepository), так и на typelib(Repository). Насколько я понимаю, такого быть не должно.
Comment 1 Yuri N. Sedunov 2019-06-20 16:55:01 MSK 
Владимир, что за прожект, cinnamon?  Мне увидеть, что Вы уже сделали.
Comment 2 Владимир Диденко 2019-06-20 17:24:36 MSK 
(In reply to comment #1)
> Владимир, что за прожект, cinnamon?  Мне увидеть, что Вы уже сделали.

Да. Я ничего не делал, просто раньше эта зависимость не генерировалась, так как точки с запятой в конце не было, а предыдущая версия rpm-buil-gir, такие случаи не обрабатывала. 

А так это апстримный код.
Comment 3 Yuri N. Sedunov 2019-06-20 17:31:39 MSK 
Пока пропишите %add_typelib_req_skiplist typelib(Repository), а я со временем разберусь.
Comment 4 Владимир Диденко 2019-06-20 17:33:29 MSK 
(In reply to comment #3)
> Пока пропишите %add_typelib_req_skiplist typelib(Repository), а я со временем
> разберусь.

Хорошо, спасибо!
Comment 5 Владимир Диденко 2019-06-27 17:44:07 MSK 
На новом cinnamon еще одна новая паразитная зависимость вылезла. Строчка 

const {get_monotonic_time} = imports.gi.GLib;

привела к появлению зависимости на typelib(get_monotonic_time)
Comment 6 Yuri N. Sedunov 2019-06-28 18:15:55 MSK 
Проверьте, пожалуйста
#233394 AWAITING #1 [test-only] sisyphus rpm-build-gir.git=0.7.3-alt1
Comment 7 Repository Robot 2019-06-29 18:18:42 MSK 
rpm-build-gir-0.7.3-alt1 -> sisyphus:

Fri Jun 28 2019 Yuri N. Sedunov <aris@altlinux> 0.7.3-alt1
- gir-js.req: modified to support more JS expressions (ALT #36929)