Re: [devel] [SCM] packages/kernel-build-tools: heads/master

Re: [devel] [SCM] packages/kernel-build-tools: heads/master

[Dmitry V. Levin]

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

On Tue, Oct 14, 2008 at 02:03:59PM +0400, Konstantin A. Lepikhov wrote:
> Update of /people/lakostis/packages/kernel-build-tools.git
[...]
> new file mode 100755
> index 0000000..66f7500
> --- /dev/null
> +++ b/modalias.prov
> @@ -0,0 +1,20 @@
> +#!/bin/sh -efu
> +#
> +# Copyright (c) 2008 Konstantin Lepikhov <lakostis@altlinux.org>.
> +# 
> +# This program is free software; you can redistribute it and/or modify
> +# it under the terms of the GNU General Public License as published by
> +# the Free Software Foundation; either version 2 of the License, or
> +# (at your option) any later version.
> +
> +. /usr/lib/rpm/functions
> +. /usr/lib/rpm/find-package
> +
> +ModaliasReq() {
> +	local f t

Поскольку тип не проверятеся, t здесь лишний.

> +	for f in "$@"; do
> +		/sbin/modinfo -F alias "$f" |sed -e 's/.*/modalias:&/' -e 's/[,]/?/g'
> +	done
> +}
> +
> +ArgvFileAction ModaliasReq "$@"
> diff --git a/modalias.prov.files b/modalias.prov.files
> new file mode 100755
> index 0000000..b209954
> --- /dev/null
> +++ b/modalias.prov.files
> @@ -0,0 +1,11 @@
> +#!/bin/sh -efu
> +buildroot="${RPM_BUILD_ROOT-}" 
> +
> +while IFS=$'\t' read -r f t; do
> +	case "$f" in
> +		"$buildroot"/lib/modules/*/kernel/drivers/*.ko|"$buildroot"/lib/modules/*/kernel/sound/*.ko)
> +			[ -z "$f" ] || echo "$f" ;;
> +		"$buildroot"/lib/modules/*/*/*.ko)
> +			[ -z "$f" ] || echo "$f" ;;

Избыточность: последний case вполне покрывает первые два.

> +	esac
> +done


-- 
ldv

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

Re: [devel] [SCM] packages/kernel-build-tools: heads/master

[Konstantin A. Lepikhov]

14.10.2008 15:55, Dmitry V. Levin пишет:
> On Tue, Oct 14, 2008 at 02:03:59PM +0400, Konstantin A. Lepikhov wrote:
>   
>> Update of /people/lakostis/packages/kernel-build-tools.git
>>     
> [...]
>   
>> new file mode 100755
>> index 0000000..66f7500
>> --- /dev/null
>> +++ b/modalias.prov
>> @@ -0,0 +1,20 @@
>> +#!/bin/sh -efu
>> +#
>> +# Copyright (c) 2008 Konstantin Lepikhov <lakostis@altlinux.org>.
>> +# 
>> +# This program is free software; you can redistribute it and/or modify
>> +# it under the terms of the GNU General Public License as published by
>> +# the Free Software Foundation; either version 2 of the License, or
>> +# (at your option) any later version.
>> +
>> +. /usr/lib/rpm/functions
>> +. /usr/lib/rpm/find-package
>> +
>> +ModaliasReq() {
>> +	local f t
>>     
>
> Поскольку тип не проверятеся, t здесь лишний.
>
>   
Можно и тип проверить ;)
>> +	for f in "$@"; do
>> +		/sbin/modinfo -F alias "$f" |sed -e 's/.*/modalias:&/' -e 's/[,]/?/g'
>> +	done
>> +}
>> +
>> +ArgvFileAction ModaliasReq "$@"
>> diff --git a/modalias.prov.files b/modalias.prov.files
>> new file mode 100755
>> index 0000000..b209954
>> --- /dev/null
>> +++ b/modalias.prov.files
>> @@ -0,0 +1,11 @@
>> +#!/bin/sh -efu
>> +buildroot="${RPM_BUILD_ROOT-}" 
>> +
>> +while IFS=$'\t' read -r f t; do
>> +	case "$f" in
>> +		"$buildroot"/lib/modules/*/kernel/drivers/*.ko|"$buildroot"/lib/modules/*/kernel/sound/*.ko)
>> +			[ -z "$f" ] || echo "$f" ;;
>> +		"$buildroot"/lib/modules/*/*/*.ko)
>> +			[ -z "$f" ] || echo "$f" ;;
>>     
>
> Избыточность: последний case вполне покрывает первые два.
>
>   
Тогда вопрос - как это сделать правильно, если нужно проверять каталоги

$buildroot"/lib/modules/<kver>/kernel/drivers/*.ko|"$buildroot"/lib/modules/<kver>/kernel/sound/*.ko

и 

$buildroot"/lib/modules/<kver>/<строго один уровень и не kernel>/*.ko

Re: [devel] [SCM] packages/kernel-build-tools: heads/master

[Dmitry V. Levin]

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

On Tue, Oct 14, 2008 at 04:07:22PM +0400, Konstantin A. Lepikhov wrote:
> 14.10.2008 15:55, Dmitry V. Levin пишет:
> > On Tue, Oct 14, 2008 at 02:03:59PM +0400, Konstantin A. Lepikhov wrote:
> >   
> >> Update of /people/lakostis/packages/kernel-build-tools.git
> >>     
> > [...]
> >   
> >> new file mode 100755
> >> index 0000000..66f7500
> >> --- /dev/null
> >> +++ b/modalias.prov
> >> @@ -0,0 +1,20 @@
> >> +#!/bin/sh -efu
> >> +#
> >> +# Copyright (c) 2008 Konstantin Lepikhov <lakostis@altlinux.org>.
> >> +# 
> >> +# This program is free software; you can redistribute it and/or modify
> >> +# it under the terms of the GNU General Public License as published by
> >> +# the Free Software Foundation; either version 2 of the License, or
> >> +# (at your option) any later version.
> >> +
> >> +. /usr/lib/rpm/functions
> >> +. /usr/lib/rpm/find-package
> >> +
> >> +ModaliasReq() {
> >> +	local f t
> >
> > Поскольку тип не проверятеся, t здесь лишний.
> >   
> Можно и тип проверить ;)

Вопрос не совсем праздный.  Например, как лучше обрабатывать ссылки.

> >> +	for f in "$@"; do
> >> +		/sbin/modinfo -F alias "$f" |sed -e 's/.*/modalias:&/' -e 's/[,]/?/g'
> >> +	done
> >> +}
> >> +
> >> +ArgvFileAction ModaliasReq "$@"
> >> diff --git a/modalias.prov.files b/modalias.prov.files
> >> new file mode 100755
> >> index 0000000..b209954
> >> --- /dev/null
> >> +++ b/modalias.prov.files
> >> @@ -0,0 +1,11 @@
> >> +#!/bin/sh -efu
> >> +buildroot="${RPM_BUILD_ROOT-}" 
> >> +
> >> +while IFS=$'\t' read -r f t; do
> >> +	case "$f" in
> >> +		"$buildroot"/lib/modules/*/kernel/drivers/*.ko|"$buildroot"/lib/modules/*/kernel/sound/*.ko)
> >> +			[ -z "$f" ] || echo "$f" ;;
> >> +		"$buildroot"/lib/modules/*/*/*.ko)
> >> +			[ -z "$f" ] || echo "$f" ;;
> >>     
> >
> > Избыточность: последний case вполне покрывает первые два.

Кстати, [ -z "$f" ] тоже лишний, простого echo достаточно.

> Тогда вопрос - как это сделать правильно, если нужно проверять каталоги
> 
> $buildroot"/lib/modules/<kver>/kernel/drivers/*.ko|"$buildroot"/lib/modules/<kver>/kernel/sound/*.ko
> 
> и 
> 
> $buildroot"/lib/modules/<kver>/<строго один уровень и не kernel>/*.ko

Если продолжать стилистику case/esac, тожно так:

"$buildroot"/lib/modules/*/kernel/drivers/*.ko|"$buildroot"/lib/modules/*/kernel/sound/*.ko)
	echo "$f" ;;
"$buildroot"/lib/modules/*/kernel/*|"$buildroot"/lib/modules/*/*/*/*.ko)
	continue ;;
"$buildroot"/lib/modules/*/*/*.ko)
	echo "$f" ;;

А стоит ли так сужать множество обрабатываемых .ko-файлов?
Разве в других местах depmod/modprobe их не найдут?


-- 
ldv

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

Re: [devel] [SCM] packages/kernel-build-tools: heads/master

[Konstantin A. Lepikhov]

14.10.2008 16:20, Dmitry V. Levin пишет:
>>>
>>> Поскольку тип не проверятеся, t здесь лишний.
>>>   
>>>       
>> Можно и тип проверить ;)
>>     
>
> Вопрос не совсем праздный.  Например, как лучше обрабатывать ссылки.
>
>   
>
any help will be very appreciated.
>
> Если продолжать стилистику case/esac, тожно так:
>
> "$buildroot"/lib/modules/*/kernel/drivers/*.ko|"$buildroot"/lib/modules/*/kernel/sound/*.ko)
> 	echo "$f" ;;
> "$buildroot"/lib/modules/*/kernel/*|"$buildroot"/lib/modules/*/*/*/*.ko)
> 	continue ;;
> "$buildroot"/lib/modules/*/*/*.ko)
> 	echo "$f" ;;
>
> А стоит ли так сужать множество обрабатываемых .ko-файлов?
> Разве в других местах depmod/modprobe их не найдут?
>
>
>   
Дело не в depmod/modprobe, а в том, что все modaliases ядра нам не нужны.

Re: [devel] [SCM] packages/kernel-build-tools: heads/master

[Led]

On Tuesday 14 October 2008 15:31:13 Konstantin A. Lepikhov wrote:
> 14.10.2008 16:20, Dmitry V. Levin пишет:
> >>> Поскольку тип не проверятеся, t здесь лишний.
> >>
> >> Можно и тип проверить ;)
> >
> > Вопрос не совсем праздный.  Например, как лучше обрабатывать ссылки.
>
> any help will be very appreciated.

readlink -q{e|m}
?

-- 
Led

Re: [devel] [SCM] packages/kernel-build-tools: heads/master

[Dmitry V. Levin]

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

On Tue, Oct 14, 2008 at 03:43:07PM +0300, Led wrote:
> On Tuesday 14 October 2008 15:31:13 Konstantin A. Lepikhov wrote:
> > 14.10.2008 16:20, Dmitry V. Levin пишет:
> > >>> Поскольку тип не проверятеся, t здесь лишний.
> > >>
> > >> Можно и тип проверить ;)
> > >
> > > Вопрос не совсем праздный.  Например, как лучше обрабатывать ссылки.
> >
> > any help will be very appreciated.
> 
> readlink -q{e|m}

Это зависит от целей нововведения.  Подождём...


-- 
ldv

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

Re: [devel] [SCM] packages/kernel-build-tools: heads/master

[Konstantin A. Lepikhov]

14.10.2008 16:44, Dmitry V. Levin пишет:
> On Tue, Oct 14, 2008 at 03:43:07PM +0300, Led wrote:
>   
>> On Tuesday 14 October 2008 15:31:13 Konstantin A. Lepikhov wrote:
>>     
>>> 14.10.2008 16:20, Dmitry V. Levin пишет:
>>>       
>>>>>> Поскольку тип не проверятеся, t здесь лишний.
>>>>>>             
>>>>> Можно и тип проверить ;)
>>>>>           
>>>> Вопрос не совсем праздный.  Например, как лучше обрабатывать ссылки.
>>>>         
>>> any help will be very appreciated.
>>>       
>> readlink -q{e|m}
>>     
>
> Это зависит от целей нововведения.  Подождём...
>
>   
В-принципе, наличие симлинков внутри /kernel/* это зло, поскольку они
там никому не нужны, а вместо них нужен правильный файлик в
/etc/modprobe.d/, который можно вкладывать в ядро. Поэтому, еще одной
проверкой в sisyphus_check станет больше :)

Re: [devel] [SCM] packages/kernel-build-tools: heads/master

[Alexey Gladkov]

Dmitry V. Levin wrote:
> А стоит ли так сужать множество обрабатываемых .ko-файлов?
> Разве в других местах depmod/modprobe их не найдут?

Дим, скоро будет анонс с описанием для чего Костя вообще сделал. Костя 
это закоммитил чтобы я мог ссылку дать :)

Там я отвечу на твой вопрос.

-- 
Rgrds, legion

Re: [devel] [SCM] packages/kernel-build-tools: heads/master

[Anton Farygin]

Dmitry V. Levin пишет:
> On Tue, Oct 14, 2008 at 02:03:59PM +0400, Konstantin A. Lepikhov wrote:
>> Update of /people/lakostis/packages/kernel-build-tools.git

а можно поподробнее, зачем изобретается такое поведение ?

Ядро будет провайдить весь pcimap ?

А кто это будет хотеть ? И как будет устроена работа с масками из pcimap ?

Re: [devel] [SCM] packages/kernel-build-tools: heads/master

[Konstantin A. Lepikhov]

14.10.2008 16:41, Anton Farygin пишет:
> Dmitry V. Levin пишет:
>> On Tue, Oct 14, 2008 at 02:03:59PM +0400, Konstantin A. Lepikhov wrote:
>>> Update of /people/lakostis/packages/kernel-build-tools.git
>
> а можно поподробнее, зачем изобретается такое поведение ?
>
> Ядро будет провайдить весь pcimap ?
>
> А кто это будет хотеть ? И как будет устроена работа с масками из
> pcimap ?
>
Аналитикам из -devel просьба пока не беспокоиться.

Re: [devel] [SCM] packages/kernel-build-tools: heads/master

[Anton Farygin]

Konstantin A. Lepikhov пишет:
> 14.10.2008 16:41, Anton Farygin пишет:
>> Dmitry V. Levin пишет:
>>> On Tue, Oct 14, 2008 at 02:03:59PM +0400, Konstantin A. Lepikhov wrote:
>>>> Update of /people/lakostis/packages/kernel-build-tools.git
>> а можно поподробнее, зачем изобретается такое поведение ?
>>
>> Ядро будет провайдить весь pcimap ?
>>
>> А кто это будет хотеть ? И как будет устроена работа с масками из
>> pcimap ?
>>
> Аналитикам из -devel просьба пока не беспокоиться.

Я просил не успокоить меня, а разъяснить - что это и зачем.

Re: [devel] [SCM] packages/kernel-build-tools: heads/master

[Konstantin A. Lepikhov]

14.10.2008 17:23, Anton Farygin пишет:
> Konstantin A. Lepikhov пишет:
>> 14.10.2008 16:41, Anton Farygin пишет:
>>> Dmitry V. Levin пишет:
>>>> On Tue, Oct 14, 2008 at 02:03:59PM +0400, Konstantin A. Lepikhov
>>>> wrote:
>>>>> Update of /people/lakostis/packages/kernel-build-tools.git
>>> а можно поподробнее, зачем изобретается такое поведение ?
>>>
>>> Ядро будет провайдить весь pcimap ?
>>>
>>> А кто это будет хотеть ? И как будет устроена работа с масками из
>>> pcimap ?
>>>
>> Аналитикам из -devel просьба пока не беспокоиться.
>
> Я просил не успокоить меня, а разъяснить - что это и зачем.
>
>
Не хочу. Будет анонс - разъясним.

Re: [devel] [SCM] packages/kernel-build-tools: heads/master

[Alexey Gladkov]

Anton Farygin wrote:
> Я просил не успокоить меня, а разъяснить - что это и зачем.

Антон, это эксперимент. Как только мы закончим со стендом, так я сразу 
всё разъясню. Это будет удобнее и понятнее.

-- 
Rgrds, legion

Re: [devel] [SCM] packages/kernel-build-tools: heads/master

[Anton Farygin]

Alexey Gladkov пишет:
> Anton Farygin wrote:
>> Я просил не успокоить меня, а разъяснить - что это и зачем.
> 
> Антон, это эксперимент. Как только мы закончим со стендом, так я сразу 
> всё разъясню. Это будет удобнее и понятнее.

Хорошо, спасибо.