Re: (cross/h8300-elf-binutils) Re: broken packages for 2013q2

[David Holland <dholland-pkgtech%netbsd.org@localhost>]

On Sat, Jun 15, 2013 at 01:24:00AM +0900, Makoto Fujiwara wrote:
 > | From: David Holland <dholland-pkgtech%NetBSD.org@localhost>
 > | Subject: broken packages for 2013q2
 > | Date: Wed, 12 Jun 2013 04:02:11 +0000
 > 
 > >    http://www.netbsd.org/~dholland/broken/
 > 
 > | cross/h8300-elf-binutils   h8300-elf-binutils      open    2013q2  2.19.1
 > |   :
 > |    problem handling the installed libiberty.a
 > 
 > On particular environment, (6.99.21, clang, amd64)
 > Following change packages. 
 >   - (6.1, gcc, amd64) does not need the change.
 > 
 > Is there any way to switch lines ?

Not really. However:

 >               h8300-elf-objdump      h8300-elf-ranlib \
 >               h8300-elf-readelf      h8300-elf-size \
 >               h8300-elf-strings      h8300-elf-strip ; do \
 > -            ${LN} -s ${H8300_PREFIX}/bin/"$$f" \
 > +            ${LN} -sf ${H8300_PREFIX}/bin/"$$f" \
 >                ${DESTDIR}${PREFIX}/bin/"$$f" || exit 1; \
 >      done

This makes me worry that the problem is that it isn't getting the
right crosscompile paths. If it has two directories the same that are
supposed to be different, those files will already exist and you'll
need -f; but if so, those symlinks will probably overwrite files that
are needed.

If not, this change should be ok on all platforms.

 > -    ${MV} ${DESTDIR}${H8300_PREFIX}/lib/libiberty.a \
 > +    ${MV} ${DESTDIR}${H8300_PREFIX}/lib/${MACHINE_ARCH}/libiberty.a \
 >            ${DESTDIR}${H8300_PREFIX}/lib/libiberty_binutils.a

What about something like

        for d in ${MACHINE_ARCH}/libiberty.a libiberty.a; do \
                if [ -f ${DESTDIR}${H8300_PREFIX}/lib/$$d ]; then
                        ${MV} ${DESTDIR}${H8300_PREFIX}/lib/$$d \
                              ${DESTDIR}${H8300_PREFIX}/lib/$$d; \
                        break; \
                fi;
        done

?

-- 
David A. Holland
dholland%netbsd.org@localhost