Skip to content

Commit

Permalink
Pass the --tempdir option from lcov to geninfo (#379)
Browse files Browse the repository at this point in the history
lcov invokes geninfo but didn't provide it with the user's --tempdir
option value (if specified); pass it down.

Signed-off-by: Paul Smith <[email protected]>
  • Loading branch information
madscientist authored Jan 9, 2025
1 parent 7959556 commit 39e7323
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
3 changes: 2 additions & 1 deletion bin/lcov
Original file line number Diff line number Diff line change
Expand Up @@ -916,7 +916,8 @@ sub lcov_geninfo(@)
push(@param, "--mcdc") if $lcovutil::mcdc_coverage;
push(@param, '--fail-under-lines', $lcovutil::fail_under_lines)
if defined($lcovutil::fail_under_lines);

push(@param, '--tempdir', $lcovutil::tempdirname)
if (defined($lcovutil::tempdirname));
foreach my $listOpt (['--comment', \@lcovutil::comments],
['--config-file', \@lcovutil::opt_config_files],
['--rc', \@lcovutil::opt_rc],
Expand Down
2 changes: 1 addition & 1 deletion lib/lcovutil.pm
Original file line number Diff line number Diff line change
Expand Up @@ -1071,7 +1071,7 @@ my (@rc_filter, @rc_ignore, @rc_exclude_patterns,
$rc_no_branch_coverage, $rc_no_func_coverage, $rc_no_checksum,
$version);
my $quiet = 0;
my $tempdirname;
our $tempdirname;

# these options used only by lcov - but moved here so that we can
# share arg parsing
Expand Down

0 comments on commit 39e7323

Please sign in to comment.