ALT Linux Team development discussions
 help / color / mirror / Atom feed
* Re: [devel] [#91829] FAILED kernel-image-3.0.git=3.0.68-alt1
  @ 2013-03-04 21:28 ` Led
  2013-03-04 21:46   ` Alexey Gladkov
  2013-03-04 21:59   ` Dmitry V. Levin
  0 siblings, 2 replies; 10+ messages in thread
From: Led @ 2013-03-04 21:28 UTC (permalink / raw)
  To: girar-builder; +Cc: ALT Linux Team development discussions

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"?

-- 
Led


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [devel] [#91829] FAILED kernel-image-3.0.git=3.0.68-alt1
  2013-03-04 21:28 ` [devel] [#91829] FAILED kernel-image-3.0.git=3.0.68-alt1 Led
@ 2013-03-04 21:46   ` Alexey Gladkov
  2013-03-04 21:58     ` Led
  2013-03-04 21:59   ` Dmitry V. Levin
  1 sibling, 1 reply; 10+ messages in thread
From: Alexey Gladkov @ 2013-03-04 21:46 UTC (permalink / raw)
  To: devel

05.03.2013 01:28, 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"?

Пакет не начинающийся с kernel- не должен иметь зависимости на ядро.

-- 
Rgrds, legion



^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [devel] [#91829] FAILED kernel-image-3.0.git=3.0.68-alt1
  2013-03-04 21:46   ` Alexey Gladkov
@ 2013-03-04 21:58     ` Led
  0 siblings, 0 replies; 10+ messages in thread
From: Led @ 2013-03-04 21:58 UTC (permalink / raw)
  To: ALT Linux Team development discussions

On Monday 04 March 2013 23:46:15 Alexey Gladkov wrote:
> 05.03.2013 01:28, 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"?
>
> Пакет не начинающийся с kernel- не должен иметь зависимости на ядро.

Но я не могу пакет с фирмварями назвать kernel-*, потому как другое "правило" sisyphus_check этого не позволяет.

-- 
Led

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [devel] [#91829] FAILED kernel-image-3.0.git=3.0.68-alt1
  2013-03-04 21:28 ` [devel] [#91829] FAILED kernel-image-3.0.git=3.0.68-alt1 Led
  2013-03-04 21:46   ` Alexey Gladkov
@ 2013-03-04 21:59   ` Dmitry V. Levin
  2013-03-04 22:16     ` Led
  2013-03-04 22:20     ` Alexey Gladkov
  1 sibling, 2 replies; 10+ messages in thread
From: Dmitry V. Levin @ 2013-03-04 21:59 UTC (permalink / raw)
  To: ALT Linux Team development discussions

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

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, но sisyphus_check этому не обучен.


-- 
ldv

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

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [devel] [#91829] FAILED kernel-image-3.0.git=3.0.68-alt1
  2013-03-04 21:59   ` Dmitry V. Levin
@ 2013-03-04 22:16     ` Led
  2013-03-05  0:11       ` Dmitry V. Levin
  2013-03-04 22:20     ` Alexey Gladkov
  1 sibling, 1 reply; 10+ messages in thread
From: Led @ 2013-03-04 22:16 UTC (permalink / raw)
  To: ALT Linux Team development discussions

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 этому не обучен. 

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

-- 
Led

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [devel] [#91829] FAILED kernel-image-3.0.git=3.0.68-alt1
  2013-03-04 21:59   ` Dmitry V. Levin
  2013-03-04 22:16     ` Led
@ 2013-03-04 22:20     ` Alexey Gladkov
  1 sibling, 0 replies; 10+ messages in thread
From: Alexey Gladkov @ 2013-03-04 22:20 UTC (permalink / raw)
  To: devel

05.03.2013 01:59, 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, но sisyphus_check этому не обучен.

У нас же было правило как именовать фирмварь. Недавно это обсуждали.

-- 
Rgrds, legion



^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [devel] [#91829] FAILED kernel-image-3.0.git=3.0.68-alt1
  2013-03-04 22:16     ` Led
@ 2013-03-05  0:11       ` Dmitry V. Levin
  2013-03-05  3:20         ` Led
  0 siblings, 1 reply; 10+ messages in thread
From: Dmitry V. Levin @ 2013-03-05  0:11 UTC (permalink / raw)
  To: ALT Linux Team development discussions

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

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.


-- 
ldv

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

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [devel] [#91829] FAILED kernel-image-3.0.git=3.0.68-alt1
  2013-03-05  0:11       ` Dmitry V. Levin
@ 2013-03-05  3:20         ` Led
  2013-03-05  8:03           ` Michael Shigorin
  2013-03-19 14:52           ` Dmitry V. Levin
  0 siblings, 2 replies; 10+ messages in thread
From: Led @ 2013-03-05  3:20 UTC (permalink / raw)
  To: ALT Linux Team development discussions

[-- 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

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [devel] [#91829] FAILED kernel-image-3.0.git=3.0.68-alt1
  2013-03-05  3:20         ` Led
@ 2013-03-05  8:03           ` Michael Shigorin
  2013-03-19 14:52           ` Dmitry V. Levin
  1 sibling, 0 replies; 10+ messages in thread
From: Michael Shigorin @ 2013-03-05  8:03 UTC (permalink / raw)
  To: ALT Linux Team development discussions

On Tue, Mar 05, 2013 at 05:20:24AM +0200, Led wrote:
> > Да, например, можно пропатчить sisyphus_check.d/100-check-deps :)
> Ну, например так как (во вложении)?

[ ! ... ] || ... отличается от [ ... ] && ... кодом возврата
в случае, если условие не выполнилось; для последней команды
или при -e это может быть критично.

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


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [devel] [#91829] FAILED kernel-image-3.0.git=3.0.68-alt1
  2013-03-05  3:20         ` Led
  2013-03-05  8:03           ` Michael Shigorin
@ 2013-03-19 14:52           ` Dmitry V. Levin
  1 sibling, 0 replies; 10+ messages in thread
From: Dmitry V. Levin @ 2013-03-19 14:52 UTC (permalink / raw)
  To: ALT Linux Team development discussions

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

On Tue, Mar 05, 2013 at 05:20:24AM +0200, Led wrote:
> 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-зависимости любым
подпакетам, собирающимся из kernel-image-*.


-- 
ldv

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

^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2013-03-19 14:52 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-03-04 21:28 ` [devel] [#91829] FAILED kernel-image-3.0.git=3.0.68-alt1 Led
2013-03-04 21:46   ` Alexey Gladkov
2013-03-04 21:58     ` Led
2013-03-04 21:59   ` Dmitry V. Levin
2013-03-04 22:16     ` Led
2013-03-05  0:11       ` Dmitry V. Levin
2013-03-05  3:20         ` Led
2013-03-05  8:03           ` Michael Shigorin
2013-03-19 14:52           ` Dmitry V. Levin
2013-03-04 22:20     ` Alexey Gladkov

ALT Linux Team development discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://lore.altlinux.org/devel/0 devel/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 devel devel/ http://lore.altlinux.org/devel \
		devel@altlinux.org devel@altlinux.ru devel@lists.altlinux.org devel@lists.altlinux.ru devel@linux.iplabs.ru mandrake-russian@linuxteam.iplabs.ru sisyphus@linuxteam.iplabs.ru
	public-inbox-index devel

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://lore.altlinux.org/org.altlinux.lists.devel


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git