From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <ldv@altlinux.org>
Date: Sun, 13 Feb 2022 01:05:00 +0300
From: "Dmitry V. Levin" <ldv@altlinux.org>
To: ALT Devel discussion list <devel@lists.altlinux.org>
Message-ID: <20220212220459.GA18800@altlinux.org>
References: <girar.task.294692.1.1@gyle.mskdc.altlinux.org>
 <girar.task.294692.2.1@gyle.mskdc.altlinux.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=koi8-r
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <girar.task.294692.2.1@gyle.mskdc.altlinux.org>
Subject: Re: [devel] [#294692] EPERM (try 2) openshot.git=2.6.1-alt1.1
X-BeenThere: devel@lists.altlinux.org
X-Mailman-Version: 2.1.12
Precedence: list
Reply-To: ALT Linux Team development discussions <devel@lists.altlinux.org>
List-Id: ALT Linux Team development discussions <devel.lists.altlinux.org>
List-Unsubscribe: <https://lists.altlinux.org/mailman/options/devel>,
 <mailto:devel-request@lists.altlinux.org?subject=unsubscribe>
List-Archive: <http://lists.altlinux.org/pipermail/devel>
List-Post: <mailto:devel@lists.altlinux.org>
List-Help: <mailto:devel-request@lists.altlinux.org?subject=help>
List-Subscribe: <https://lists.altlinux.org/mailman/listinfo/devel>,
 <mailto:devel-request@lists.altlinux.org?subject=subscribe>
X-List-Received-Date: Sat, 12 Feb 2022 22:05:00 -0000
Archived-At: <http://lore.altlinux.org/devel/20220212220459.GA18800@altlinux.org/>
List-Archive: <http://lore.altlinux.org/devel/>
List-Post: <mailto:devel@altlinux.org>

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