[devel] (legion) packages/bootloader-utils: heads/master

[devel] (legion) packages/bootloader-utils: heads/master

[Alexey Tourbin]

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

On Sat, Feb 10, 2007 at 01:06:09PM +0300, Alexey Gladkov wrote:
> Update of /people/legion/packages/bootloader-utils.git
> 
> Changes statistics since `0.3.1-alt1.1' follows:
>  bootloader-utils-lilo-params-quoting.patch |   19 +++++++++++++++++++
>  bootloader-utils.spec                      |    8 +++++++-
>  2 files changed, 26 insertions(+), 1 deletions(-)
> 
> Changelog since `0.3.1-alt1.1' follows:
> commit 3c26b8cfb09e3291774cb56aa8f32c1e67ee9b00
> Author: Alexey Gladkov <legion@altlinux.org>
> Date:   Sat Feb 10 13:06:04 2007 +0300
> 
>     0.3.1-alt1.1.1
>     
>     - NMU:
>         - quote lilo parameters.
>     
>     Signed-off-by: Alexey Gladkov <legion@altlinux.org>
> 
> Full diff since `0.3.1-alt1.1' follows:
> diff --git a/bootloader-utils-lilo-params-quoting.patch b/bootloader-utils-lilo-params-quoting.patch
> new file mode 100644
> index 0000000..497b41d
> --- /dev/null
> +++ b/bootloader-utils-lilo-params-quoting.patch
> @@ -0,0 +1,19 @@
> +--- bootloader-utils/lilo.fix	2007-02-09 18:45:27 +0300
> ++++ bootloader-utils/lilo	2007-02-09 18:45:42 +0300
> +@@ -53,12 +53,12 @@ sub add_kernel {
> + 	my $label = kernel_label($version);
> + 	my $root = getroot();
> + 	/\/vmlinuz-\Q$version\E\s/
> + 		or $_ .= <<EOF;
> +-image=/boot/vmlinuz-$version
> +-	initrd=/boot/initrd-$version.img
> +-	label=$label
> +-	root=$root
> ++image="/boot/vmlinuz-$version"
> ++	initrd="/boot/initrd-$version.img"
> ++	label="$label"
> ++	root="$root"
> + 	read-only
> + 	optional
> + EOF
> + 	return $_;

Теперь записи из lilo.conf будут неправильно удаляться, потому что
в шаблоне на удаление не продусмотрены кавчки.

Почему появилась нужда в кавычках?

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

Re: [devel] (legion) packages/bootloader-utils: heads/master

[Alexey Gladkov]

Alexey Tourbin wrote:
Теперь записи из lilo.conf будут неправильно удаляться, потому что
> в шаблоне на удаление не продусмотрены кавчки.

Моя вина. Проглядел.

> Почему появилась нужда в кавычках?

Потому что если в /etc/fstab вместо девайсов используются LABEL/UUID,
то installkernel добавлял в lilo.conf строки вида root=UUID=bna ...
что портило конфиг... нужно root="UUID=bna"

Вообще, все значения параметров в конфиге lilo.conf стоит брать в кавычки.

-- 
Rgrds, legion

Re: [devel] (legion) packages/bootloader-utils: heads/master

[Alexey Tourbin]

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

On Thu, Feb 22, 2007 at 01:18:53AM +0300, Alexey Gladkov wrote:
> Alexey Tourbin wrote:
> Теперь записи из lilo.conf будут неправильно удаляться, потому что
> > в шаблоне на удаление не продусмотрены кавчки.
> Моя вина. Проглядел.

На самом деле просто не будет удаляться.

> > Почему появилась нужда в кавычках?
> 
> Потому что если в /etc/fstab вместо девайсов используются LABEL/UUID,
> то installkernel добавлял в lilo.conf строки вида root=UUID=bna ...
> что портило конфиг... нужно root="UUID=bna"

А откуда insatllkernel берет root="UUID=bna"?  Из /usr/sbin/rdev?

Там на самом деле три способа определения root, в порядке предпочтения:
1) /proc/cmdline
2) fstab + mtab;
3) /usr/sbin/rdev ...

> Вообще, все значения параметров в конфиге lilo.conf стоит брать в кавычки.

И как быть в случае с grub, или gurb у нас пока не актуален?

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

Re: [devel] (legion) packages/bootloader-utils: heads/master

[Alexey Gladkov]

Alexey Tourbin wrote:
> On Thu, Feb 22, 2007 at 01:18:53AM +0300, Alexey Gladkov wrote:
>> Alexey Tourbin wrote:
>> Теперь записи из lilo.conf будут неправильно удаляться, потому что
>>> в шаблоне на удаление не продусмотрены кавчки.
>> Моя вина. Проглядел.
> 
> На самом деле просто не будет удаляться.

Как выяснилось, шаблон на добавление тоже не ожидает кавычек (#10903)

> 
>>> Почему появилась нужда в кавычках?
>> Потому что если в /etc/fstab вместо девайсов используются LABEL/UUID,
>> то installkernel добавлял в lilo.conf строки вида root=UUID=bna ...
>> что портило конфиг... нужно root="UUID=bna"
> 
> А откуда insatllkernel берет root="UUID=bna"?  Из /usr/sbin/rdev?

В будущем дистрибутиве в fstab будут UUID вместо имён устройств и в
создаваемом первом lilo.conf тоже ... собственно отсюда я и нарвался
на эту проблему.

> Там на самом деле три способа определения root, в порядке предпочтения:
> 1) /proc/cmdline
> 2) fstab + mtab;
> 3) /usr/sbin/rdev ...

> И как быть в случае с grub, или gurb у нас пока не актуален?

Я видел, что для grub кавычки тоже не используются, но про него
сказать ничего не могу ...

-- 
Rgrds, legion

Re: [devel] (legion) packages/bootloader-utils: heads/master

[Alexey Tourbin]

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

On Thu, Feb 22, 2007 at 01:32:48AM +0300, Alexey Gladkov wrote:
> Alexey Tourbin wrote:
> > On Thu, Feb 22, 2007 at 01:18:53AM +0300, Alexey Gladkov wrote:
> >> Alexey Tourbin wrote:
> >> Теперь записи из lilo.conf будут неправильно удаляться, потому что
> >>> в шаблоне на удаление не продусмотрены кавчки.
> >> Моя вина. Проглядел.
> > 
> > На самом деле просто не будет удаляться.
> 
> Как выяснилось, шаблон на добавление тоже не ожидает кавычек (#10903)
> 
> > 
> >>> Почему появилась нужда в кавычках?
> >> Потому что если в /etc/fstab вместо девайсов используются LABEL/UUID,
> >> то installkernel добавлял в lilo.conf строки вида root=UUID=bna ...
> >> что портило конфиг... нужно root="UUID=bna"
> > 
> > А откуда insatllkernel берет root="UUID=bna"?  Из /usr/sbin/rdev?
> 
> В будущем дистрибутиве в fstab будут UUID вместо имён устройств и в
> создаваемом первом lilo.conf тоже ... собственно отсюда я и нарвался
> на эту проблему.
> 
> > Там на самом деле три способа определения root, в порядке предпочтения:
> > 1) /proc/cmdline
> > 2) fstab + mtab;
> > 3) /usr/sbin/rdev ...
> 
> > И как быть в случае с grub, или gurb у нас пока не актуален?
> 
> Я видел, что для grub кавычки тоже не используются, но про него
> сказать ничего не могу ...

Было достаточно брать в кавычки только root="...".
В общем не знаю как лучше сделать, сейчас как раз ковыряю...

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

Re: [devel] (legion) packages/bootloader-utils: heads/master

[Alexey Gladkov]

Alexey Tourbin wrote:
> Было достаточно брать в кавычки только root="...".
> В общем не знаю как лучше сделать, сейчас как раз ковыряю...

В принципе да, но:

[root@legion tmp]# grep label /tmp/lilo.conf
        label="linux=1"
        label=failsafe

[root@legion tmp]# lilo -C /tmp/lilo.conf -t
Added linux=1
Added failsafe

[root@legion tmp]# grep label /tmp/lilo.conf
        label=linux=1
        label=failsafe

[root@legion tmp]# lilo -C /tmp/lilo.conf -t
Syntax error at or above line 12 in file '/tmp/lilo.conf'

Как видишь '=' допустимо и в label. :(

-- 
Rgrds, legion

Re: [devel] (legion) packages/bootloader-utils: heads/master

[Alexey Tourbin]

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

On Thu, Feb 22, 2007 at 01:42:29AM +0300, Alexey Gladkov wrote:
> Alexey Tourbin wrote:
> > Было достаточно брать в кавычки только root="...".
> > В общем не знаю как лучше сделать, сейчас как раз ковыряю...
> 
> В принципе да, но:
> 
> [root@legion tmp]# grep label /tmp/lilo.conf
>         label="linux=1"
>         label=failsafe
> 
> [root@legion tmp]# lilo -C /tmp/lilo.conf -t
> Added linux=1
> Added failsafe
> 
> [root@legion tmp]# grep label /tmp/lilo.conf
>         label=linux=1
>         label=failsafe
> 
> [root@legion tmp]# lilo -C /tmp/lilo.conf -t
> Syntax error at or above line 12 in file '/tmp/lilo.conf'
> 
> Как видишь '=' допустимо и в label. :(

Хорошо, будем всё квотить.

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

Re: [devel] (legion) packages/bootloader-utils: heads/master

[Alexey Tourbin]

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

On Thu, Feb 22, 2007 at 01:47:56AM +0300, Alexey Tourbin wrote:
> On Thu, Feb 22, 2007 at 01:42:29AM +0300, Alexey Gladkov wrote:
> > Alexey Tourbin wrote:
> > > Было достаточно брать в кавычки только root="...".
> > > В общем не знаю как лучше сделать, сейчас как раз ковыряю...
> > 
> > В принципе да, но:
> > 
> > [root@legion tmp]# grep label /tmp/lilo.conf
> >         label="linux=1"
> >         label=failsafe
> > 
> > [root@legion tmp]# lilo -C /tmp/lilo.conf -t
> > Added linux=1
> > Added failsafe
> > 
> > [root@legion tmp]# grep label /tmp/lilo.conf
> >         label=linux=1
> >         label=failsafe
> > 
> > [root@legion tmp]# lilo -C /tmp/lilo.conf -t
> > Syntax error at or above line 12 in file '/tmp/lilo.conf'
> > 
> > Как видишь '=' допустимо и в label. :(
> 
> Хорошо, будем всё квотить.

vim как-то нехорошо подсвечивает текст в кавычках (filetype=lilo)

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

Re: [devel] (legion) packages/bootloader-utils: heads/master

[Alexey I. Froloff]

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

* Alexey Tourbin <at@> [070222 02:39]:
> vim как-то нехорошо подсвечивает текст в кавычках (filetype=lilo)
Гадкий, негодный vim!  Будем проводить раз'яснительную работу...

P.S. Ты не проверил тот патчик на тему 'backupskip'?  А то бы я и
его приложил...

-- 
Regards, Alexey I. Froloff
AIF5-RIPN, AIF5-RIPE
-------------------------------------------
  Inform-Mobil, Ltd. System Administrator
       http://www.inform-mobil.ru/

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

Re: [devel] (legion) packages/bootloader-utils: heads/master

[Alexey Tourbin]

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

On Thu, Feb 22, 2007 at 10:38:54AM +0300, Alexey I. Froloff wrote:
> * Alexey Tourbin <at@> [070222 02:39]:
> > vim как-то нехорошо подсвечивает текст в кавычках (filetype=lilo)
> Гадкий, негодный vim!  Будем проводить раз'яснительную работу...
> 
> P.S. Ты не проверил тот патчик на тему 'backupskip'?  А то бы я и
> его приложил...

Нет.  Прикладывай.

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

Re: [devel] (legion) packages/bootloader-utils: heads/master

[Sergey Vlasov]

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

On Thu, Feb 22, 2007 at 01:24:28AM +0300, Alexey Tourbin wrote:
> On Thu, Feb 22, 2007 at 01:18:53AM +0300, Alexey Gladkov wrote:
> > Alexey Tourbin wrote:
> > Теперь записи из lilo.conf будут неправильно удаляться, потому что
> > > в шаблоне на удаление не продусмотрены кавчки.
> > Моя вина. Проглядел.
> 
> На самом деле просто не будет удаляться.
> 
> > > Почему появилась нужда в кавычках?
> > 
> > Потому что если в /etc/fstab вместо девайсов используются LABEL/UUID,
> > то installkernel добавлял в lilo.conf строки вида root=UUID=bna ...
> > что портило конфиг... нужно root="UUID=bna"
> 
> А откуда insatllkernel берет root="UUID=bna"?  Из /usr/sbin/rdev?
> 
> Там на самом деле три способа определения root, в порядке предпочтения:
> 1) /proc/cmdline
> 2) fstab + mtab;
> 3) /usr/sbin/rdev ...
> 
> > Вообще, все значения параметров в конфиге lilo.conf стоит брать в кавычки.
> 
> И как быть в случае с grub, или gurb у нас пока не актуален?

Там этой проблемы нет (root=... - это такой же параметр в команде kernel,
как и все остальные; разбивкой строки на отдельные параметры занимается
либо ядро, либо скрипты из initramfs).

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