Bug#409092: menu: [install-menu] Parser does not handle comments (#) correctly

2007-02-07 Thread Bill Allombert 
On Fri, Jan 26, 2007 at 05:16:50PM +0200, Jari Aalto wrote:
> Package: menu
> Version: 2.1.33
> Severity: normal
> 
> Given following comment in debian/menu-method file
> 
># substitute all q(") with q(\")
> 
> The calls
> 
>update-menus --stdout > /tmp/menu-stdin
>install -m 755 menu-method /etc/menu-methods/mytest
>/etc/menu-methods/mytest -v < /tmp/menu-stdin
> 
> Gives cryptic error:
> 
>install-menu: Warning: Unknown identifier `41' on line 
> /etc/menu-methods/jwm in file . Ignoring.

Hello Jari,

I cannot reproduce this problem from the data alone. Furthermore your
error message is inconsistent: you are launching
/etc/menu-methods/mytest but the error point to /etc/menu-methods/jwm.

Could you send me your file /etc/menu-methods/mytest so I could try
to reproduce the issue ?

Cheers,
-- 
Bill. <[EMAIL PROTECTED]>

Imagine a large blue swirl here. 


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Bug#409092: menu: [install-menu] Parser does not handle comments (#) correctly

2007-02-07 Thread Jari Aalto 
Bill Allombert <[EMAIL PROTECTED]> writes:

> On Fri, Jan 26, 2007 at 05:16:50PM +0200, Jari Aalto wrote:
>> Package: menu
>> Version: 2.1.33
>> Severity: normal
>> 
>> Given following comment in debian/menu-method file
>> 
>># substitute all q(") with q(\")
>> 
>> The calls
>> 
>>update-menus --stdout > /tmp/menu-stdin
>>install -m 755 menu-method /etc/menu-methods/mytest
>>/etc/menu-methods/mytest -v < /tmp/menu-stdin
>> 
>> Gives cryptic error:
>> 
>>install-menu: Warning: Unknown identifier `41' on line 
>> /etc/menu-methods/jwm in file . Ignoring.
>
> Hello Jari,
>
> I cannot reproduce this problem from the data alone. Furthermore your
> error message is inconsistent: you are launching
> /etc/menu-methods/mytest but the error point to /etc/menu-methods/jwm.

It should read "mytest" everywhere. 

> Could you send me your file /etc/menu-methods/mytest so I could try
> to reproduce the issue ?

Doh, I repeated this exactly how I did it many times I sent the bug
and I can't reproduce it either (scrathing my head). Let's close this
and get back to it if I happen to see it again.

Btw, thanks Bill about your bug reports to other packages.

Thanks,
Jari



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Bug#409092: menu: [install-menu] Parser does not handle comments (#) correctly

2007-01-30 Thread Jari Aalto 
Package: menu
Version: 2.1.33
Severity: normal

Given following comment in debian/menu-method file

   # substitute all q(") with q(\")

The calls

   update-menus --stdout > /tmp/menu-stdin
   install -m 755 menu-method /etc/menu-methods/mytest
   /etc/menu-methods/mytest -v < /tmp/menu-stdin

Gives cryptic error:

   install-menu: Warning: Unknown identifier `41' on line /etc/menu-methods/jwm 
in file . Ignoring.

If the comment line is written without the quotes and backslashes:

   # substitute all quotes with backslash+quote

Then no errors are displayed.


-- System Information:
Debian Release: 4.0
  APT prefers unstable
  APT policy: (500, 'unstable'), (500, 'stable')
Architecture: i386 (i686)
Shell:  /bin/sh linked to /bin/dash
Kernel: Linux 2.6.18-3-686
Locale: LANG=C, LC_CTYPE=C (charmap=ISO-8859-1) (ignored: LC_ALL set to en_US)

Versions of packages menu depends on:
ii  dpkg1.13.25  package maintenance system for Deb
ii  libc6   2.3.6.ds1-10 GNU C Library: Shared libraries
ii  libgcc1 1:4.1.1-21   GCC support library
ii  libstdc++6  4.1.1-21 The GNU Standard C++ Library v3

menu recommends no packages.

-- no debconf information


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]