[devel] I: kernel-build-scripts
Dmitry V. Levin
=?iso-8859-1?q?ldv_=CE=C1_altlinux=2Eorg?=
Ср Фев 27 01:30:42 MSK 2008
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
----------- следующая часть -----------
Было удалено вложение не в текстовом формате...
Имя : =?iso-8859-1?q?=CF=D4=D3=D5=D4=D3=D4=D7=D5=C5=D4?=
Тип : application/pgp-signature
Размер : 197 байтов
Описание: =?iso-8859-1?q?=CF=D4=D3=D5=D4=D3=D4=D7=D5=C5=D4?=
Url : <http://lists.altlinux.org/pipermail/devel/attachments/20080227/5b6ba262/attachment-0002.bin>
Подробная информация о списке рассылки Devel