Re: pkg/42907 (Errors in REQUIRES/PROVIDES variables in pkg_summary(5))

[Aleksey Cheusov <cheusov%tut.by@localhost>]

> Synopsis: Errors in REQUIRES/PROVIDES variables in pkg_summary(5)

> State-Changed-From-To: open->closed
> State-Changed-By: dholland%NetBSD.org@localhost
> State-Changed-When: Sat, 31 Jul 2010 23:27:32 +0000
> State-Changed-Why:
> reed@ fixed it
> (if I misunderstand and it's not fully fixed, let me know)

No, please reopen it. I sent working patch to Jeremy >2 months ago and
still wait for his response. I tested my patch on several Linux bulk builds
(rebuild all packages) and it worked perfectly.  There are only two
remaining issues: misc/openoffice2-bin and mail/sympa.

   l: not_found /usr/pkg/ mail/sympa sympa-5.4.4nb2

   l: not_found 
/usr/pkg/openoffice.org2.4/program/components/../libmsgbaseutil.so 
misc/openoffice2-bin openoffice2-bin-2.4.3
   l: not_found /usr/pkg/openoffice.org2.4/program/libfreetype.so.6 
misc/openoffice2-bin openoffice2-bin-2.4.3
   l: not_found /usr/pkg/openoffice.org2.4/program/components/../libnss3.so 
misc/openoffice2-bin openoffice2-bin-2.4.3
   l: not_found /usr/pkg/openoffice.org2.4/program/components/../libsmime3.so 
misc/openoffice2-bin openoffice2-bin-2.4.3
   l: not_found /usr/pkg/openoffice.org2.4/program/components/../libmozz.so 
misc/openoffice2-bin openoffice2-bin-2.4.3
   l: not_found /usr/pkg/openoffice.org2.4/program/components/../libgcc_s.so.1 
misc/openoffice2-bin openoffice2-bin-2.4.3
   l: not_found /usr/pkg/openoffice.org2.4/program/components/../libnspr4.so 
misc/openoffice2-bin openoffice2-bin-2.4.3
   l: not_found /usr/pkg/openoffice.org2.4/program/components/../libsoftokn3.so 
misc/openoffice2-bin openoffice2-bin-2.4.3
   l: not_found /usr/pkg/openoffice.org2.4/program/components/../libplc4.so 
misc/openoffice2-bin openoffice2-bin-2.4.3
   l: not_found /usr/pkg/openoffice.org2.4/program/components/../libmozjs.so 
misc/openoffice2-bin openoffice2-bin-2.4.3
   l: not_found 
/usr/pkg/openoffice.org2.4/program/components/../libxpcom_compat.so 
misc/openoffice2-bin openoffice2-bin-2.4.3
   l: not_found /usr/pkg/openoffice.org2.4/program/components/../libprldap50.so 
misc/openoffice2-bin openoffice2-bin-2.4.3
   l: not_found /usr/pkg/openoffice.org2.4/program/components/../libxpcom.so 
misc/openoffice2-bin openoffice2-bin-2.4.3
   l: not_found /usr/pkg/openoffice.org2.4/program/components/../libstdc++.so.6 
misc/openoffice2-bin openoffice2-bin-2.4.3
   l: not_found /usr/pkg/openoffice.org2.4/program/components/../libssl3.so 
misc/openoffice2-bin openoffice2-bin-2.4.3
   l: not_found /usr/pkg/openoffice.org2.4/program/components/../libplds4.so 
misc/openoffice2-bin openoffice2-bin-2.4.3
   l: not_found /usr/pkg/openoffice.org2.4/program/components/../libldap50.so 
misc/openoffice2-bin openoffice2-bin-2.4.3

Obviously OpenOffice can easily be fixed, the problem
in it is in .. in a path.

Again, you can use 'pkg_lint_summary -l' for testing.
For comparison run the following command:
   ftp -o - \
      
ftp://ftp.NetBSD.org/pub/pkgsrc/packages/NetBSD/i386/5.0.2_2010Q2/All/pkg_summary.bz2
 |
   bzcat | pkg_lint_summary -l

pkg_lint_summary is a part of wip/pkg_summary-utils.

Patch is below.

Index: mk/flavor/pkg/metadata.mk
===================================================================
RCS file: /cvsroot/pkgsrc/mk/flavor/pkg/metadata.mk,v
retrieving revision 1.34
diff -u -r1.34 metadata.mk
--- mk/flavor/pkg/metadata.mk   3 May 2010 16:46:36 -0000       1.34
+++ mk/flavor/pkg/metadata.mk   2 Aug 2010 17:06:00 -0000
@@ -64,11 +64,11 @@
        bins=`${AWK} '/(^|\/)(bin|sbin|libexec)\// { print 
"${DESTDIR}${PREFIX}/" $$0 } END { exit 0 }' ${_PLIST_NOKEYWORDS}`; \
        case ${OBJECT_FMT:Q}"" in                                       \
        ELF)                                                            \
-               libs=`${AWK} '/(^|\/)lib\/lib.*\.so(\.[0-9]+)?$$/ { print 
"${DESTDIR}${PREFIX}/" $$0 } END { exit 0 }' ${_PLIST_NOKEYWORDS}`; \
+               libs=`${AWK} '/\/lib.*\.so(\.[0-9]+)*$$/ { print 
"${DESTDIR}${PREFIX}/" $$0 } END { exit 0 }' ${_PLIST_NOKEYWORDS}`; \
                if ${TEST} -n "$$bins" -o -n "$$libs"; then             \
                        requires=`($$ldd $$bins $$libs 2>/dev/null || ${TRUE}) 
| ${AWK} '$$2 == "=>" && $$3 ~ "/" { print $$3 }' | ${SORT} -u`; \
                fi;                                                     \
-               linklibs=`${AWK} '/.*\.so(\.[0-9]+)?$$/ { print 
"${DESTDIR}${PREFIX}/" $$0 }' ${_PLIST_NOKEYWORDS}`; \
+               linklibs=`${AWK} '/.*\.so(\.[0-9]+)*$$/ { print 
"${DESTDIR}${PREFIX}/" $$0 }' ${_PLIST_NOKEYWORDS}`; \
                for i in $$linklibs; do                                 \
                        if ${TEST} -r $$i -a ! -x $$i -a ! -h $$i; then \
                                ${TEST} ${PKG_DEVELOPER:Uno:Q}"" = "no" || \
@@ -78,7 +78,7 @@
                done;                                                   \
                ;;                                                      \
        Mach-O)                                                         \
-               libs=`${AWK} '/(^|\/)lib\/lib.*\.dylib/ { print 
"${DESTDIR}${PREFIX}/" $$0 } END { exit 0 }' ${_PLIST_NOKEYWORDS}`; \
+               libs=`${AWK} '/\/lib.*\.dylib/ { print "${DESTDIR}${PREFIX}/" 
$$0 } END { exit 0 }' ${_PLIST_NOKEYWORDS}`; \
                if ${TEST} "$$bins" != "" -o "$$libs" != ""; then       \
                        requires=`($$ldd $$bins $$libs 2>/dev/null || ${TRUE}) 
| ${AWK} '/compatibility version/ { print $$1 }' | ${SORT} -u`; \
                fi;                                                     \
@@ -89,7 +89,6 @@
                ${SORT} | uniq -c | awk '$$1 == 2 {print $$2}'`; \
        for i in "" $$libs; do                                          \
                ${TEST} "$$i" != "" || continue;                        \
-               ${TEST} -h "$$i" && echo "$$i" | grep '[.]so$$' > /dev/null && 
continue;        \
                ${ECHO} "PROVIDES=$${i}";                               \
        done | ${SED} -e 's,^PROVIDES=${DESTDIR},PROVIDES=,'            \
                >> ${.TARGET}.tmp;                                      \

-- 
Best regards, Aleksey Cheusov.