ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: "Денис Назаров" <nenderus@altlinux.org>
To: "Vladimir D. Seleznev" <vseleznv@altlinux.org>
Cc: devel@lists.altlinux.org
Subject: Re: [devel] [#268534] DONE SDL2.git=2.0.14-alt3
Date: Sun, 28 Mar 2021 19:45:48 +0300
Message-ID: <47ec2ab0-3150-132e-ff6f-f243da3d096b@altlinux.org> (raw)
In-Reply-To: <YGCK1NO42ZrMLihh@portlab>


[-- Attachment #1.1: Type: text/plain, Size: 4112 bytes --]

28.03.2021 16:55, Vladimir D. Seleznev пишет:
> On Sun, Mar 28, 2021 at 02:19:52AM +0300, Денис Назаров wrote:
>> 28.03.2021 02:15, Vladimir D. Seleznev пишет:
>>> On Sun, Mar 28, 2021 at 02:07:40AM +0300, Arseny Maslennikov wrote:
>>>> On Sun, Mar 28, 2021 at 01:55:43AM +0300, Денис Назаров wrote:
>>>>> 28.03.2021 01:30, Vladimir D. Seleznev пишет:
>>>> [skip]
>>>>> Казалось, что сборка через cmake более современная.
>>>> Современность — не преступление, и собирать при помощи CMake — само по
>>>> себе тоже не преступление. Но при этом не нужно убирать из пакета ту
>>>> функциональность, которая в нём была. Если сборка при помощи cmake не
>>>> поддерживает правильную комбинацию кнобов или вообще кнобы, а та, что
>>>> была — поддерживает, то вторая лучше. Надо, как обычно, вникнуть и
>>>> разобраться.
>>> Вроде CMake и поддерживает, но тут два момента: вопрос с чем неудобнее
>>> работать, и тот момент, что сборка с CMake'ом по-умолчанию собрала
>>> библиотеку, которые все необходимые ей библиотеки открывает через
>>> dlopen(3).
>> А что в dlopen нехорошего-то?
>>>
>>
> 
> Для библиотек-зависимостей мы используем динамическое связывание. Во
> время сборки пакетов rpm-build прописывает в зависимости пакету все
> требуемые SONAME'ы для каждого упакованного в пакет ELF'а, более того
> генерирует set-versions на основе используемых символов. В результате у
> пакета появляются необходимые для работы зависимости, а у нас и
> пользователей контроль зависимостей на уровне репозитория и целевой
> системы.
> 
> В общем случае стоит использовать динамическое связывание.
> 
> Я не обратил внимание, но похоже, что предыдущие сборки тоже
> использовали dlopen, как и эта. Более того, Debian и Fedora собирают с
> явно включённым dlopen. Сходу непонятно какой смысл так делать, возможно
> это стоит исследовать, но мне кажется правильнее в нашем случае
> использовать динамическую линковку.
> 

Нельзя просто так взять и выключить sdl-dlopen )

Добавление --disable-sdl-dlopen в параметры configure приводит к ошибке 
сборки с поддержкой Vulkan (а отключать я так понимаю совсем не 
целесообразно):

/usr/src/RPM/BUILD/SDL2-2.0.14/src/video/x11/SDL_x11vulkan.c: In 
function 'X11_Vulkan_LoadLibrary':
/usr/src/RPM/BUILD/SDL2-2.0.14/src/video/x11/SDL_x11vulkan.c:47:5: 
error: unknown type name 'VkExtensionProperties'
    47 |     VkExtensionProperties *extensions = NULL;
       |     ^~~~~~~~~~~~~~~~~~~~~
/usr/src/RPM/BUILD/SDL2-2.0.14/src/video/x11/SDL_x11vulkan.c:73:13: 
error: 'VK_NULL_HANDLE' undeclared (first use in this function); did you 
mean 'VK_DEFINE_HANDLE'?
    73 |             VK_NULL_HANDLE, 
"vkEnumerateInstanceExtensionProperties");
       |             ^~~~~~~~~~~~~~
       |             VK_DEFINE_HANDLE

Попробую разобраться



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 203 bytes --]

  parent reply	other threads:[~2021-03-28 16:45 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-03-27 22:02 ` Dmitry V. Levin
2021-03-27 22:30   ` Vladimir D. Seleznev
2021-03-27 22:52     ` Dmitry V. Levin
2021-03-27 22:56       ` Денис Назаров
2021-03-27 23:03         ` Dmitry V. Levin
2021-03-27 22:55     ` Денис Назаров
2021-03-27 23:05       ` Денис Назаров
2021-03-27 23:07       ` Vladimir D. Seleznev
2021-03-27 23:10         ` Vladimir D. Seleznev
2021-03-27 23:34           ` Денис Назаров
2021-03-28  8:59             ` Arseny Maslennikov
2021-03-27 23:07       ` Arseny Maslennikov
2021-03-27 23:11         ` Денис Назаров
2021-03-27 23:15         ` Vladimir D. Seleznev
2021-03-27 23:19           ` Денис Назаров
2021-03-28 13:55             ` Vladimir D. Seleznev
2021-03-28 14:11               ` Yuri Sedunov
2021-03-28 18:15                 ` Денис Назаров
2021-03-28 16:45               ` Денис Назаров [this message]
2021-03-27 22:53   ` Денис Назаров

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=47ec2ab0-3150-132e-ff6f-f243da3d096b@altlinux.org \
    --to=nenderus@altlinux.org \
    --cc=devel@lists.altlinux.org \
    --cc=vseleznv@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