Bug 40863

Summary: newt52: no devel-static any more
Product: Sisyphus Reporter: Alexey Sheplyakov <asheplyakov>
Component: newt52Assignee: Vitaly Lipatov <lav>
Status: CLOSED FIXED QA Contact: qa-sisyphus
Severity: blocker    
Priority: P5 CC: antohami, cas, cow, evg, george, grenka, imz, kotopesutility, lav, nbr, qa_viy, rider, sem, shaba, sin, slev, toni, vitty, viy
Version: unstable   
Hardware: x86_64   
OS: Linux   
Bug Depends on:    
Bug Blocks: 40766    

Description Alexey Sheplyakov 2021-09-06 11:46:22 MSK
В версии 0.52.21-alt1 был удалён -devel-static пакет, который является сборочной зависимостью propagator.
Comment 1 Vitaly Lipatov 2021-09-06 12:13:35 MSK
(Ответ для Alexey Sheplyakov на комментарий #0)
> В версии 0.52.21-alt1 был удалён -devel-static пакет, который является
> сборочной зависимостью propagator.
Да, проглядел. Кто-то хочет правильно починить сборку static?

process-lto: ERROR: ./usr/lib64/libnewt.a: contains __gnu_lto_slim only. Perhaps, you need to %global optflags_lto %optflags_lto -ffat-lto-objects
Comment 2 Alexey Sheplyakov 2021-09-06 13:09:05 MSK
(In reply to Vitaly Lipatov from comment #1)
> (Ответ для Alexey Sheplyakov на комментарий #0)
> > В версии 0.52.21-alt1 был удалён -devel-static пакет, который является
> > сборочной зависимостью propagator.
> Да, проглядел. Кто-то хочет правильно починить сборку static?
> 
> process-lto: ERROR: ./usr/lib64/libnewt.a: contains __gnu_lto_slim only.
> Perhaps, you need to %global optflags_lto %optflags_lto -ffat-lto-objects

#284665 EPERM #3 sisyphus newt52.git=0.52.21-alt2
Comment 3 Vitaly Lipatov 2021-09-06 13:28:05 MSK
(Ответ для Alexey Sheplyakov на комментарий #2)
> (In reply to Vitaly Lipatov from comment #1)
> > (Ответ для Alexey Sheplyakov на комментарий #0)
> > > В версии 0.52.21-alt1 был удалён -devel-static пакет, который является
> > > сборочной зависимостью propagator.
> > Да, проглядел. Кто-то хочет правильно починить сборку static?
> > 
> > process-lto: ERROR: ./usr/lib64/libnewt.a: contains __gnu_lto_slim only.
> > Perhaps, you need to %global optflags_lto %optflags_lto -ffat-lto-objects
> 
> #284665 EPERM #3 sisyphus newt52.git=0.52.21-alt2

Спасибо, я отправил
284670 AWAITING #4 [test-only] sisyphus newt52.git=0.52.21-alt2
Comment 4 Repository Robot 2021-09-06 13:37:53 MSK
newt52-0.52.21-alt2 -> sisyphus:

 Mon Sep 06 2021 Alexey Sheplyakov <asheplyakov@altlinux> 0.52.21-alt2
 - Unconditionally enabled -devel-static, required for propagator (closes: #40863)
Comment 5 Vitaly Lipatov 2021-09-06 13:41:16 MSK
(Ответ для Repository Robot на комментарий #4)
> newt52-0.52.21-alt2 -> sisyphus:
> 
>  Mon Sep 06 2021 Alexey Sheplyakov <asheplyakov@altlinux> 0.52.21-alt2
>  - Unconditionally enabled -devel-static, required for propagator (closes:
> #40863)

И зачем 
+BuildRequires: libslang2-devel-static
+Requires: libslang2-devel-static


И зачем ручку было удалять?
-%if_disabled static
-rm -v %buildroot%_libdir/*.*a
-%endif
Comment 6 Alexey Sheplyakov 2021-09-06 13:53:16 MSK
(In reply to Vitaly Lipatov from comment #5)

> И зачем ручку было удалять?
> -%if_disabled static
> -rm -v %buildroot%_libdir/*.*a
> -%endif

Чтобы не было соблазна за неё ещё раз дёрнуть.
Comment 7 Vitaly Lipatov 2021-09-06 14:04:31 MSK
(Ответ для Alexey Sheplyakov на комментарий #6)
> (In reply to Vitaly Lipatov from comment #5)
> 
> > И зачем ручку было удалять?
> > -%if_disabled static
> > -rm -v %buildroot%_libdir/*.*a
> > -%endif
> 
> Чтобы не было соблазна за неё ещё раз дёрнуть.
Тогда у меня только вопрос, в какую из этих групп ACL вы входите?
gear.alt:sisyphus ACL for newt52	@python @qa
Comment 8 Alexey Sheplyakov 2021-09-06 14:10:15 MSK
(In reply to Vitaly Lipatov from comment #5)

> И зачем 
> +Requires: libslang2-devel-static

Для автоматической установки нужных библиотек:

$ sudo apt-get install libnewt-devel-static
Reading Package Lists... Done
Building Dependency Tree... Done
The following extra packages will be installed:
  libnewt-devel libslang2-devel libslang2-devel-static
The following NEW packages will be installed:
  libnewt-devel libnewt-devel-static libslang2-devel libslang2-devel-static
0 upgraded, 4 newly installed, 0 removed and 0 not upgraded.
Need to get 0B/1814kB of archives.
After unpacking 6402kB of additional disk space will be used.
Do you want to continue? [Y/n]
Comment 9 Олег Соловьев 2021-09-06 14:10:39 MSK
(Ответ для Vitaly Lipatov на комментарий #7)
> Тогда у меня только вопрос, в какую из этих групп ACL вы входите?
> gear.alt:sisyphus ACL for newt52	@python @qa

http://git.altlinux.org/tasks/284665/logs/events.4.3.log:
2021-Sep-06 10:32:08 :: task #284665 for sisyphus resumed by iv:
[...]
girar-check-perms: access to newt52 ALLOWED for iv: member of approved group
check-subtask-perms: #200: newt52: approved by iv
[...]
Comment 10 Alexey Sheplyakov 2021-09-06 14:14:02 MSK
> Тогда у меня только вопрос, в какую из этих групп ACL вы входите?
> gear.alt:sisyphus ACL for newt52	@python @qa


for group in python qa; do ssh gyle.alt acl sisyphus @${group} show; done | grep asheplyakov | wc -l
0
Comment 11 Vitaly Lipatov 2021-09-06 14:18:31 MSK
(Ответ для Alexey Sheplyakov на комментарий #10)
> > Тогда у меня только вопрос, в какую из этих групп ACL вы входите?
> > gear.alt:sisyphus ACL for newt52	@python @qa
> 
> 
> for group in python qa; do ssh gyle.alt acl sisyphus @${group} show; done |
> grep asheplyakov | wc -l
> 0
Спасибо! Не знал, что так можно.
Comment 12 Vitaly Lipatov 2021-09-06 16:26:00 MSK
Вернул обратно «ручку» и добавил комментарий
@@ -1,9 +1,11 @@
 %define _name newt
+# devel-static subpackage needed for propagator (see altbug #40863)
+%def_enable static
 %def_enable tk
Comment 13 Alexey Sheplyakov 2021-09-06 17:05:05 MSK
> И зачем 
> +Requires: libslang2-devel-static

Более подробно. Для динамических библиотек подобные Requires создаются автоматически на основе DT_NEEDED из DSO. Статическая библиотека - просто архив с объектными файлами, никаких метаданных в ней нет. Поэтому Requires надо писать вручную, не полагаясь на роботов.