Skip to content

fix: Correct variable name#25147

Open
perlpunk wants to merge 1 commit intoos-autoinst:masterfrom
perlpunk:fix-openssl_nodejs
Open

fix: Correct variable name#25147
perlpunk wants to merge 1 commit intoos-autoinst:masterfrom
perlpunk:fix-openssl_nodejs

Conversation

@perlpunk
Copy link
Copy Markdown
Contributor

This error is shown when strict is enabled:

Global symbol "$incident_repo" requires explicit package name (did you
forget to declare "my $incident_repo"?) at
./tests/console/openssl_nodejs.pm line 44.

Since in that branch the variable $incident_repo was never defined, this can never have been working as expected.

Also it might be easier to just do:

$increment_repo =~ s{/$}{};

This error is shown when strict is enabled:

    Global symbol "$incident_repo" requires explicit package name (did you
    forget to declare "my $incident_repo"?) at
    ./tests/console/openssl_nodejs.pm line 44.

Since in that branch the variable `$incident_repo` was never defined,
this can never have been working as expected.

Also it might be easier to just do:

    $increment_repo =~ s{/$}{};
@perlpunk perlpunk force-pushed the fix-openssl_nodejs branch from da78646 to cc2f8d4 Compare March 30, 2026 18:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant