Re: CVS commit: pkgsrc/mk/pkgformat/pkg

[Joerg Sonnenberger <joerg%bec.de@localhost>]

On Sat, Jun 06, 2020 at 09:33:56AM +0000, Roland Illig wrote:
> @@ -105,12 +106,12 @@ ${_BUILD_INFO_FILE}: ${_PLIST_NOKEYWORDS
>       requires=`{ for i in $$requires $$requires; do echo $$i; done; \
>               ${AWK} '{ print "${PREFIX}/" $$0 }' ${_PLIST_NOKEYWORDS}; } | \
>               ${SORT} | uniq -c | awk '$$1 == 2 {print $$2}'`; \
> -     for i in "" $$libs; do                                          \
> +     for i in $$libs; do                                             \
>               ${TEST} "$$i" != "" || continue;                        \
> -             ${ECHO} "PROVIDES=$${i}";                               \
> +             ${ECHO} "PROVIDES=$$i";                                 \
>       done | ${SED} -e 's,^PROVIDES=${DESTDIR},PROVIDES=,'            \
>               >> ${.TARGET}.tmp;                                      \
> -     for req in "" $$requires; do                                    \
> +     for req in $$requires; do                                       \
>               ${TEST} "$$req" != "" || continue;                      \
>               ${ECHO} "REQUIRES=$$req" >> ${.TARGET}.tmp;             \
>       done

This is broken for shells that reject empty for iteration.

Joerg