[d-kernel] Q: module spec template format change (no explicit @kreleasebuild@)

[d-kernel] Q: module spec template format change (no explicit @kreleasebuild@)

[Sergey Vlasov]

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

Hello!

В процессе переписывания скриптов сборки ядра и модулей в сторону
полного перехода на git возникла идея несколько изменить формат
шаблонов spec-файлов модулей - убрать оттуда ".@kreleasebuild@" и
добавлять эту часть непосредственно в строку "Release: ...".
Изменения, вносимые в скрипты, сейчас можно наблюдать в
http://git.altlinux.org/people/vsu/packages/?p=kernel-build-scripts.git;a=shortlog;h=new-template-format

Суть изменений:

1) Из поля Release в шаблонах убирается ".@kreleasebuild@"; в
   результате при выполнении add_changelog в процессе правки шаблона
   этот текст перестаёт попадать в записи changelog.  Из старых
   записей в changelog этот текст тоже убирается (в spec-файле не
   должно быть ни одного вхождения @kreleasebuild@, чтобы этот файл
   обрабатывался новым способом).

2) При сборке пакета для конкретного ядра изменяется непосредственно
   поле Release в spec-файле:

	subst "s,^release:.*$,&.$KERNEL_CODE.$KERNEL_BUILDRELEASE,1I" "$spec"

   После этого вызывается add_changelog для добавления дополнительной
   записи о сборке модулей для конкретного варианта ядра:

	add_changelog -e "- Build for kernel-image-$KERNEL_FLAVOUR-$KERNEL_VERSION-$KERNEL_RELEASE." "$spec"

В результате этих изменений, во-первых, из changelog убирается не
соответствующая действительности информация о старых релизах пакетов
(поскольку ".@kreleasebuild@" из старых релизов никто не вычищал, все
эти элементы заменялись на текущую версию ядра), во-вторых, в
собранных пакетах последняя запись changelog будет содержать имя того,
кто фактически собирал этот пакет, и реальную дату сборки (а не автора
и дату последнего изменения шаблона, как это происходит сейчас, из-за
чего в sisyphus_check пришлось даже добавлять обход некоторых проверок
для пакетов с модулями).

Какие будут мнения по данному вопросу?

-- 
Sergey Vlasov

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

Re: [d-kernel] Q: module spec template format change (no explicit @kreleasebuild@)

[Michael Shigorin]

On Wed, Nov 07, 2007 at 03:51:49PM +0300, Sergey Vlasov wrote:
> В результате этих изменений, во-первых, из changelog убирается
> не соответствующая действительности информация о старых релизах
> пакетов (поскольку ".@kreleasebuild@" из старых релизов никто
> не вычищал, все эти элементы заменялись на текущую версию
> ядра), во-вторых, в собранных пакетах последняя запись
> changelog будет содержать имя того, кто фактически собирал этот
> пакет, и реальную дату сборки (а не автора и дату последнего
> изменения шаблона, как это происходит сейчас, из-за чего в
> sisyphus_check пришлось даже добавлять обход некоторых проверок
> для пакетов с модулями).

Очень хорошо, а то cybertalk читать сложновато, когда модули
прилетают. :)

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

Re: [d-kernel] Q: module spec template format change (no explicit @kreleasebuild@)

[Sergey Vlasov]

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

On Wed, Nov 07, 2007 at 09:47:01PM +0200, Michael Shigorin wrote:
> Очень хорошо, а то cybertalk читать сложновато, когда модули
> прилетают. :)

Читать всё равно будет сложно - теперь последняя запись в changelog,
генерируемая автоматически, при обновлении пакета будет просто
пропадать, заменяясь на новую аналогичную (плюс добавившиеся записи из
шаблона, если он обновлялся).

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

Re: [d-kernel] Q: module spec template format change (no explicit @kreleasebuild@)

[Konstantin A. Lepikhov]

Hi Sergey!

Wednesday 07, at 03:51:49 PM you wrote:

> Hello!
> 
> В процессе переписывания скриптов сборки ядра и модулей в сторону
> полного перехода на git возникла идея несколько изменить формат
> шаблонов spec-файлов модулей - убрать оттуда ".@kreleasebuild@" и
> добавлять эту часть непосредственно в строку "Release: ...".
> Изменения, вносимые в скрипты, сейчас можно наблюдать в
> http://git.altlinux.org/people/vsu/packages/?p=kernel-build-scripts.git;a=shortlog;h=new-template-format
> 
> Суть изменений:
> 
> 1) Из поля Release в шаблонах убирается ".@kreleasebuild@"; в
>    результате при выполнении add_changelog в процессе правки шаблона
>    этот текст перестаёт попадать в записи changelog.  Из старых
>    записей в changelog этот текст тоже убирается (в spec-файле не
>    должно быть ни одного вхождения @kreleasebuild@, чтобы этот файл
>    обрабатывался новым способом).
> 
> 2) При сборке пакета для конкретного ядра изменяется непосредственно
>    поле Release в spec-файле:
> 
> 	subst "s,^release:.*$,&.$KERNEL_CODE.$KERNEL_BUILDRELEASE,1I" "$spec"
> 
>    После этого вызывается add_changelog для добавления дополнительной
>    записи о сборке модулей для конкретного варианта ядра:
> 
> 	add_changelog -e "- Build for kernel-image-$KERNEL_FLAVOUR-$KERNEL_VERSION-$KERNEL_RELEASE." "$spec"
> 
> В результате этих изменений, во-первых, из changelog убирается не
> соответствующая действительности информация о старых релизах пакетов
> (поскольку ".@kreleasebuild@" из старых релизов никто не вычищал, все
> эти элементы заменялись на текущую версию ядра), во-вторых, в
> собранных пакетах последняя запись changelog будет содержать имя того,
> кто фактически собирал этот пакет, и реальную дату сборки (а не автора
> и дату последнего изменения шаблона, как это происходит сейчас, из-за
> чего в sisyphus_check пришлось даже добавлять обход некоторых проверок
> для пакетов с модулями).
> 
> Какие будут мнения по данному вопросу?
а запатчит существующие шаблоны добрая фея? :)

-- 
WBR et al.

Re: [d-kernel] Q: module spec template format change (no explicit @kreleasebuild@)

[Sergey Vlasov]

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

On Thu, Nov 08, 2007 at 02:25:02AM +0300, Konstantin A. Lepikhov wrote:
> а запатчит существующие шаблоны добрая фея? :)

И зовут эту фею sed -i 's/\.@kreleasebuild@$//g' */*.spec

(как обычно, где-то попадался изврат, с которым это не работает, но
его очень мало).

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

Re: [d-kernel] Q: module spec template format change (no explicit @kreleasebuild@)

[Konstantin A. Lepikhov]

Hi Sergey!

Thursday 08, at 01:49:23 PM you wrote:

> On Thu, Nov 08, 2007 at 02:25:02AM +0300, Konstantin A. Lepikhov wrote:
> > а запатчит существующие шаблоны добрая фея? :)
> 
> И зовут эту фею sed -i 's/\.@kreleasebuild@$//g' */*.spec
> 
> (как обычно, где-то попадался изврат, с которым это не работает, но
> его очень мало).
OK, ну раз все запатчили, значит, пусть так и будет.

-- 
WBR et al.

Re: [d-kernel] Q: module spec template format change (no explicit @kreleasebuild@)

[Sergey Vlasov]

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

On Thu, Nov 08, 2007 at 02:05:54PM +0300, Konstantin A. Lepikhov wrote:
> Hi Sergey!
> 
> Thursday 08, at 01:49:23 PM you wrote:
> 
> > On Thu, Nov 08, 2007 at 02:25:02AM +0300, Konstantin A. Lepikhov wrote:
> > > а запатчит существующие шаблоны добрая фея? :)
> > 
> > И зовут эту фею sed -i 's/\.@kreleasebuild@$//g' */*.spec
> > 
> > (как обычно, где-то попадался изврат, с которым это не работает, но
> > его очень мало).
> OK, ну раз все запатчили, значит, пусть так и будет.

Запатчено; соответствующие скрипты, как и раньше, в
git://git.altlinux.org/people/vsu/packages/kernel-build-scripts.git
(теперь в ветке master).

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