ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: alexei@taf.ru
To: ALT Linux Team development discussions <devel@lists.altlinux.org>
Subject: Re: [devel] Сборка расширений   PostgreSQL
Date: Wed, 9 Feb 2022 21:25:43 +0800 (IRKT)
Message-ID: <531156089.888975.1644413143761.JavaMail.zimbra@taf.ru> (raw)
In-Reply-To: <20220209131453.GA30682@altlinux.org>



----- Исходное сообщение -----
> От: "ldv" <ldv@altlinux.org>
> Кому: "alexei" <alexei@taf.ru>
> Копия: "ALT Linux Team development discussions" <devel@lists.altlinux.org>
> Отправленные: Среда, 9 Февраль 2022 г 21:14:54
> Тема: Re: [devel] Сборка расширений   PostgreSQL

> On Wed, Feb 09, 2022 at 08:51:08PM +0800, alexei@taf.ru wrote:
> [...]
>> Поправил
>> 
>> https://git.altlinux.org/tasks/294683/logs/events.4.1.log
>> 
>> girar-check-perms: access to kde5-kdb DENIED for taf: does not belong to
>> approved builders list: zerg
> 
> Я посмотрел, какие изменения понадобилось сделать в этом пакете:
> https://git.altlinux.org/tasks/294683/gears/1100/git?p=git;a=commitdiff;h=master
> 
> На мой взгляд, следующая часть изменения концепутально неправильная:
> 
> diff --git a/.gear/kdb.spec b/.gear/kdb.spec
> index 7b37cc91..c68e9055 100644
> --- a/.gear/kdb.spec
> +++ b/.gear/kdb.spec
> @@ -2,6 +2,7 @@
> 
> %define sover 4
> %define libkdb3 libkdb3%sover
> +%define pg_ver 14
> 
> Name: kde5-%rname
> Version: 3.2.0
> @@ -21,7 +22,7 @@ Source: %rname-%version.tar
> BuildRequires(pre): rpm-build-kf5
> BuildRequires: extra-cmake-modules qt5-base-devel qt5-wayland-devel
> BuildRequires: libicu-devel
> -BuildRequires: libmysqlclient-devel postgresql-devel sqlite3 libsqlite3-devel
> +BuildRequires: libmysqlclient-devel postgresql%pg_ver-server-devel sqlite3
> libsqlite3-devel
> BuildRequires: kf5-kcoreaddons-devel
> BuildRequires: rpm-build-python3 python3(shlex)
> 
> Пакет kdb не должен зашивать версию postgresql-server-devel в сборочные
> зависимости.  Более того, никакой обычный пакет не должен этого делать,
> вместо этого следует всегда использовать то, что является версией по
> умолчанию в репозитории на момент сборки.
> 
> Например, одна из реализаций может содержать
> "Provides: postgresql-server-devel".

Идея как раз в том, что если кому-то надо именно серверный API, то он должен
четко понимать под какую ветку он собирает приложение, так как это API,
в отличии от libpq обратно не совместимо.

Тут скорее надо смотреть внутрь самого приложения, реально ли ему надо
лезть в серверный API, или может это какой-то атавизм, и вполне достаточно
интерфейса libpq.


  reply	other threads:[~2022-02-09 13:25 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-02-04 17:50 alexei
2022-02-04 17:56 ` Anton V. Boyarshinov
2022-02-04 18:41 ` Dmitry V. Levin
2022-02-09 12:51   ` alexei
2022-02-09 13:14     ` Dmitry V. Levin
2022-02-09 13:25       ` alexei [this message]
2022-02-09 13:29         ` alexei
2022-02-09 13:34         ` Dmitry V. Levin
2022-02-09 13:43         ` Sergey V Turchin
2022-02-09 14:04           ` alexei
2022-02-09 14:15             ` Sergey V Turchin
2022-02-09 14:34               ` alexei
2022-02-09 14:45                 ` Sergey V Turchin
2022-02-09 15:06                   ` alexei
2022-02-10  6:51                     ` Sergey V Turchin
2022-02-09 14:48                 ` Sergey V Turchin
2022-02-09 15:03                   ` alexei
2022-02-10  6:53                     ` Sergey V Turchin
2022-02-10  9:41                       ` alexei
2022-02-10  9:58                         ` Sergey V Turchin
2022-02-10 10:15                           ` alexei
2022-02-10 10:52                             ` Sergey V Turchin
2022-02-10 11:10                               ` alexei
2022-02-10 11:22                                 ` Sergey V Turchin
2022-02-10 11:44                                   ` alexei
2022-02-10 12:16                                     ` Sergey V Turchin
2022-02-09 16:03                 ` Dmitry V. Levin
2022-02-09 13:30       ` Sergey V Turchin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=531156089.888975.1644413143761.JavaMail.zimbra@taf.ru \
    --to=alexei@taf.ru \
    --cc=devel@lists.altlinux.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

ALT Linux Team development discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://lore.altlinux.org/devel/0 devel/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 devel devel/ http://lore.altlinux.org/devel \
		devel@altlinux.org devel@altlinux.ru devel@lists.altlinux.org devel@lists.altlinux.ru devel@linux.iplabs.ru mandrake-russian@linuxteam.iplabs.ru sisyphus@linuxteam.iplabs.ru
	public-inbox-index devel

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://lore.altlinux.org/org.altlinux.lists.devel


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git