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

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

The following reply was made to PR pkg/42907; it has been noted by GNATS.

From: Aleksey Cheusov <cheusov%tut.by@localhost>
To: gnats-bugs%NetBSD.org@localhost
Cc: pkg-manager%netbsd.org@localhost, pkgsrc-bugs%netbsd.org@localhost, 
gnats-admin%netbsd.org@localhost,
         dholland%NetBSD.org@localhost
Subject: Re: pkg/42907 (Errors in REQUIRES/PROVIDES variables in pkg_summary(5))
Date: Mon, 02 Aug 2010 20:21:26 +0300

 > 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.