Experimental Physics and
Industrial Control System

Andrew Johnson <[email protected]> · Mon, 13 Nov 2017 10:14:13 -0600

Note that variable should be spelled $(MSI3_15)

- Andrew


On 11/10/2017 07:53 PM, Bruce Hill wrote:
> Hi Kristi,
> 
> In addition to Ben's suggestions, for R3.15.5, you'll also need to
> change $(MSI) to $(MSI_3_15) in your rules.
> 
> If that doesn't get it working, send me the path to your build top and
> I'll look into it further.
> 
> Cheers,
> 
> - Bruce
> 
> 
> On 11/10/2017 04:38 AM, Benjamin Franksen wrote:
>> [sorry for possible duplicates, forgot to reply to the list]
>>
>> On 11/09/2017 11:59 PM, Luchini, Kristi L. wrote:
>>> However, my RULES file from R3.14.12, now either generate errors
>>> during the build or it will generate an empty alhConfig which it
>>> installs in to $TOP/alh.
>>> Is there an example of a RULES_<file type> that I can used. Below is
>>> what I am using, which clearly is wrong.
>>>
>>> File configure/Makefile
>>> CHECK_RELEASE = YES
>>> TARGETS = $(CONFIG_TARGETS)
>>> CONFIGS += $(subst ../,,$(wildcard $(CONFIG_INSTALLS)))
>>> CFG +=CONFIG_ALARMS RULES_ALARMS
>>> CFG +=CONFIG_PROTOS
>>> include $(TOP)/configure/RULES
>>>
>>> File: configure/CONFIG_ALARMS
>>> FILE_TYPE+=ALHS
>>> INSTALL_ALHS = $(INSTALL_LOCATION)/alh
>> This is no longer needed as there are now a similar definitions in base,
>> which use ALH, not ALHS. You can keep you won FILE_TYPE, though, since
>> due to the different name there is no conflict.
>>
>>> File:  configure/RULES_ALARMS    --- similar to R3.14.12   RULES.Db
>> This is a mistake. RULES.Db is an overly complicated monster. I wish
>> there was a way to completely disable it.
>>
>> All you need is the vpaths:
>>
>>> vpath %.alhConfig $(USR_VPATH) $(GENERIC_SRC_DIRS) $(dir $(ALH))
>>> vpath %.alhsubstitutions $(USR_VPATH) $(GENERIC_SRC_DIRS) $(COMMON_DIR)
>> however, these are not good. I would rather use
>>
>> vpath %.alhConfig $(USR_VPATH) $(GENERIC_SRC_DIRS) $(COMMON_DIR)
>> vpath %.alhsubstitutions $(USR_VPATH) $(GENERIC_SRC_DIRS)
>>
>> (Presumably alhsubstitutions are sources, while alhConfig may be sources
>> or produced by the rule below.)
>>
>> The only rule you need is
>>
>>> $(COMMON_DIR)/%.alhConfig: %.alhsubstitutions
>>>          @echo "Inflating alhConfig from $< using local rule"
>>>          @$(RM) $@
>>>          @echo "$(MSI) -S$< > $(@F) </dev/null"
>>>          @$(MSI) -I.. $(ALHFLAGS) -S$< </dev/null > $@
>> which I would re-write as
>>
>> $(COMMON_DIR)/%.alhConfig: %.alhsubstitutions
>>     $(ECHO) "Inflating alhConfig from $< using local rule"
>>     $(MSI) -I.. $(ALHFLAGS) -S$< > [email protected]
>>     $(MV) [email protected] $@
>>
>> (Why suppress echoing of the command line and then echo manually in a
>> separate command, just so you don't see the $(ALHFLAGS) parameter? The
>> "output to .tmp, then mv" pattern is to make parallel make more
>> reliable.)
>>
>> Also, the include of RULES_ALARMS below
>>
>>> File:  Makefile:
>>> TOP=../../..
>>> include $(TOP)/configure/CONFIG
>>> ALHS += ntwk_inj.alhConfig
>>> include $(TOP)/configure/RULES
>>> include $(TOP)/configure/RULES_ALARMS
>> is automatically done by the base rules.
>>
>> Cheers
>> Ben
> 

-- 
Arguing for surveillance because you have nothing to hide is no
different than making the claim, "I don't care about freedom of
speech because I have nothing to say." -- Edward Snowdon

Experimental Physics and Industrial Control System

Experimental Physics and
Industrial Control System