RE: undefine variable in RELEASE.local

["Pearson, Matthew R. via Tech-talk" <tech-talk at aps.anl.gov>]

Hi Andrew,

 

That patch does indeed fix it, thanks!

 

I should have mentioned that I was using base 7.0.6, and I didn’t realize I was using a new feature from 7.0.5, and I wasn’t too familiar with how the RELEASE files are parsed, but it makes sense now.

 

Cheers,

Matt

 

From: Johnson, Andrew N. <anj at anl.gov>
Sent: Tuesday, October 26, 2021 4:22 PM
To: Pearson, Matthew R. <pearsonmr at ornl.gov>
Cc: EPICS tech-talk <tech-talk at aps.anl.gov>
Subject: [EXTERNAL] Re: undefine variable in RELEASE.local

 

Hi Matt,

 

On Oct 26, 2021, at 9:26 AM, Pearson, Matthew R. via Tech-talk <tech-talk at aps.anl.gov> wrote:

 

undefine SNCSEQ
EPICS_BASE=/home/controls/common/base/main

Although this causes errors like these when building asyn:

make[2]: Entering directory '/home/controls/common/asyn/main/iocBoot/ioctest'
perl -CSD /home/controls/common/base/main/bin/linux-x86_64/convertRelease.pl -t /home/controls/common/asyn/main envPaths
Use of uninitialized value $iocpath in substitution (s///) at /home/controls/common/base/main/bin/linux-x86_64/convertRelease.pl line 220.
Use of uninitialized value $path in -d at /home/controls/common/base/main/bin/linux-x86_64/convertRelease.pl line 221.

 

Oops! This patch to Base should fix the problem, please try it:

 

diff --git a/src/tools/EPICS/Release.pm b/src/tools/EPICS/Release.pm

index 7caa6f4c6..e6aaf1e0c 100644

--- a/src/tools/EPICS/Release.pm

+++ b/src/tools/EPICS/Release.pm

@@ -67,6 +67,7 @@ sub readRelease {

         my ($uvar) = m/^ undefine \s+ ($MVAR)/x;

         if ($uvar ne '') {

             delete $Rmacros->{$uvar};

+            @$Rapps = grep($_ ne $uvar, @$Rapps);

             next;

         }

 

 

Do you know what's causing these errors? I suppose the script just isn't silently dealing with the 'undefine' syntax? 

 

Yup, and I evidently didn’t test the code properly when I added support for undefine (in Base 7.0.5 I believe).

 

Thanks,

 

- Andrew

 

-- 

Complexity comes for free, simplicity you have to work for.