search

Xilinx / PYNQ

Python Productivity for ZYNQ
http://www.pynq.io/
BSD 3-Clause "New" or "Revised" License
1.98k stars 816 forks source link

It really looks like these were typos and thus dependencies are missed. #1440

Closed keithpenney closed 5 months ago

keithpenney commented 6 months ago

To be honest, I didn't do much more than just:

grep -r --include="*akefile" --include="*.mk" --include="*.sh" --include="*.cmake" EMBEDDED_DIR
# exactly two hits; both empty dependencies with no associated targets

grep -r --include="*akefile" --include="*.mk" --include="*.sh" --include="*.cmake" EMBEDDEDSW_DIR
# 23 hits including defining the variable and an associated recipe

I won't pretend to understand the full build system, so it's entirely possible that this is actually intentional and there's some meta-scripting magic making this all work. Looks like a typo though.