ALT Linux Team development discussions
 help / color / mirror / Atom feed
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 тоже
сознательное.


  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