Bug#391055: -d for debugging requires argument

2006-11-03 Thread Frederik Dannemare
On Wednesday 04 October 2006 17:44, Nico Golde wrote:
 Package: motion
 Version: 3.2.3-1+b1
 Severity: normal
 Tags: patch
 Hi,
 If you call motion with -d for debugging it complains about
 amissing parameter. Referring to the manpage it does not
 need a parameter.
 A fix would be:
 Line 1107 of conf.c
 - while ((c=getopt(conf-argc, conf-argv, c:d:hns?))!=EOF)
 + while ((c=getopt(conf-argc, conf-argv, c:dhns?))!=EOF)

Just tried this but got a segfault trying to run 'motion -d'. Running 
motion with using -d works fine, though.

I have just finished a new build which I would like to see in testing as 
soon as possible, so with this next release I will merely make a 
comment in the manpage that using -d also requires -c.

Best regards,
Frederik Dannemare


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



Bug#391055: -d for debugging requires argument

2006-11-03 Thread Nico Golde
Hi,
* Frederik Dannemare [EMAIL PROTECTED] [2006-11-03 15:16]:
 On Wednesday 04 October 2006 17:44, Nico Golde wrote:
  Package: motion
  Version: 3.2.3-1+b1
  Severity: normal
  Tags: patch
  Hi,
  If you call motion with -d for debugging it complains about
  amissing parameter. Referring to the manpage it does not
  need a parameter.
  A fix would be:
  Line 1107 of conf.c
  - while ((c=getopt(conf-argc, conf-argv, c:d:hns?))!=EOF)
  + while ((c=getopt(conf-argc, conf-argv, c:dhns?))!=EOF)
 
 Just tried this but got a segfault trying to run 'motion -d'. Running 
 motion with using -d works fine, though.

Sure if the code says something different than the manpage 
it will not work. My point was that the manpage says (and 
you call it like this also) without a parameter, just -d but
the code says it has to be called with a parameter -d 
parameter. Thats why its 'd:' not 'd' in getopt.

Kind regards
Nico
-- 
Nico Golde - http://www.ngolde.de
JAB: [EMAIL PROTECTED] - GPG: 0x73647CFF
Forget about that mouse with 3/4/5 buttons,
gimme a keyboard with 103/104/105 keys!


pgpuse7PIcphZ.pgp
Description: PGP signature


Bug#391055: -d for debugging requires argument

2006-10-04 Thread Nico Golde
Package: motion
Version: 3.2.3-1+b1
Severity: normal
Tags: patch
Hi,
If you call motion with -d for debugging it complains about 
amissing parameter. Referring to the manpage it does not 
need a parameter.
A fix would be:
Line 1107 of conf.c
- while ((c=getopt(conf-argc, conf-argv, c:d:hns?))!=EOF)
+ while ((c=getopt(conf-argc, conf-argv, c:dhns?))!=EOF)

Kind regards
Nico


-- System Information:
Debian Release: unstable
  APT prefers unstable
  APT policy: (500, 'unstable')
Architecture: i386 (i686)
Shell:  /bin/sh linked to /bin/bash
Kernel: Linux 2.6.16-2-686
Locale: [EMAIL PROTECTED], [EMAIL PROTECTED] (charmap=ISO-8859-15)

-- 
Nico Golde - http://www.ngolde.de
JAB: [EMAIL PROTECTED] - GPG: 0x73647CFF
Forget about that mouse with 3/4/5 buttons,
gimme a keyboard with 103/104/105 keys!


signature.asc
Description: Digital signature