Skip to content

Commit c1c6770

Browse files
committed
fix(docker): exclude install dir from LICENSE find and use IFS= read -r
Prevent find from re-discovering already-copied license files in the install destination, and handle paths with spaces correctly. Signed-off-by: Jack Luar <jluar@precisioninno.com>
1 parent de23328 commit c1c6770

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

docker/Dockerfile.builder

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,8 @@ EOF
5151
RUN find /OpenROAD-flow-scripts/tools \( -name "*LICENSE*" -o -name "*LICENSES*" \) \
5252
| grep -v '/OpenROAD/src/sta/' \
5353
| grep -v '/AutoTuner/' \
54-
| while read -r f; do \
54+
| grep -v '^/OpenROAD-flow-scripts/tools/install/' \
55+
| while IFS= read -r f; do \
5556
rel="${f#/OpenROAD-flow-scripts/tools/}"; \
5657
mkdir -p "/OpenROAD-flow-scripts/tools/install/licenses/$(dirname "$rel")"; \
5758
cp -r "$f" "/OpenROAD-flow-scripts/tools/install/licenses/$rel"; \

0 commit comments

Comments
 (0)