Re: [devel] [#91829] FAILED kernel-image-3.0.git=3.0.68-alt1

[Led <led@altlinux.ru>]

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

On Tuesday 05 March 2013 02:11:25 Dmitry V. Levin wrote:
> On Tue, Mar 05, 2013 at 12:16:56AM +0200, Led wrote:
> > On Monday 04 March 2013 23:59:58 Dmitry V. Levin wrote:
> > > On Mon, Mar 04, 2013 at 11:28:51PM +0200, Led wrote:
> > > > On Monday 04 March 2013 23:21:26 Girar Builder robot wrote:
> > > > > http://git.altlinux.org/tasks/91829/logs/events.1.1.log
> > > > > /.out/firmware-kernel-led-ws-3.0.68-alt1.x86_64.rpm: forbidden
> > > > > dependencies: kernel-image-led-ws sisyphus_check: check-deps ERROR:
> > > > > package dependencies violation
> > > > > hsh-rebuild: pkg.tar: sisyphus_check failed.
> > > >
> > > > В чём смысл этого "violation"?
> > >
> > > Смысл в том, что userspace-пакеты не должны иметь зависимость на
> > > kernelspace-пакеты.  Другое дело, что firmware-kernel-led-ws - это,
> > > очевидно, практически такой же kernelspace-пакет, как и
> > > kernel-modules-*-led-ws,
> >
> > Естественно. Никому другому, кроме как соответствующему kernel-image-* он
> > не нужен.
> >
> > > но sisyphus_check этому не обучен.
> >
> > Может можно как-то обучить?
>
> Да, например, можно пропатчить sisyphus_check.d/100-check-deps :)

Ну, например так как (во вложении)?

>
> > А заодно обучить, чтобы не считал ошибкой зависимость
> > kernel-modules-foo-* от kernel-modules-bar-*, собранных из одного
> > src.rpm.

А с этим что делать? Здесь тоже весёлая ситуация:

kernel-modules-foo-%flavour
Requires: kernel-image-%flavour = %version-%release
Requires: kernel-modules-bar-%flavour = %version-%release

Шибко умный rpm говорит, что kernel-modules-bar-%flavour и так уже требует 
 "kernel-image-%flavour = %version-%release", и выкидывает зависимость "kernel-image-%flavour = %version-%release"

Потом приходит sisyphus_check (тоже умный) и говорит: "а хде у тебя в kernel-modules-foo-%flavour зависимость 
на  "kernel-image-%flavour = %version-%release"? нету? тогда иди нафиг!"

Кого здесь патчить?
 
-- 
Led

[-- Attachment #2: 100-check-deps.diff --]
[-- Type: text/x-diff, Size: 356 bytes --]

--- 100-check-deps.orig	2013-01-24 02:43:53.000000000 +0200
+++ 100-check-deps	2013-03-05 04:22:24.502917098 +0200
@@ -62,7 +62,7 @@
 		rc=1
 	fi
 
-	[ -n "${rpm_name##kernel-*}" ] ||
+	[ -z "${rpm_name##kernel-*}" -o -z "${rpm_name##firmware-*}" ] &&
 		return $rc
 
 	if bad=$(printf %s "$rpm_deps" |cut -d' ' -f1 |egrep '^kernel-(image|modules)'); then