Re: [devel] I: kernel-build-scripts

["Dmitry V. Levin" <ldv@altlinux.org>]

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

On Tue, Feb 26, 2008 at 07:50:59PM +0300, Sergey Vlasov wrote:
> On Mon, Feb 25, 2008 at 10:33:23PM +0300, Dmitry V. Levin wrote:
> > - Используемый для сборки модулей add_changelog должен быть пропатчен.
> 
> Патченый add_changelog необходим только в частном случае - когда
> формируются пакеты для архитектуры, не совпадающей с архитектурой
> хост-системы (в настоящий момент единственный случай, когда возникает
> такая ситуация - сборка пакетов для i586 на хосте x86_64).  Проблема в
> том, что add_changelog при своей работе использует rpm -q --specfile
> для извлечения информации о версии и т.п. из обрабатываемого
> spec-файла, при этом выполняется раскрытие макросов с использованием
> /usr/lib/rpm/*/macros, однако при запуске скриптов под setarch i586
> ищется файл макросов для i586, а не для x86_64.

Интересно, а зачем так сделано?
Я имею в виду запуск add_changelog'а в хост-системе.

> -[ -n "$RPMARG" ] || RPMARG="-q --qf $(Quote "$qf")"
> +RPMARG="$RPMARG -q --qf $(Quote "$qf")"
>  for spec in "$@"; do
>  	eval "$(Quote "$RPM")" $RPMARG --specfile "$(Quote "$spec")" |head -1
>  done
> 
> Т.е., раньше опция -a полностью заменяла все параметры, передаваемые
> rpm; после этого изменения переданные параметры добавляются к тем,
> которые сформировал скрипт stamp_spec.  Допустимо ли подобное
> изменение, или придётся завести дополнительную опцию?  (В текущем
> варианте опция -a фактически бесполезна, поскольку скрипт stamp_spec в
> этом случае фактически сводится к вызову rpm с указанными параметрами
> с добавлением --specfile.)

Наверное, допустимо.
Вообще там очень старый код, сейчас бы я так не написал.

> 2. Добавление --define '_allow_undefined_macros 1' в параметры вызовов
> rpm, выполняемых скриптом add_changelog:
> 
> --- /usr/bin/add_changelog	2007-12-27 12:17:30 +0300
> +++ /home/vsu/bin/add_changelog	2008-01-14 22:37:30 +0300
> @@ -21,9 +21,9 @@
>  
>  PROG="${0##*/}"
>  
> -STAMPER=/usr/bin/stamp_spec
> +STAMPER=/home/vsu/bin/stamp_spec
>  RPM=/bin/rpm
> -RPMARG=
> +RPMARG="--define '_allow_undefined_macros 1'"
>  ENTRY='- '
>  NOCHECK=
>  
> @@ -105,8 +105,8 @@
>  		continue
>  	fi
>  	if [ -z "$NOCHECK" ]; then
> -		oldver=`$RPM -q --qf '%{CHANGELOGNAME}\n' --specfile "$spec" |sed -ne '/^(none)$/q;s/[^<]\+<[^>]\+> *\(.\+\)$/\1/pg'`
> -		newver=`$RPM -q --qf '%|SERIAL?{%{SERIAL}:}|%{VERSION}-%{RELEASE}\n' --specfile "$spec" |head -1`
> +		oldver=`$RPM --define '_allow_undefined_macros 1' -q --qf '%{CHANGELOGNAME}\n' --specfile "$spec" |sed -ne '/^(none)$/q;s/[^<]\+<[^>]\+> *\(.\+\)$/\1/pg'`
> +		newver=`$RPM --define '_allow_undefined_macros 1' -q --qf '%|SERIAL?{%{SERIAL}:}|%{VERSION}-%{RELEASE}\n' --specfile "$spec" |head -1`
>  		if [ "$oldver" = "$newver" ]; then
>  			echo "$PROG: $spec: version \"$oldver\" unchanged, skipping"
>  			exitcode=1
> 
> В принципе можно переделать этот скрипт, чтобы значение $RPMARG не
> только передавалось в stamp_spec, но и использовалось при вызовах rpm
> из самого скрипта add_changelog;

Можно представить себе пример разумного RPMARG, который сломает вычисление
oldver/newver?

> тогда можно будет не менять значение
> по умолчанию, а просто при вызове add_changelog из buildmodules
> передавать опцию -a "--define '_allow_undefined_macros 1'".  Возможно,
> так и стоит сделать, а по умолчанию оставить выдачу ошибок при
> обнаружении неопределённых макросов?

По хорошему надо было бы завести отдельные параметры для всех запусков rpm
вообще и для каждого типа в отдельности.  А сейчас, наверное, проще
добавить _allow_undefined_macros по умолчанию.


-- 
ldv

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