[d-kernel] kernel-build-tools macros update (Re: [devel] IA: incomiger rebuild)

[d-kernel] kernel-build-tools macros update (Re: [devel] IA: incomiger rebuild)

[Sergey Vlasov]

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

On Mon, May 08, 2006 at 02:15:10AM +0400, Alexey Gladkov wrote:
> Dmitry V. Levin wrote:
> > А buildtime не учитывается?
> 
> Нет. Если apt говорит что два пакета можно собрать, то робот их
> собирает ... при этом на разных серверах параллельно.

В таком случае в kernel-build-tools нужно будет внести следующие
изменения:

diff --git a/build/tools/SOURCES/kernel-macros b/build/tools/SOURCES/kernel-macros
index 2b57b26..0054c29 100644
--- a/build/tools/SOURCES/kernel-macros
+++ b/build/tools/SOURCES/kernel-macros
@@ -14,9 +14,14 @@ #=======================================
 %_patch_list		%nil
 %set_patch_list()	%global _patch_list %*
 %add_patch_list()	%global _patch_list %_patch_list %*
-%get_patch_list()	%_patch_list
+%get_patch_list()	%( \
+	for p in %_patch_list; do \
+		dep=`rpmquery --dbpath %_dbpath --qf '%%{NAME} >= %%|SERIAL?{%%{SERIAL}:}|%%{VERSION}-%%{RELEASE}' "$p" 2>/dev/null` || dep="$p" \
+		echo -n " $dep " \
+	done \
+)
 
-%format_patch_list      %(rpmquery --queryformat '\\n\\t%%{NAME}-%%{VERSION}-%%{RELEASE}' %get_patch_list)
+%format_patch_list      %(rpmquery --dbpath %_dbpath --qf '\\n\\t%%{NAME}-%%{VERSION}-%%{RELEASE}' %_patch_list)
 
 %_src_list	%nil
 
diff --git a/build/tools/kernel-build-tools.spec b/build/tools/kernel-build-tools.spec
index db0d061..85e1c04 100644
--- a/build/tools/kernel-build-tools.spec
+++ b/build/tools/kernel-build-tools.spec
@@ -3,8 +3,8 @@ # $Id: kernel-build-tools.spec,v 1.6 200
 ##
 
 Name: kernel-build-tools
-Version: 0.8
-Release: alt2
+Version: 0.9
+Release: alt1
 
 Summary: RPM macroses for kernel builds
 License: GPL
@@ -38,6 +38,13 @@ mkdir -p %buildroot%_bindir
 %doc *
 
 %changelog
+* Mon May 08 2006 Sergey Vlasov <vsu@altlinux.ru> 0.9-alt1
+- kernel-macros:
+  + %%get_patch_list: add package versions to the returned list (needed to get
+    correct BuildRequires for kernel packages);
+  + %%get_patch_list, %%format_patch_list: pass current %%_dbpath to rpmquery
+    to get correct results when using build scripts from kernel CVS.
+
 * Sat Dec 03 2005 Sergey Vlasov <vsu@altlinux.ru> 0.8-alt2
 - kernel-macros: fix by Dmitry V. Levin <ldv@altlinux>:
   + %%_src_list: quote %% to avoid unneeded macro expansion

После этого пакеты kernel-image-* будут при сборке получать в src.rpm
зависимость на пакеты с патчами с указанием минимальной версии и
сборки, что предотвратит ошибочную пересборку этих src.rpm с
использованием устаревших патчей.

Есть какие-то замечания по этому поводу?

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

Re: [d-kernel] kernel-build-tools macros update

[Dmitry V. Levin]

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

On Mon, May 08, 2006 at 09:11:35PM +0400, Sergey Vlasov wrote:
[...]
> После этого пакеты kernel-image-* будут при сборке получать в src.rpm
> зависимость на пакеты с патчами с указанием минимальной версии и
> сборки, что предотвратит ошибочную пересборку этих src.rpm с
> использованием устаревших патчей.
> 
> Есть какие-то замечания по этому поводу?

Выглядит нормально, звучит логично.


-- 
ldv

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

Re: [d-kernel] kernel-build-tools macros update

[Konstantin A. Lepikhov]

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

Hi Dmitry!

Wednesday 10, at 01:33:39 PM you wrote:

> On Mon, May 08, 2006 at 09:11:35PM +0400, Sergey Vlasov wrote:
> [...]
> > После этого пакеты kernel-image-* будут при сборке получать в src.rpm
> > зависимость на пакеты с патчами с указанием минимальной версии и
> > сборки, что предотвратит ошибочную пересборку этих src.rpm с
> > использованием устаревших патчей.
> > 
> > Есть какие-то замечания по этому поводу?
> 
> Выглядит нормально, звучит логично.
+1

-- 
WBR et al.

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