Re: [devel] [#294692] EPERM (try 2) openshot.git=2.6.1-alt1.1

Re: [devel] [#294692] EPERM (try 2) openshot.git=2.6.1-alt1.1

[Dmitry V. Levin]

Hi,

On Mon, Feb 07, 2022 at 08:44:49AM +0000, Girar awaiter (zerg) wrote:
> https://git.altlinux.org/tasks/294692/logs/events.2.1.log
> 
> subtask  name      aarch64  armh  i586  ppc64le  x86_64
>    #100  openshot     2:02     -     -     2:25    1:18
> 
> 2022-Feb-07 08:38:13 :: task #294692 for sisyphus resumed by zerg:
> #100 build 2.6.1-alt1.1 from /people/zerg/packages/openshot.git fetched at 2022-Feb-04 08:27:47

Меня попросили заапрувить это изменение, аргументировав это тем,
что мантейнер недоступен, а изменение важное.

[...]
> 2022-Feb-07 08:41:16 :: plan: src +1 -1 =17506, aarch64 +1 -1 =29275, ppc64le +1 -1 =28981, x86_64 +1 -1 =30245
> #100 openshot 2.6.1-alt1 -> 2.6.1-alt1.1
>  Fri Feb 04 2022 Sergey V Turchin <zerg@altlinux> 2.6.1-alt1.1
>  - using qtwebkit instead of qtwebengine on e2k and ppc64le

Грамматически неправильная фраза, но не это главное, а

--- a/openshot.spec
+++ b/openshot.spec
...
+#%ifarch %e2k ppc64le
+%def_disable qtwebengine
+#%else
+#%def_enable qtwebengine
+#%endif
...
-# drop obsolete QtWebkit deps
-%add_python3_req_skip QtWebKitWidgets
+%if_enabled qtwebengine
+%add_python3_req_skip PyQt5.QtWebKit PyQt5.QtWebKitWidgets
+%else
+%add_python3_req_skip PyQt5.QtWebEngineCore PyQt5.QtWebEngineWidgets
+%endif

$ compare_packages -a -R -- \
  Sisyphus/files/x86_64/RPMS/openshot-2.6.1-alt1.x86_64.rpm -- \
  tasks/294692/build/100/x86_64/rpms/openshot-2.6.1-alt1.1.x86_64.rpm 
-python3(PyQt5.QtWebEngineWidgets) < 0

Получается, что изменение в пакете не соответствует записи в %changelog'е,
поскольку не специфично для %e2k ppc64le.

Я не мантейнер пакета, я не знаю, правильно ли безусловно заменять
%add_python3_req_skip QtWebKitWidgets
на
%add_python3_req_skip PyQt5.QtWebEngineCore PyQt5.QtWebEngineWidgets
для всех архитектур.

В любом случае описание изменения не должно противоречить самому изменению.

> girar-check-perms: access to openshot DENIED for zerg: does not belong to approved builders list: aris 
> check-subtask-perms: #100: openshot: Operation not permitted

Я такое заапрувить не могу.


-- 
ldv

Re: [devel] [#294692] EPERM (try 2) openshot.git=2.6.1-alt1.1

[Sergey V Turchin]

On Sunday, 13 February 2022 01:05:00 MSK Dmitry V wrote:

[...]
> $ compare_packages -a -R -- \
>   Sisyphus/files/x86_64/RPMS/openshot-2.6.1-alt1.x86_64.rpm -- \
>   tasks/294692/build/100/x86_64/rpms/openshot-2.6.1-alt1.1.x86_64.rpm
> -python3(PyQt5.QtWebEngineWidgets) < 0
> Получается, что изменение в пакете не соответствует записи в %changelog'е,
> поскольку не специфично для %e2k ppc64le.
Коментарии забыл убрать, пока проверял. У меня ж нет ppc64le.
Спасибо! Сейчас сделаю.

[...]

-- 
Regards, Sergey.

Re: [devel] [#294692] EPERM (try 2) openshot.git=2.6.1-alt1.1

[Sergey V Turchin]

On Monday, 14 February 2022 12:02:16 MSK Sergey V wrote:
> On Sunday, 13 February 2022 01:05:00 MSK Dmitry V wrote:
> 
> [...]
> 
> > $ compare_packages -a -R -- \
> > 
> >   Sisyphus/files/x86_64/RPMS/openshot-2.6.1-alt1.x86_64.rpm -- \
> >   tasks/294692/build/100/x86_64/rpms/openshot-2.6.1-alt1.1.x86_64.rpm
> > 
> > -python3(PyQt5.QtWebEngineWidgets) < 0
> > Получается, что изменение в пакете не соответствует записи в %changelog'е,
> > поскольку не специфично для %e2k ppc64le.
> 
> Коментарии забыл убрать, пока проверял. У меня ж нет ppc64le.
> Спасибо! Сейчас сделаю.
Сделал.
 
> [...]


-- 
Regards, Sergey.

Re: [devel] [#294692] EPERM (try 2) openshot.git=2.6.1-alt1.1

[Sergey V Turchin]

On Sunday, 13 February 2022 01:05:00 MSK Dmitry V wrote:

[...]
> Я не мантейнер пакета, я не знаю, правильно ли безусловно заменять
> %add_python3_req_skip QtWebKitWidgets
Эта строка вообще не работала.
https://bugzilla.altlinux.org/show_bug.cgi?id=40362#c3

[...]

-- 
Regards, Sergey.

Re: [devel] [#294692] EPERM (try 2) openshot.git=2.6.1-alt1.1

[Sergey V Turchin]

On Sunday, 13 February 2022 01:05:00 MSK Dmitry V wrote:
> Hi,
> 
> On Mon, Feb 07, 2022 at 08:44:49AM +0000, Girar awaiter (zerg) wrote:
> > https://git.altlinux.org/tasks/294692/logs/events.2.1.log
> > subtask  name      aarch64  armh  i586  ppc64le  x86_64
> >    #100  openshot     2:02     -     -     2:25    1:18
> > 2022-Feb-07 08:38:13 :: task #294692 for sisyphus resumed by zerg:
> > #100 build 2.6.1-alt1.1 from /people/zerg/packages/openshot.git fetched at
> > 2022-Feb-04 08:27:47
> Меня попросили заапрувить это изменение, аргументировав это тем,
> что мантейнер недоступен, а изменение важное.
Да. Прохождению сборки #293807 мешает только openshot, мешая обновлению qt5-
webengine вообще и тормозит обновление части моих пакетов.

[...]

-- 
Regards, Sergey.