From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <rider@altlinux.com>
Message-ID: <3FC4938B.1060609@altlinux.com>
Date: Wed, 26 Nov 2003 14:50:35 +0300
From: Anton Farygin <rider@altlinux.com>
Organization: ALT Linux
User-Agent: Mozilla/5.0 (X11; U; Linux i686; ru-RU; rv:1.5) Gecko/20031108
X-Accept-Language: en-us, en
MIME-Version: 1.0
To: ALT Linux kernel packages development <devel-kernel@altlinux.ru>
Subject: Re: [d-kernel] Re: Fwd: [Bug 3318] New: =?KOI8-R?Q?=F0=D2=C9_=D5?=
	=?KOI8-R?Q?=D3=D4=C1=CE=CF=D7=CB=C5_=CC=C0=C2=CF=C7=CF_=D1=C4=D2=C1_?=
	=?KOI8-R?Q?=CE=C5_=D0=D2=CF=D0=C9=D3=D9=D7=C1=C5=D4=D3=D1_=D3=CF=CF=D4?=
	=?KOI8-R?Q?=D7=C5=D3=D4=D7=D5=C0=DD=C1=D1_=D3=C5=CB=C3=C9=D1_=D7_=CD?=
	=?KOI8-R?Q?=C5=CE=C0_grub?=
References: <20031125140935.GJ2377@julia.office.altlinux.ru>	<20031125141644.GC22171@master.mivlgu.local>	<20031125142522.GK2377@julia.office.altlinux.ru>	<20031125143609.GL2377@julia.office.altlinux.ru>
	<20031125191915.GR2377@julia.office.altlinux.ru>
In-Reply-To: <20031125191915.GR2377@julia.office.altlinux.ru>
Content-Type: text/plain; charset=KOI8-R; format=flowed
Content-Transfer-Encoding: 8bit
X-BeenThere: devel-kernel@altlinux.ru
X-Mailman-Version: 2.1.3
Precedence: list
Reply-To: ALT Linux kernel packages development <devel-kernel@altlinux.ru>
List-Id: ALT Linux kernel packages development <devel-kernel.altlinux.ru>
List-Unsubscribe: <listinfo/devel-kernel>,
	<mailto:devel-kernel-request@altlinux.ru?subject=unsubscribe>
List-Archive: </pipermail/devel-kernel>
List-Post: <mailto:devel-kernel@altlinux.ru>
List-Help: <mailto:devel-kernel-request@altlinux.ru?subject=help>
List-Subscribe: <listinfo/devel-kernel>,
	<mailto:devel-kernel-request@altlinux.ru?subject=subscribe>
X-List-Received-Date: Wed, 26 Nov 2003 11:45:38 -0000
Archived-At: <http://lore.altlinux.org/devel-kernel/3FC4938B.1060609@altlinux.com/>
List-Archive: <http://lore.altlinux.org/devel-kernel/>
List-Post: <mailto:devel-kernel@altlinux.org>

Alexey Tourbin wrote:
> On Tue, Nov 25, 2003 at 05:36:09PM +0300, Alexey Tourbin wrote:
> 
>>--- /usr/share/loader/grub-	2003-03-19 17:41:06 +0300
>>+++ /usr/share/loader/grub	2003-11-25 17:34:24 +0300
>>@@ -154,6 +154,7 @@ sub get_boot_partitions {
>>     local *F;
>>     open F, '/etc/fstab'; 
>>     while (<F>) {
>>+	next if /^#/;
>>  	my @s = split ' ';
>> 	$part = $s[0] if $s[1] =~ m|/$| and not $part;
>> 	if ($s[1] =~ m|/boot$|) {
>>
>>Вообще, там есть что по части перла зафиксить.
>>Или на shell'е переписать.
> 
> 
> В mdk юмористы (сравниваю с новой версией):
> 
> @@ -154,7 +187,7 @@ sub get_boot_partitions {
>      local *F;
>      open F, '/etc/fstab';
>      while (<F>) {
> -       next if /^#/;
> +       s/#.*//;
>         my @s = split ' ';
>         $part = $s[0] if $s[1] =~ m|/$| and not $part;
>         if ($s[1] =~ m|/boot$|) {
> 
> hackish

Мда... мне кажется, что я даже знаю - зачем это они сделали ... ;-)

Иначе им будет тяжело выводить об ошибке парсинга в какой-то 
определенной строке fstab.

Т.е. - я бы это реализовал например так (C++)

ifstream fd("/etc/fstab");
char line[512];
unsigned int linenumber = 0;

while(fd.getline(line, sizeof(line), '\n'))
{
    ++linenumber;
    if(line[0] == "#")
	continue;
     parse(line,linenumber);
}