From: Mikhail Zabaluev Date: Fri, 12 Oct 2001 12:47:47 +0000 (+0400) Subject: (replaced with #12600) X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=commitdiff_plain;h=6a43d2f9dcd01889e3e3e78ab5de0a2ee646713a;p=p5sagit%2Fp5-mst-13.2.git (replaced with #12600) Subject: [PATCH 5.6.1 perldoc] use File::Temp Message-Id: <20011012124747.E13918@localhost.localdomain> p4raw-id: //depot/perl@12406 --- diff --git a/utils/perldoc.PL b/utils/perldoc.PL index cfb773e..22fdd1c 100644 --- a/utils/perldoc.PL +++ b/utils/perldoc.PL @@ -360,50 +360,6 @@ sub filter_nroff { join "\n\n", @data; } -sub printout { - my ($file, $tmp, $filter) = @_; - my $err; - - if ($opt_t) { - # why was this append? - sysopen(OUT, $tmp, O_WRONLY | O_EXCL | O_CREAT, 0600) - or die ("Can't open $tmp: $!"); - Pod::Text->new()->parse_from_file($file,\*OUT); - close OUT or die "can't close $tmp: $!"; - } - elsif (not $opt_u) { - my $cmd = catfile($bindir, 'pod2man') . " --lax $file | $opt_n -man"; - $cmd .= " | col -x" if $^O =~ /hpux/; - my $rslt = `$cmd`; - $rslt = filter_nroff($rslt) if $filter; - unless (($err = $?)) { - # why was this append? - sysopen(TMP, $tmp, O_WRONLY | O_EXCL | O_CREAT, 0600) - or die "Can't open $tmp: $!"; - print TMP $rslt - or die "Can't print $tmp: $!"; - close TMP - or die "Can't close $tmp: $!"; - } - } - if ($opt_u or $err or -z $tmp) { # XXX: race with -z - # why was this append? - sysopen(OUT, $tmp, O_WRONLY | O_EXCL | O_CREAT, 0600) - or die "Can't open $tmp: $!"; - open(IN,"<", $file) or die("Can't open $file: $!"); - my $cut = 1; - local $_; - while () { - $cut = $1 eq 'cut' if /^=(\w+)/; - next if $cut; - print OUT - or die "Can't print $tmp: $!"; - } - close IN or die "Can't close $file: $!"; - close OUT or die "Can't close $tmp: $!"; - } -} - sub page { my ($tmp, $no_tty, @pagers) = @_; if ($no_tty) { @@ -428,6 +384,7 @@ sub page { sub cleanup { my @files = @_; for (@files) { + next unless defined; if ($Is_VMS) { 1 while unlink($_); # XXX: expect failure } else { @@ -450,14 +407,14 @@ foreach (@pages) { next; } print STDERR "Searching for $_\n" if $opt_v; - # We must look both in @INC for library modules and in $bindir - # for executables, like h2xs or perldoc itself. - my @searchdirs = ($bindir, @INC); if ($opt_F) { next unless -r; push @found, $_ if $opt_m or containspod($_); next; } + # We must look both in @INC for library modules and in $bindir + # for executables, like h2xs or perldoc itself. + my @searchdirs = ($bindir, @INC); unless ($opt_m) { if ($Is_VMS) { my($i,$trn); @@ -516,61 +473,27 @@ my $no_tty; if (! -t STDOUT) { $no_tty = 1 } END { close(STDOUT) || die "Can't close STDOUT: $!" } -# until here we could simply exit or die -# now we create temporary files that we have to clean up -# namely $tmp, $buffer -# that's because you did it wrong, should be descriptor based --tchrist - -my $tmp; -my $buffer; if ($Is_MSWin32) { - $tmp = "$ENV{TEMP}\\perldoc1.$$"; - $buffer = "$ENV{TEMP}\\perldoc1.b$$"; push @pagers, qw( more< less notepad ); unshift @pagers, $ENV{PAGER} if $ENV{PAGER}; for (@found) { s,/,\\,g } } elsif ($Is_VMS) { - $tmp = 'Sys$Scratch:perldoc.tmp1_'.$$; - $buffer = 'Sys$Scratch:perldoc.tmp1_b'.$$; push @pagers, qw( most more less type/page ); } elsif ($Is_Dos) { - $tmp = "$ENV{TEMP}/perldoc1.$$"; - $buffer = "$ENV{TEMP}/perldoc1.b$$"; - $tmp =~ tr!\\/!//!s; - $buffer =~ tr!\\/!//!s; push @pagers, qw( less.exe more.com< ); unshift @pagers, $ENV{PAGER} if $ENV{PAGER}; } else { if ($^O eq 'os2') { - require POSIX; - $tmp = POSIX::tmpnam(); - $buffer = POSIX::tmpnam(); unshift @pagers, 'less', 'cmd /c more <'; } - else { - # XXX: this is not secure, because it doesn't open it - ($tmp, $buffer) = eval { require POSIX } - ? (POSIX::tmpnam(), POSIX::tmpnam() ) - : ("/tmp/perldoc1.$$", "/tmp/perldoc1.b$$" ); - } push @pagers, qw( more less pg view cat ); unshift @pagers, $ENV{PAGER} if $ENV{PAGER}; } unshift @pagers, $ENV{PERLDOC_PAGER} if $ENV{PERLDOC_PAGER}; -# make sure cleanup called -eval q{ - sub END { cleanup($tmp, $buffer) } - 1; -} || die; - -# exit/die in a windows sighandler is dangerous, so let it do the -# default thing, which is to exit -eval q{ use sigtrap qw(die INT TERM HUP QUIT) } unless $^O eq 'MSWin32'; - if ($opt_m) { foreach my $pager (@pagers) { if (system($pager, @found) == 0) { @@ -658,22 +581,74 @@ EOD } } +# until here we could simply exit or die +# now we create temporary files that we have to clean up +# namely $tmp, $buffer +# that's because you did it wrong, should be descriptor based --tchrist + +my $tmp; +my $tmpfd; +my $buffer; + +require File::Temp; + +($tmpfd, $tmp) = File::Temp::tempfile(); + +# make sure cleanup called +eval q{ + sub END { cleanup($tmp, $buffer) } + 1; +} || die; + +# exit/die in a windows sighandler is dangerous, so let it do the +# default thing, which is to exit +eval q{ use sigtrap qw(die INT TERM HUP QUIT) } unless $^O eq 'MSWin32'; + my $filter; if (@pod) { - sysopen(TMP, $buffer, O_WRONLY | O_EXCL | O_CREAT) - or die("Can't open $buffer: $!"); - print TMP "=over 8\n\n"; - print TMP @pod or die "Can't print $buffer: $!"; - print TMP "=back\n"; - close TMP or die "Can't close $buffer: $!"; + my $buffd; + ($buffd, $buffer) = File::Temp::tempfile(); + print $buffd "=over 8\n\n"; + print $buffd @pod or die "Can't print $buffer: $!"; + print $buffd "=back\n"; + close $buffd or die "Can't close $buffer: $!"; @found = $buffer; $filter = 1; } foreach (@found) { - printout($_, $tmp, $filter); + my $file = $_; + my $err; + + if ($opt_t) { + Pod::Text->new()->parse_from_file($file, $tmpfd); + } + elsif (not $opt_u) { + my $cmd = catfile($bindir, 'pod2man') . " --lax $file | $opt_n -man"; + $cmd .= " | col -x" if $^O =~ /hpux/; + my $rslt = `$cmd`; + $rslt = filter_nroff($rslt) if $filter; + unless (($err = $?)) { + print $tmpfd $rslt + or die "Can't print $tmp: $!"; + } + } + if ($opt_u or $err) { + open(IN,"<", $file) or die("Can't open $file: $!"); + my $cut = 1; + local $_; + while () { + $cut = $1 eq 'cut' if /^=(\w+)/; + next if $cut; + print $tmpfd $_ + or die "Can't print $tmp: $!"; + } + close IN or die "Can't close $file: $!"; + } } +close $tmpfd + or die "Can't close $tmp: $!"; page($tmp, $no_tty, @pagers); exit;