From: Alexey Tourbin <at@altlinux.ru>
To: ALT Devel discussion list <devel@lists.altlinux.org>
Subject: Re: [devel] cpp.req
Date: Wed, 9 Mar 2011 01:57:18 +0300
Message-ID: <20110308225718.GB30094@altlinux.org> (raw)
In-Reply-To: <20110308164742.GB22446@altlinux.org>
On Tue, Mar 08, 2011 at 07:47:43PM +0300, Dmitry V. Levin wrote:
> On Mon, Feb 28, 2011 at 03:30:59PM +0300, Dmitry V. Levin wrote:
> > Нет, отчего же. К сожалению, если у библиотеки CFLAGS формируются с
> > помощью %_bindir/*-config, то они в cpp.req не подхватятся. Правда, таких
> > библиотек осталось немного.
>
> Вот реальный пример: правильный gpgme.pc отсутствует, и cpp.req не справляется:
> http://git.altlinux.org/tasks/39999/build/100/i586/log
>
> /usr/src/tmp/gpgme-buildroot/usr/include/gpgme.h:89:2: error: #error GPGME was compiled with _FILE_OFFSET_BITS = 64, please see the section "Largefile support (LFS)" in the GPGME manual.
> cpp.req: /usr/src/tmp/gpgme-buildroot/usr/include/gpgme.h: cpp failed, trying c++ mode
> /usr/src/tmp/gpgme-buildroot/usr/include/gpgme.h:89:2: error: #error GPGME was compiled with _FILE_OFFSET_BITS = 64, please see the section "Largefile support (LFS)" in the GPGME manual.
> cpp.req: WARNING: /usr/src/tmp/gpgme-buildroot/usr/include/gpgme.h: cpp failed
>
> Впрочем, gpgme-config --cflags тоже не помог бы.
>
> P.S. Зависимости у libgpgme-devel-1.3.0-alt5.i586.rpm и все равно
> получились одинаковые. Повезло.
Зависимости получились одинаковые, потому что в пакете libgpgme-devel
вручную уже добавлена зависимость на libgpg-error-devel. Если бы cpp.req
отрабтал правильно, то зависимость можно было бы не добавлять/ удалить уже
добавленную зависимость.
Есть пакеты, в которых неправильно указаны такого рода *-devel
зависимости. Например, в пакете libqt4-devel есть зависимость на xorg-devel,
тогда как cpp.req генерирует "нормальную" зависимость только на libGL-devel,
да и то это требуется только для поддержки OpenGL. То есть зависимость на
xorg-devel ничем не обоснована, кроме фантазии мейнтейнера.
Я исправил некоторые пакеты с такими зависимостями, но не все. Во-первых,
не все мейнтейнеры довольны, когда я пересобираю их пакеты, и не хотелось
бы создавать лишних конфликтов по такой простой причине. Во-вторых,
воротилы от бизнеса форсируют создание нового "стабильного" бранча для
Национальной ОС. Если убрать в libqt4-devel зависимость на xorg-devel,
то сборка каких-то пакетов может сломаться из-за недостаточных зависимостей,
что вызовет неудовольствие воротил от бизнеса, и они не подпустят к распилу
государственных средств. Что конечно тоже нежелательно, тем более по
такой простой причине. Получается, лучше всего самоустраниться из этого
процесса.-(
Случай с gpgme.h показывает, что не только cpp.req не справляется
(например, по причине отсуствия нужных флагов -I/dir, как в некоторых
других случаях), но и сам этот хедер не поддается включению в дефолтной
конфигурации. Можно было бы попытаться каким-нибудь хитрым способом
заменить #error на #warning. С другой стороны, существуют "частные"
конфугации, не предназначенные для включения (например, файлы с winsock2.h
исправлять не нужно, т.к. они предназначены для ненациональной ОС).
Так что я думаю, что логику cpp.req нужно оставить прежней - анализировать
завсимости только тех хедеров, которые удается "полностью" включить.
С *-config файлами есть принципиальная проблема, которая препятствует их
использованию в cpp.req: они предполагают, что всё ихнее добро уже
установлено в хост-систему. Я помню (ещё три года назад это было),
что в пакете wxGTK выполнялись какие-то файлтесты для libs, типа
[ -f ${libdir}/... ]. Всё это не слишком тривиально адаптируется
к RPM_BUILD_ROOT. Так что нежелание использовать *-config тоже
сознательное.
next prev parent reply other threads:[~2011-03-08 22:57 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-02-27 14:38 [devel] pkg-config Requires.private - полчища саранчи и враги народа Alexey Tourbin
2011-02-27 16:45 ` Damir Shayhutdinov
2011-02-27 17:01 ` Dmitry V. Levin
2011-02-28 9:35 ` Damir Shayhutdinov
2011-02-28 12:30 ` Dmitry V. Levin
2011-03-08 16:47 ` [devel] cpp.req Dmitry V. Levin
2011-03-08 22:57 ` Alexey Tourbin [this message]
2011-03-09 11:42 ` Sergey V Turchin
2011-03-09 12:46 ` Dmitry V. Levin
2011-03-09 13:13 ` Sergey V Turchin
2011-03-09 13:16 ` Dmitry V. Levin
2011-03-10 8:47 ` Michael Shigorin
2011-03-10 13:10 ` Dmitry V. Levin
2011-02-27 17:17 ` [devel] pkg-config Requires.private - полчища саранчи и враги народа Alexey Tourbin
2011-03-02 23:29 ` Alexey Tourbin
2011-03-02 23:49 ` Dmitry V. Levin
2011-03-02 5:50 ` Yuri N. Sedunov
2011-03-02 9:10 ` Dmitry V. Levin
2011-03-02 14:02 ` Alexey Tourbin
2011-03-02 13:43 ` Alexey Tourbin
2011-03-02 13:50 ` Yuri N. Sedunov
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=20110308225718.GB30094@altlinux.org \
--to=at@altlinux.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