Re: minor TMPDIR fix

From: Sam Ravnborg
Date: Wed Sep 03 2003 - 15:30:00 EST


On Wed, Sep 03, 2003 at 12:27:26PM -0700, carbonated beverage wrote:
> Hi,
>
> Just a small fix to make the makeman script use $TMPDIR from the
> environment if it's set.
Michael Still contributed this script - I have added him in to:

Sam

> Sent it to the listed maintainer, mec, but he said he's not active
> in the kernel development anymore. I'll send out a patch to update the
> MAINTAINERS file if anyone can tell me who it should be set to.
>
> -- DN
> Daniel
>
> --- 1.2/scripts/makeman Thu Aug 14 18:17:42 2003
> +++ edited/scripts/makeman Wed Sep 3 01:51:53 2003
> @@ -12,7 +12,7 @@
> ## $3 -- the filename which contained the sgmldoc output
> ## (I need this so I know which manpages to convert)
>
> -my($LISTING, $GENERATED, $INPUT, $OUTPUT, $front, $mode, $filename);
> +my($LISTING, $GENERATED, $INPUT, $OUTPUT, $front, $mode, $filename, $tmpdir);
>
> if($ARGV[0] eq ""){
> die "Usage: makeman [convert | install] <dir> <file>\n";
> @@ -132,9 +132,13 @@
> }
> }
> close INPUT;
> -
> - system("cd $ARGV[1]; docbook2man $filename.sgml; mv $filename.9 /tmp/$$.9\n");
> - open GENERATED, "< /tmp/$$.9";
> + if($ENV{'TMPDIR'}) {
> + $tmpdir=$ENV{'TMPDIR'};
> + } else {
> + $tmpdir="/tmp";
> + }
> + system("cd $ARGV[1]; docbook2man $filename.sgml; mv $filename.9 $tmpdir/$$.9\n");
> + open GENERATED, "< $tmpdir/$$.9";
> open OUTPUT, "> $ARGV[1]/$filename.9";
>
> print OUTPUT "$front";
> @@ -146,7 +150,7 @@
> close GENERATED;
>
> system("gzip -f $ARGV[1]/$filename.9\n");
> - unlink("/tmp/$filename.9");
> + unlink("$tmpdir/$filename.9");
> }
> }
> elsif($ARGV[0] eq "install"){
>
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/