Fix input files list emission to avoid bash "too many arguments" error

This makes the expansion of "cat $(VLSI_RTL)" happen as a child of the shell that runs the for loop.

The existing version will sometimes produce a bash "too many arguments" error because the $(shell cat $(VLSI_RTL)) is expanded first and then passed as a giant command to bash.
This commit is contained in:
Sagar Karandikar
2023-02-15 17:57:26 -08:00
committed by GitHub
parent df903f0cb6
commit 0c4cfc8742

View File

@@ -122,7 +122,7 @@ $(SYN_CONF): $(VLSI_RTL)
echo "synthesis.inputs:" >> $@
echo " top_module: $(VLSI_TOP)" >> $@
echo " input_files:" >> $@
for x in $(shell cat $(VLSI_RTL)); do \
for x in $$(cat $(VLSI_RTL)); do \
echo ' - "'$$x'"' >> $@; \
done