Skip Menu |

This queue is for tickets about the PAR CPAN distribution.

Report information
The Basics
Id: 5793
Status: resolved
Priority: 0/
Queue: PAR

People
Owner: Nobody in particular
Requestors: david.dyck [...] fluke.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Date: Wed, 24 Mar 2004 16:17:43 -0800 (PST)
From: David Dyck <david.dyck [...] fluke.com>
To: bug-par [...] rt.cpan.org
Subject: par_setenv usage inconsistent
I noticed in myldr/env.c that /* * setenv -- * Set the value of the environmental variable "name" to be * "value". If rewrite is set, replace any current value. */ static int par_setenv(name, value, rewrite) par_setenv is declared with 3 arguments, but the calls to par_setenv in utils.c and mktmpdir.c don't pass the the rewrite argument in. This is probably not the reason that PAR-0.80 doesn't build with perl 5.9.2, but since I heard nothing from the previous email I sent to this list, I have started looking into the problem myself. utils.c:36 par_setenv("PAR_PROGNAME", prog); utils.c:53 par_setenv("PAR_PROGNAME", prog); utils.c:60 par_setenv("PAR_PROGNAME", filename); utils.c:66 par_setenv("PAR_PROGNAME", prog); utils.c:134 par_setenv("PAR_DEBUG", buf); utils.c:138 par_setenv("PAR_TEMP", buf); utils.c:141 par_setenv("PAR_CLEAN", buf); utils.c:146 par_setenv("PAR_CLEAN", buf + strlen("PAR_CLEAN=")); utils.c:150 par_setenv("PAR_INITIALIZED", "1"); mktmpdir.c:113 par_setenv("PAR_CLEAN", "1"); mktmpdir.c:123 par_setenv("PAR_TEMP" , stmpdir); mktmpdir.c:129 par_setenv(key, stmpdir); mktmpdir.c:142 par_setenv(key, ld_path_env);
[david.dyck@fluke.com - Wed Mar 24 19:17:50 2004]: Show quoted text
> I noticed in myldr/env.c that > /* > * setenv -- > * Set the value of the environmental variable "name" to be > * "value". If rewrite is set, replace any current value. > */ > static int > par_setenv(name, value, rewrite) > > par_setenv is declared with 3 arguments, but the calls > to par_setenv in utils.c and mktmpdir.c don't pass > the the rewrite argument in.
Hi there, sorry for the much belated reply. This has since been fixed by Alan Stewarts on April 17th, so there is no extra 'rewrite' now. Can you help testing the snapshot: http://p4.elixus.org/snap/PAR.tar.gz and let me know if it works for you? Thanks, /Autrijus/