ALT Linux Team development discussions
 help / color / mirror / Atom feed
* [devel] pkgconfiglib.req
@ 2007-08-28 17:11 Alexey M. Tourbin
  2007-08-28 19:55 ` Alexey Tourbin
                   ` (2 more replies)
  0 siblings, 3 replies; 39+ messages in thread
From: Alexey M. Tourbin @ 2007-08-28 17:11 UTC (permalink / raw)
  To: devel


Иногда бывает так, что с новым devel-пакетом перестает что-то собираться
из-за добавившихся библиотек для линковки в *.pc файле.  Здесь есть два
подхода.  Первый подход -- это, если добавленные библиотеки на самом
деле излишни, то мы получаем ценную информацию, чтобы дать кому-то
подзатыльник (точнее, чтобы убрать ненужные библиотеки из Libs в новом
пакете).  С другой стороны, раздача подзатыльников методом поломки
репозитария не кажется мне вполне технологичным развлечением.

Второй подход -- это, если добавленные библиотеки на самом деле не
лишние, добить новых *-devel зависимостей на соответствующие пакеты.
Кроме всего прочего, этот подход относительно легко перепоручить
автоматике.

Поэтому предлагаю замыкать зависимости между *-devel пакетами по
содержимому поля Libs в *.pc файлах.

Алгоритм, который реализован ниже, не слишком точно мимикрирует
перебор путей в ld (в частности, *.a библиотеки не просматриваются).
Тем не мене, мне пока не приходит в голову конфигурация, даже, допустим,
патологическая, при которой может получиться неправильный результат.

ДАННОЕ ИЗМЕНЕНИЕ, и не одно оно, ОКОНЧАТЕЛЬНО ПЕРЕВОДИТ *.pc ФАЙЛЫ
В СТАТУС "ДЛЯ *-devel ПАКЕТОВ".  Уважаемые товарищи maintaner'ы!
Кладите *.pc файл в *-devel подпакет, либо не пакуйте его вообще,
до тех пор, пока он кому-нибудь не понадобится.

В своей хост-системе я обнаружил 2 *.pc файла из не-devel пакетов,
которые имеют некоторые проблемы при попытке их обработки:

$ rpm -qf /usr/lib/pkgconfig/libgdiplus.pc
libgdiplus-1.2.4-alt1
$ rpm -qf /usr/lib/pkgconfig/avahi-qt3.pc 
libavahi-qt3-0.6.20-alt2
$ 

Некоторые подробности на этот счет приведены ниже.


Changelog since `4.0.4-alt77-67-g63ac11f' follows:
commit 0e085c0a8388be44d06afe1a1f4b88aa8895ad2e
Author: Alexey Tourbin <at@altlinux>
Date:   Tue Aug 28 20:10:35 2007 +0400

    pkgconfiglib.req: new pkgconfig.req mode (makes dependencies on Libs)
    
    This will grab libraries from ^Libs: clause and map each library
    to rpm dependency, which is typically lib*-devel package.
    
    $ grep ^Libs: /usr/lib/pkgconfig/directfb.pc
    Libs: -ldirectfb -lpthread -ldl -lz
    $
    
    It works like this:
    
    $ ln -s pkgconfig.req.in scripts/pkgconfiglib.req.in
    $ scripts/pkgconfiglib.req.in -v /usr/lib/pkgconfig/directfb.pc
    pkgconfiglib.req.in: /usr/lib/pkgconfig/directfb.pc: libdirectfb.so -> libdirectfb-devel
    libdirectfb-devel
    pkgconfiglib.req.in: /usr/lib/pkgconfig/directfb.pc: libz.so -> zlib-devel
    zlib-devel
    pkgconfiglib.req.in: /usr/lib/pkgconfig/directfb.pc: libfusion.so -> libdirectfb-devel
    libdirectfb-devel
    pkgconfiglib.req.in: /usr/lib/pkgconfig/directfb.pc: libdirect.so -> libdirectfb-devel
    libdirectfb-devel
    pkgconfiglib.req.in: /usr/lib/pkgconfig/directfb.pc: libpthread.so -> glibc-devel (skip)
    pkgconfiglib.req.in: /usr/lib/pkgconfig/directfb.pc: libdl.so -> glibc-devel (skip)
    $
    
    Some minor problems:
    
    $ scripts/pkgconfiglib.req.in /usr/lib/pkgconfig/*.pc >/dev/null
    pkgconfiglib.req.in: /usr/lib/pkgconfig/avahi-qt3.pc: cannot find libavahi-qt3.so library path (skip)
    pkgconfiglib.req.in: /usr/lib/pkgconfig/libgdiplus.pc: cannot find libexif.so library path (skip)
    pkgconfiglib.req.in: /usr/lib/pkgconfig/valgrind.pc: cannot find libcoregrind.so library path (skip)
    pkgconfiglib.req.in: /usr/lib/pkgconfig/valgrind.pc: cannot find libvex.so library path (skip)
    pkgconfiglib.req.in: /usr/lib/pkgconfig/valgrind.pc: cannot find libgcc.so library path (skip)
    $

Full diff since `4.0.4-alt77-67-g63ac11f' follows:
diff --git a/rpm-4_0.spec b/rpm-4_0.spec
index a7ace2a..c167300 100644
--- a/rpm-4_0.spec
+++ b/rpm-4_0.spec
@@ -474,6 +474,7 @@ fi
 %rpmattr %_rpmlibdir/lib.*
 %rpmattr %_rpmlibdir/pam.*
 %rpmattr %_rpmlibdir/pkgconfig.*
+%rpmattr %_rpmlibdir/pkgconfiglib.*
 %rpmattr %_rpmlibdir/shell.*
 %rpmattr %_rpmlibdir/shebang.*
 %rpmattr %_rpmlibdir/static.*
diff --git a/scripts/Makefile.am b/scripts/Makefile.am
index d93d477..e58108a 100644
--- a/scripts/Makefile.am
+++ b/scripts/Makefile.am
@@ -51,4 +51,6 @@ config_SCRIPTS = \
 
 install-data-local:
 	@LN_S@ pkgconfig.req $(DESTDIR)$(configdir)/pkgconfig.prov
+	@LN_S@ pkgconfig.req $(DESTDIR)$(configdir)/pkgconfiglib.req
 	@LN_S@ pkgconfig.req.files $(DESTDIR)$(configdir)/pkgconfig.prov.files
+	@LN_S@ pkgconfig.req.files $(DESTDIR)$(configdir)/pkgconfiglib.req.files
diff --git a/scripts/pkgconfig.req.in b/scripts/pkgconfig.req.in
index 37391d4..95d0210 100755
--- a/scripts/pkgconfig.req.in
+++ b/scripts/pkgconfig.req.in
@@ -58,8 +58,55 @@ PkgconfigProv()
 	done
 }
 
+PkgconfigLibReq()
+{
+	local f="$1" l L; shift
+	l=$(pkg-config --print-errors --libs-only-l "$f") || Fatal "failed to process $f"
+	L=$(pkg-config --print-errors --libs-only-L "$f") || Fatal "failed to process $f"
+	l=$(echo '' $l |sed -e 's/ -l/ /g')
+	L=$(echo '' $L |sed -e 's/ -L/ /g')
+	local lib libdir
+	for lib in $l; do
+		lib=lib$lib.so
+		if [ -n "${RPM_BUILD_ROOT-}" ]; then
+			for libdir in $L "$RPM_LIBDIR"; do
+				libdir=${libdir%/}
+				[ -f "$RPM_BUILD_ROOT$libdir/$lib" ] || continue
+				# The library is under RPM_BUILD_ROOT.
+				# Nothing is required.  Do next lib.
+				Verbose "$f: $lib -> \$RPM_BUILD_ROOT$libdir/$lib (skip)"
+				continue 2
+			done
+		fi
+		for libdir in $L "$RPM_LIBDIR"; do
+			libdir=${libdir%/}
+			[ -f "$libdir/$lib" ] || continue
+			# The library is found in the host system.
+			# Generate rpm dependency and do next lib.
+			local pkg n
+			pkg=$(rpmquery --whatprovides --queryformat='%{NAME}\n' "$libdir/$lib" |sort -u)
+			n=$(set -- $pkg; echo $#)
+			if [ "$pkg" = glibc-devel ]; then
+				Verbose "$f: $lib -> $pkg (skip)"
+			elif [ $n -eq 1 ]; then
+				Verbose "$f: $lib -> $pkg"
+				printf '%s\n' "$pkg"
+			elif [ $n -gt 1 ]; then
+				Info "$f: $libdir/$lib provided by:$(echo '' $pkg)"
+				Info "$f: $lib -> $libdir/$lib (raw, ambiguous)"
+				printf '%s\n' "$libdir/$lib"
+			else
+				Info "$f: cannot map $lib to rpm dependency (skip)"
+			fi
+			continue 2
+		done
+		Info "$f: cannot find $lib library path (skip)"
+	done
+}
+
 case "${0##*/}" in
 	pkgconfig.req*) ArgvFileAction PkgconfigReq "$@" ;;
 	pkgconfig.prov*) ArgvFileAction PkgconfigProv "$@" ;;
+	pkgconfiglib.req*) ArgvFileAction PkgconfigLibReq "$@" ;;
 	*) Fatal "req/prov method not recognized" ;;
 esac


^ permalink raw reply	[flat|nested] 39+ messages in thread
end of thread, other threads:[~2007-09-19 22:22 UTC | newest]

Thread overview: 39+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-08-28 17:11 [devel] pkgconfiglib.req Alexey M. Tourbin
2007-08-28 19:55 ` Alexey Tourbin
2007-08-28 20:12   ` [devel] /usr/lib/pkgconfig vs noarch Dmitry V. Levin
2007-08-28 20:47     ` Alexey Tourbin
2007-08-28 21:07       ` Alexey Tourbin
2007-08-28 21:32         ` [devel] sisyphus_check noarch Alexey Tourbin
2007-09-01 13:12         ` [devel] /usr/lib/pkgconfig vs noarch Денис Смирнов
2007-08-29 15:56       ` Dmitry V. Levin
2007-08-29 17:05         ` [devel] giter-factory Alexey Tourbin
2007-08-29 19:29           ` Dmitry V. Levin
2007-08-29 19:37             ` Alexey Tourbin
2007-08-29 21:02               ` Alexey Tourbin
2007-08-29 21:11                 ` Dmitry V. Levin
2007-08-29 21:47                   ` Alexey Tourbin
2007-08-29 21:55                     ` Dmitry V. Levin
2007-08-29 22:26                       ` Alexey Tourbin
2007-08-30  8:40                         ` Kirill A. Shutemov
2007-09-01 23:47                       ` Alexey Tourbin
2007-08-30  8:53                 ` Kirill A. Shutemov
2007-09-16 21:13           ` Michael Shigorin
2007-09-16 21:36             ` Alexey Tourbin
2007-09-16 21:32               ` Aleksey Avdeev
2007-09-16 22:15                 ` [devel] giter-factory idea Alexey Tourbin
2007-09-16 23:01                   ` Aleksey Avdeev
2007-09-17  5:42                     ` Alexey Tourbin
2007-09-17 10:41                       ` Aleksey Avdeev
2007-09-18  9:32               ` [devel] giter-factory Michael Shigorin
2007-08-28 20:29   ` [devel] pkgconfiglib.req Alexey I. Froloff
2007-08-28 20:46   ` Alexey Rusakov
2007-08-29 13:51 ` Igor Zubkov
2007-09-16 21:17 ` Michael Shigorin
2007-09-16 22:50   ` Alexey Rusakov
2007-09-17  5:36     ` Alexey Tourbin
2007-09-18 10:09       ` [devel] UQ: git-clone git.alt: unable to chdir or not a git archive Michael Shigorin
2007-09-18 10:09         ` Pavlov Konstantin
2007-09-18 10:15           ` [devel] отбой, спасибо, сам дурак Michael Shigorin
2007-09-18 11:44       ` [devel] pkgconfiglib.req Michael Shigorin
2007-09-18 11:49         ` Alexey Rusakov
2007-09-19 22:22         ` [devel] *.pc -devel Alexey Tourbin

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