[devel] 0ldconfig.filetrigger (packages/rpm: heads/altlinux-4.1)

[devel] 0ldconfig.filetrigger (packages/rpm: heads/altlinux-4.1)

[Alexey M. Tourbin]

[-- Attachment #1: Type: text/plain, Size: 2265 bytes --]

Позволяет ли это удалить разнообразные вызовы ldconfig
из *всех* пакетов?

Update of /people/at/packages/rpm.git

Changes statistics since `4.0.4-alt95.M41.5-4-g70f8851' follows:
 scripts/0ldconfig.filetrigger |   40 +++++++++++++++++++++++++++++++++++++---
 1 files changed, 37 insertions(+), 3 deletions(-)

Changelog since `4.0.4-alt95.M41.5-4-g70f8851' follows:
commit f46b034eba0103c7652378643baaccfe5d841036
Author: Alexey Tourbin <at@altlinux>
Date:   Thu Nov 6 16:26:54 2008 +0300

    0ldconfig.filetrigger: execute "/sbin/update_chrooted lib" for /lib and /lib64

Full diff since `4.0.4-alt95.M41.5-4-g70f8851' follows:
diff --git a/scripts/0ldconfig.filetrigger b/scripts/0ldconfig.filetrigger
index b17f85a..6ec01c0 100755
--- a/scripts/0ldconfig.filetrigger
+++ b/scripts/0ldconfig.filetrigger
@@ -7,11 +7,29 @@
 # the Free Software Foundation; either version 2 of the License, or
 # (at your option) any later version.
 
+# library flag
+lib=
+
+# system library flag
+sys=
+
 while read -r f; do
+	# 0) Short circuit condition: if a library is already found,
+	# skip the remaining list of non-system libraries.  Note that
+	# the input file list is sorted, and "/lic" collates after
+	# "/lib" and "/lib64".
+	if [ -n "$lib" ] && LC_ALL=C [ "$f" '>' "/lic" ]; then
+		#echo >&2 break="$f"
+		break
+	fi
 	# 1) Check if basename looks like a shared library.
 	case "${f##*/}" in
-		lib*.so*) ;;
-		*) continue ;;
+		lib*.so*)
+			#echo >&2 lib="$f"
+			lib=1
+			;;
+		*) continue
+			;;
 	esac
 	# 2) It is also possible to check dirname and skip private
 	# directories (public ones are /lib, /lib64, /usr/lib, and
@@ -25,5 +43,21 @@ while read -r f; do
 	# exist, then $f appears to be a compile-time symbolic link.
 	# However, /etc/ld.so.cache seems to include both sonames and
 	# non-soname lib*.so entries.
-	exec /sbin/ldconfig
+	# 4) Check whether it is a system library.
+	case "${f%/*}" in
+		/lib | /lib/* | /lib64 | /lib64/* )
+			#echo >&2 sys="$f"
+			sys=1
+			break
+			;;
+	esac
 done
+
+if [ -n "$lib" ]; then
+	/sbin/ldconfig
+fi
+
+update_chrooted=/sbin/update_chrooted
+if [ -n "$sys" ] && [ -x "$update_chrooted" ]; then
+	"$update_chrooted" lib
+fi

[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]

Re: [devel] 0ldconfig.filetrigger (packages/rpm: heads/altlinux-4.1)

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 512 bytes --]

On Thu, Nov 06, 2008 at 07:13:31PM +0300, Alexey M. Tourbin wrote:
> --- a/scripts/0ldconfig.filetrigger
> +++ b/scripts/0ldconfig.filetrigger
[...]
> +	# 0) Short circuit condition: if a library is already found,
> +	# skip the remaining list of non-system libraries.  Note that
> +	# the input file list is sorted, and "/lic" collates after
> +	# "/lib" and "/lib64".
> +	if [ -n "$lib" ] && LC_ALL=C [ "$f" '>' "/lic" ]; then

Тогда лучше написать без обиняков: [ "$f" '>' "/lib64" ]


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]

Re: [devel] 0ldconfig.filetrigger (packages/rpm: heads/altlinux-4.1)

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 302 bytes --]

On Thu, Nov 06, 2008 at 07:13:31PM +0300, Alexey M. Tourbin wrote:
> Позволяет ли это удалить разнообразные вызовы ldconfig
> из *всех* пакетов?

Изо всех, кроме пакета с ld.so; в принципе, можно убрать и оттуда,
понадеявшись на гарантии обратной совместимости формата ld.so.cache.


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]

Re: [devel] 0ldconfig.filetrigger (packages/rpm: heads/altlinux-4.1)

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 1779 bytes --]

On Sat, Nov 08, 2008 at 05:29:47PM +0300, Dmitry V. Levin wrote:
> On Thu, Nov 06, 2008 at 07:13:31PM +0300, Alexey M. Tourbin wrote:
> > --- a/scripts/0ldconfig.filetrigger
> > +++ b/scripts/0ldconfig.filetrigger
> [...]
> > +	# 0) Short circuit condition: if a library is already found,
> > +	# skip the remaining list of non-system libraries.  Note that
> > +	# the input file list is sorted, and "/lic" collates after
> > +	# "/lib" and "/lib64".
> > +	if [ -n "$lib" ] && LC_ALL=C [ "$f" '>' "/lic" ]; then
> 
> Тогда лучше написать без обиняков: [ "$f" '>' "/lib64" ]

Не совсем.  Нужно обрубать цикл только тогда, когда все файлы
"/lib64/*" уже пройдены.  А так любой файл "/lib64/*" будет больше
"/lib64", а ведь флаг sys=1 ещё не выставлен.  То есть то, что ты
предлагаешь, -- неверно.  Но идея понятна: условие окончание цикла
можно выразить ещё немного более точно.  Если во всех локалях
"/lib64" > "/lib/" (цифра больше слеша), то в принципе можно сравнивать
с "/lib7".

Тут есть ещё одна тонкость, на которую я обратл внимание: в bash(1)
сказано
       string1 > string2
              True if string1 sorts after string2 lexicographically in the current locale.

Но на самом деле в bash/test.c:binary_test() используется strcmp:
   379    else if ((op[0] == '>' || op[0] == '<') && op[1] == '\0')
   380      return ((op[0] == '>') ? (strcmp (arg1, arg2) > 0) : (strcmp (arg1, arg2) < 0));
то есть, вопреки документации, локаль не используется.

А в ash(1) другая формулировка, которая вроде бы подразумевает strcmp:
            s1 > s2       True if string s1 comes after s2 based on the ASCII value of their characters.

В общем, всё это заставило меня придумать вот такую странную,
но вроде бы наиболее безопасную конструкцию.

[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]

Re: [devel] 0ldconfig.filetrigger (packages/rpm: heads/altlinux-4.1)

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 1073 bytes --]

On Sat, Nov 08, 2008 at 05:32:40PM +0300, Dmitry V. Levin wrote:
> On Thu, Nov 06, 2008 at 07:13:31PM +0300, Alexey M. Tourbin wrote:
> > Позволяет ли это удалить разнообразные вызовы ldconfig
> > из *всех* пакетов?
> 
> Изо всех, кроме пакета с ld.so; в принципе, можно убрать и оттуда,
> понадеявшись на гарантии обратной совместимости формата ld.so.cache.

Если более точно следовать логике glibc/elf/ldconfig.c,
то можно сделать вот так:

--- scripts/0ldconfig.filetrigger-      2008-11-08 14:26:43 +0000
+++ scripts/0ldconfig.filetrigger       2008-11-08 15:06:48 +0000
@@ -24,7 +24,7 @@ while read -r f; do
        fi
        # 1) Check if basename looks like a shared library.
        case "${f##*/}" in
-               lib*.so*)
+               lib*.so* | ld-*.so* )
                        #echo >&2 lib="$f"
                        lib=1
                        ;;

Но стандартный ld-linux-*.so* и сам ldconfig  обновляются одновременно
с libc (в пределах одного пакета glibc-core), так что триггер в этом
случае сработает на libc.so.6.

[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]

Re: [devel] 0ldconfig.filetrigger (packages/rpm: heads/altlinux-4.1)

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 901 bytes --]

On Sat, Nov 08, 2008 at 06:10:35PM +0300, Alexey Tourbin wrote:
> On Sat, Nov 08, 2008 at 05:32:40PM +0300, Dmitry V. Levin wrote:
> > On Thu, Nov 06, 2008 at 07:13:31PM +0300, Alexey M. Tourbin wrote:
> > > Позволяет ли это удалить разнообразные вызовы ldconfig
> > > из *всех* пакетов?
> > 
> > Изо всех, кроме пакета с ld.so; в принципе, можно убрать и оттуда,
> > понадеявшись на гарантии обратной совместимости формата ld.so.cache.

А, я понял.  Если обновился стандартный ld.so, то может быть резон
вызвать ldconfig сразу же (в %post скрипте).  Конечно, posttrans
filetrigger здесь ничего сделать не может (но это отвечает на вопрос,
почему в этом месте вызов ldconfig может потребоваться оставить).

> Но стандартный ld-linux-*.so* и сам ldconfig  обновляются одновременно
> с libc (в пределах одного пакета glibc-core), так что триггер в этом
> случае сработает на libc.so.6.

[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]

Re: [devel] 0ldconfig.filetrigger (packages/rpm: heads/altlinux-4.1)

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 1203 bytes --]

On Sat, Nov 08, 2008 at 05:52:20PM +0300, Alexey Tourbin wrote:
> On Sat, Nov 08, 2008 at 05:29:47PM +0300, Dmitry V. Levin wrote:
> > On Thu, Nov 06, 2008 at 07:13:31PM +0300, Alexey M. Tourbin wrote:
> > > --- a/scripts/0ldconfig.filetrigger
> > > +++ b/scripts/0ldconfig.filetrigger
> > [...]
> > > +	# 0) Short circuit condition: if a library is already found,
> > > +	# skip the remaining list of non-system libraries.  Note that
> > > +	# the input file list is sorted, and "/lic" collates after
> > > +	# "/lib" and "/lib64".
> > > +	if [ -n "$lib" ] && LC_ALL=C [ "$f" '>' "/lic" ]; then
> > 
> > Тогда лучше написать без обиняков: [ "$f" '>' "/lib64" ]
> 
> Не совсем.  Нужно обрубать цикл только тогда, когда все файлы
> "/lib64/*" уже пройдены.  А так любой файл "/lib64/*" будет больше
> "/lib64", а ведь флаг sys=1 ещё не выставлен.  То есть то, что ты
> предлагаешь, -- неверно.  Но идея понятна: условие окончание цикла
> можно выразить ещё немного более точно.  Если во всех локалях
> "/lib64" > "/lib/" (цифра больше слеша), то в принципе можно сравнивать
> с "/lib7".

Поскольку следующий символ после "/" это "0", можно сравнивать с "/lib640". :)


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]

Re: [devel] 0ldconfig.filetrigger (packages/rpm: heads/altlinux-4.1)

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 831 bytes --]

On Sat, Nov 08, 2008 at 06:26:30PM +0300, Alexey Tourbin wrote:
> On Sat, Nov 08, 2008 at 06:10:35PM +0300, Alexey Tourbin wrote:
> > On Sat, Nov 08, 2008 at 05:32:40PM +0300, Dmitry V. Levin wrote:
> > > On Thu, Nov 06, 2008 at 07:13:31PM +0300, Alexey M. Tourbin wrote:
> > > > Позволяет ли это удалить разнообразные вызовы ldconfig
> > > > из *всех* пакетов?
> > > 
> > > Изо всех, кроме пакета с ld.so; в принципе, можно убрать и оттуда,
> > > понадеявшись на гарантии обратной совместимости формата ld.so.cache.
> 
> А, я понял.  Если обновился стандартный ld.so, то может быть резон
> вызвать ldconfig сразу же (в %post скрипте).  Конечно, posttrans
> filetrigger здесь ничего сделать не может (но это отвечает на вопрос,
> почему в этом месте вызов ldconfig может потребоваться оставить).

Да.


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]