Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DEV-1232: unbreak ingest_status.pl #129

Merged
merged 1 commit into from
Jun 13, 2024
Merged

DEV-1232: unbreak ingest_status.pl #129

merged 1 commit into from
Jun 13, 2024

Conversation

mwarin
Copy link
Contributor

@mwarin mwarin commented Jun 13, 2024

I can't quickly tell what's wrong with the tidy version, so reverting to untidy version, which is at least error free.

Tidy version, broken:

$ git checkout b92716d -- bin/ingest_status.pl
...
$ perl -c bin/ingest_status.pl 
syntax error at bin/ingest_status.pl line 40, near ") and"
Can't redeclare "my" in "my" at bin/ingest_status.pl line 51, near "my"
Global symbol "@volumes" requires explicit package name (did you forget to declare "my @volumes"?) at bin/ingest_status.pl line 54.
Global symbol "@volumes" requires explicit package name (did you forget to declare "my @volumes"?) at bin/ingest_status.pl line 68.
Global symbol "@volumes" requires explicit package name (did you forget to declare "my @volumes"?) at bin/ingest_status.pl line 81.
Global symbol "@volumes" requires explicit package name (did you forget to declare "my @volumes"?) at bin/ingest_status.pl line 104.
bin/ingest_status.pl had compilation errors.

Previous, untidy version, works:

$ git checkout 042f52f -- bin/ingest_status.pl
...
$ perl -c bin/ingest_status.pl 
bin/ingest_status.pl syntax OK

… to untidy version, which is at least error free
@mwarin mwarin merged commit 27c73fd into main Jun 13, 2024
1 check passed
@mwarin mwarin deleted the DEV-1232-ingest-status branch June 13, 2024 21:32
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.

None yet

1 participant