-
Notifications
You must be signed in to change notification settings - Fork 17
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
FCM1: fix config parser lc problem #156
Conversation
It has started to given out a lot of warnings in newer versions of Perl5.
@tomgreen66 Do you think you can test this branch in your environment? |
@matthewrmshin I have just run it through an incremental build of the UM test job (vn8.5) and it still generates the following: 3 Use of uninitialized value in uc at /software/tools/fcm/fix-fcm1-cfg-lc/bin/../lib/FCM1/BuildSrc.pm line 409.
22904 Use of uninitialized value in uc at /software/tools/fcm/fix-fcm1-cfg-lc/bin/../lib/FCM1/CfgFile.pm line 244. Does that help understand the issue - not sure how I missed those if they were being generated with the earlier messages. |
I'll take another look. |
@tomgreen66 please take another look. |
@matthewrmshin - just rerun the test (and also a forced rebuild) and it seems to have fixed the warnings so thats great and a big thanks from me. Is there anything else I can do? |
FCM1: fix config parser lc problem
@tomgreen66 I'll create an FCM release in the not so distant future. |
Configuration logic was broken by metomi#156.
It has started to given out a lot of warnings in newer versions of Perl5.
Fix #153.