gpm: creates wrong `append=' line

Bug #7567 reported by Debian Bug Importer
4
Affects Status Importance Assigned to Milestone
gpm (Debian)
Fix Released
Unknown
gpm (Ubuntu)
Invalid
Low
Unassigned

Bug Description

Automatically imported from Debian bug report #267478 http://bugs.debian.org/267478

Revision history for this message
Debian Bug Importer (debzilla) wrote :

Automatically imported from Debian bug report #267478 http://bugs.debian.org/267478

Revision history for this message
Debian Bug Importer (debzilla) wrote :

Message-ID: <email address hidden>
Date: Sun, 22 Aug 2004 23:00:16 +0200
From: "Mario 'BitKoenig' Holbe" <email address hidden>
To: <email address hidden>
Subject: gpm: creates wrong `append=' line

Package: gpm
Version: 1.19.6-16
Severity: grave

Hello,

the new gpm package generates a wrong escaped config-file:

root@darkside:/etc# diff gpm.conf gpm.conf.dpkg-dist
...
18c18
< append="-l \"a-zA-Z0-9_.:~/\300-\326\330-\366\370-\377\""
---
> append="-l "a-zA-Z0-9_.:~/\300-\326\330-\366\370-\377""

gpm.conf is the one created by 1.19.6-14
gpm.conf.dpkg-dist is the one created by 1.19.6-16

regards,
   Mario
--
It is practically impossible to teach good programming style to students
that have had prior exposure to BASIC: as potential programmers they are
mentally mutilated beyond hope of regeneration. -- Dijkstra

Revision history for this message
In , Mario 'BitKoenig' Holbe (mario-holbe) wrote :

On Sun, Aug 22, 2004 at 11:00:16PM +0200, Mario 'BitKoenig' Holbe wrote:
> < append="-l \"a-zA-Z0-9_.:~/\300-\326\330-\366\370-\377\""

ah, if this is of interest: the line above is the same as the
line in gpm.conf before 1.19.6-14 (I guess this was the first
version that uses debconf). So it was correctly written back
by 1.19.6-14.
The line has been imported to debconf as follows (I just
checked and accepted but not modified the values presented
by 1.19.6-14):

Name: gpm/append
Template: gpm/append
Value: -l "a-zA-Z0-9_.:~/\300-\326\330-\366\370-\377"
Owners: gpm
Flags: seen

regards,
   Mario
--
> As Luke Leighton said once on samba-ntdom, "now, what was that about
> rebooting? that was so long ago, i had to look it up with man -k."

Revision history for this message
Debian Bug Importer (debzilla) wrote :

Message-ID: <email address hidden>
Date: Mon, 23 Aug 2004 09:46:30 +0200
From: "Mario 'BitKoenig' Holbe" <email address hidden>
To: <email address hidden>
Subject: Re: gpm: creates wrong `append=' line

On Sun, Aug 22, 2004 at 11:00:16PM +0200, Mario 'BitKoenig' Holbe wrote:
> < append="-l \"a-zA-Z0-9_.:~/\300-\326\330-\366\370-\377\""

ah, if this is of interest: the line above is the same as the
line in gpm.conf before 1.19.6-14 (I guess this was the first
version that uses debconf). So it was correctly written back
by 1.19.6-14.
The line has been imported to debconf as follows (I just
checked and accepted but not modified the values presented
by 1.19.6-14):

Name: gpm/append
Template: gpm/append
Value: -l "a-zA-Z0-9_.:~/\300-\326\330-\366\370-\377"
Owners: gpm
Flags: seen

regards,
   Mario
--
> As Luke Leighton said once on samba-ntdom, "now, what was that about
> rebooting? that was so long ago, i had to look it up with man -k."

Revision history for this message
Matt Zimmerman (mdz) wrote :

gpm (the daemon) isn't in main, so this bug is not RC for Warty

Revision history for this message
In , Peter Samuelson (peter-p12n) wrote :

[Mario 'BitKoenig' Holbe]
> < append="-l \"a-zA-Z0-9_.:~/\300-\326\330-\366\370-\377\""
> ---
> > append="-l "a-zA-Z0-9_.:~/\300-\326\330-\366\370-\377""

Found the bug, working on a fix.
Thanks very much!

Peter

Revision history for this message
Debian Bug Importer (debzilla) wrote :

Message-ID: <email address hidden>
Date: Tue, 24 Aug 2004 23:59:55 -0500
From: Peter Samuelson <email address hidden>
To: Mario 'BitKoenig' Holbe <email address hidden>,
 <email address hidden>
Subject: Re: gpm: creates wrong `append=' line

--Tln/wzp9jsNjmSUr
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

[Mario 'BitKoenig' Holbe]
> < append="-l \"a-zA-Z0-9_.:~/\300-\326\330-\366\370-\377\""
> ---
> > append="-l "a-zA-Z0-9_.:~/\300-\326\330-\366\370-\377""

Found the bug, working on a fix.
Thanks very much!

Peter

--Tln/wzp9jsNjmSUr
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: Digital signature
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.5 (GNU/Linux)

iD8DBQFBLBzLXk7sIRPQRh0RAhamAKC1q+NpI/KOEu9RIxOtsBzS5rdc/wCgx+mp
WkPYjuNbfzjA6eokjBeuFmI=
=xa/W
-----END PGP SIGNATURE-----

--Tln/wzp9jsNjmSUr--

Revision history for this message
In , Peter Samuelson (peter-p12n) wrote : tags

# fixed in svn
tags 267478 pending
tags 267549 pending

Revision history for this message
Debian Bug Importer (debzilla) wrote :

Message-ID: <email address hidden>
Date: Wed, 25 Aug 2004 04:18:53 -0500
From: Peter Samuelson <email address hidden>
To: <email address hidden>
Subject: tags

# fixed in svn
tags 267478 pending
tags 267549 pending

Revision history for this message
In , Guillem Jover (guillem) wrote : Bug#267478: fixed in gpm 1.19.6-17

Source: gpm
Source-Version: 1.19.6-17

We believe that the bug you reported is fixed in the latest version of
gpm, which is due to be installed in the Debian FTP archive:

gpm_1.19.6-17.diff.gz
  to pool/main/g/gpm/gpm_1.19.6-17.diff.gz
gpm_1.19.6-17.dsc
  to pool/main/g/gpm/gpm_1.19.6-17.dsc
gpm_1.19.6-17_i386.deb
  to pool/main/g/gpm/gpm_1.19.6-17_i386.deb
libgpmg1-dev_1.19.6-17_i386.deb
  to pool/main/g/gpm/libgpmg1-dev_1.19.6-17_i386.deb
libgpmg1_1.19.6-17_i386.deb
  to pool/main/g/gpm/libgpmg1_1.19.6-17_i386.deb

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed. If you
have further comments please address them to <email address hidden>,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Guillem Jover <email address hidden> (supplier of updated gpm package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing <email address hidden>)

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.7
Date: Thu, 26 Aug 2004 04:58:36 +0200
Source: gpm
Binary: libgpmg1 gpm libgpmg1-dev
Architecture: source i386
Version: 1.19.6-17
Distribution: unstable
Urgency: medium
Maintainer: Debian GPM Team <email address hidden>
Changed-By: Guillem Jover <email address hidden>
Description:
 gpm - General Purpose Mouse Interface
 libgpmg1 - General Purpose Mouse - shared library
 libgpmg1-dev - General Purpose Mouse - development files
Closes: 267478 267549
Changes:
 gpm (1.19.6-17) unstable; urgency=medium
 .
   [ Peter Samuelson ]
   * Add po/da.po from Claus Hindsgaul. (Closes: #267549)
   * Fix shell quoting on 'append'. It would be ugly to do it as it was
     done before, AFAICT, but the new version should work like the old one,
     at least. (Closes: #267478)
   * Fix init script bug which has been present since hamm:
     the append= in gpm.conf uses a quoted string of args which themselves
     might be quoted and need to be unquoted. "eval" does this. The
     practical effect is that systems installed from hamm or slink will no
     longer consider " to be a word character. It was never intended to be
     one, but has been for 6 years.
Files:
 38c6f3fee12f14b73a15895e94f3216a 808 misc optional gpm_1.19.6-17.dsc
 09eb01d346ed4d66eaa677726fc044d0 53811 misc optional gpm_1.19.6-17.diff.gz
 bb7fd38883455484e42317fc2e960a6b 189696 misc optional gpm_1.19.6-17_i386.deb
 c2d1397b866dff46666232a3cdb88429 48640 libs standard libgpmg1_1.19.6-17_i386.deb
 0ce087714dfb4c708d6fe89fdef81878 50910 libdevel optional libgpmg1-dev_1.19.6-17_i386.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.5 (GNU/Linux)

iD8DBQFBLVL1uW9ciZ2SjJsRAq0HAKCNH0n31D/neXCIfYhriVtNhk23UACgj3ku
/wLhP51wOGkoNHz5SLs3McI=
=yHuB
-----END PGP SIGNATURE-----

Revision history for this message
Debian Bug Importer (debzilla) wrote :

Message-Id: <email address hidden>
Date: Wed, 25 Aug 2004 23:17:13 -0400
From: Guillem Jover <email address hidden>
To: <email address hidden>
Subject: Bug#267478: fixed in gpm 1.19.6-17

Source: gpm
Source-Version: 1.19.6-17

We believe that the bug you reported is fixed in the latest version of
gpm, which is due to be installed in the Debian FTP archive:

gpm_1.19.6-17.diff.gz
  to pool/main/g/gpm/gpm_1.19.6-17.diff.gz
gpm_1.19.6-17.dsc
  to pool/main/g/gpm/gpm_1.19.6-17.dsc
gpm_1.19.6-17_i386.deb
  to pool/main/g/gpm/gpm_1.19.6-17_i386.deb
libgpmg1-dev_1.19.6-17_i386.deb
  to pool/main/g/gpm/libgpmg1-dev_1.19.6-17_i386.deb
libgpmg1_1.19.6-17_i386.deb
  to pool/main/g/gpm/libgpmg1_1.19.6-17_i386.deb

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed. If you
have further comments please address them to <email address hidden>,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Guillem Jover <email address hidden> (supplier of updated gpm package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing <email address hidden>)

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.7
Date: Thu, 26 Aug 2004 04:58:36 +0200
Source: gpm
Binary: libgpmg1 gpm libgpmg1-dev
Architecture: source i386
Version: 1.19.6-17
Distribution: unstable
Urgency: medium
Maintainer: Debian GPM Team <email address hidden>
Changed-By: Guillem Jover <email address hidden>
Description:
 gpm - General Purpose Mouse Interface
 libgpmg1 - General Purpose Mouse - shared library
 libgpmg1-dev - General Purpose Mouse - development files
Closes: 267478 267549
Changes:
 gpm (1.19.6-17) unstable; urgency=medium
 .
   [ Peter Samuelson ]
   * Add po/da.po from Claus Hindsgaul. (Closes: #267549)
   * Fix shell quoting on 'append'. It would be ugly to do it as it was
     done before, AFAICT, but the new version should work like the old one,
     at least. (Closes: #267478)
   * Fix init script bug which has been present since hamm:
     the append= in gpm.conf uses a quoted string of args which themselves
     might be quoted and need to be unquoted. "eval" does this. The
     practical effect is that systems installed from hamm or slink will no
     longer consider " to be a word character. It was never intended to be
     one, but has been for 6 years.
Files:
 38c6f3fee12f14b73a15895e94f3216a 808 misc optional gpm_1.19.6-17.dsc
 09eb01d346ed4d66eaa677726fc044d0 53811 misc optional gpm_1.19.6-17.diff.gz
 bb7fd38883455484e42317fc2e960a6b 189696 misc optional gpm_1.19.6-17_i386.deb
 c2d1397b866dff46666232a3cdb88429 48640 libs standard libgpmg1_1.19.6-17_i386.deb
 0ce087714dfb4c708d6fe89fdef81878 50910 libdevel optional libgpmg1-dev_1.19.6-17_i386.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.5 (GNU/Linux)

iD8DBQFBLVL1uW9ciZ2SjJsRAq0HAKCNH0n31D/neXCIfYhriVtNhk23UACgj3ku
/wLhP51wOGkoNHz5SLs3McI=
=yHuB
-----END PGP SIGNATURE-----

Revision history for this message
Thom May (thombot) wrote :

Doesn't affect us anyway, we still have the ancient version of gpm.

Changed in gpm:
status: Unknown → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.