Date: Tue, 15 Mar 2016 07:16:41 +0100 From: Baptiste Daroussin <bapt@FreeBSD.org> To: Ed Maste <emaste@FreeBSD.org> Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r296769 - in head/sys/boot/efi: boot1 loader Message-ID: <20160315061641.GI11339@ivaldir.etoilebsd.net> In-Reply-To: <201603122144.u2CLiXv5023698@repo.freebsd.org> References: <201603122144.u2CLiXv5023698@repo.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--1E1Oui4vdubnXi3o Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Sat, Mar 12, 2016 at 09:44:33PM +0000, Ed Maste wrote: > Author: emaste > Date: Sat Mar 12 21:44:33 2016 > New Revision: 296769 > URL: https://svnweb.freebsd.org/changeset/base/296769 >=20 > Log: > boot/efi: Prefer nm to objdump > =20 > Both objdump and nm are equally capable of reporting undefined symbols. > =20 > This gets us a step closer to building without binutils as we have an nm > implementation from ELF Tool Chain. > =20 > Reviewed by: bdrewery > MFC after: 1 month > Sponsored by: The FreeBSD Foundation > Differential Revision: https://reviews.freebsd.org/D5613 >=20 > Modified: > head/sys/boot/efi/boot1/Makefile > head/sys/boot/efi/loader/Makefile >=20 > Modified: head/sys/boot/efi/boot1/Makefile > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D > --- head/sys/boot/efi/boot1/Makefile Sat Mar 12 20:05:23 2016 (r296768) > +++ head/sys/boot/efi/boot1/Makefile Sat Mar 12 21:44:33 2016 (r296769) > @@ -73,8 +73,8 @@ LDADD+=3D -lstand > =20 > DPADD+=3D ${LDSCRIPT} > =20 > +NM?=3D nm > OBJCOPY?=3D objcopy > -OBJDUMP?=3D objdump > =20 > .if ${MACHINE_CPUARCH} =3D=3D "amd64" > EFI_TARGET=3D efi-app-x86_64 > @@ -85,8 +85,8 @@ EFI_TARGET=3D binary > .endif > =20 > boot1.efi: ${PROG} > - if [ `${OBJDUMP} -t ${.ALLSRC} | fgrep '*UND*' | wc -l` !=3D 0 ]; then \ > - ${OBJDUMP} -t ${.ALLSRC} | fgrep '*UND*'; \ > + if ${NM} ${.ALLSRC} | grep ' U '; then \ grep -q maybe to avoid extra noise? Bapt --1E1Oui4vdubnXi3o Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQIcBAEBCAAGBQJW56i0AAoJEGOJi9zxtz5a+uQQAKNpUwDXswYgeQn0fJBmMRNW 3a2HRkBbDkmFsndjR8+dFqggdu+mVd7DdnKW9J4lwUfreh2eTh82Axh+fQgncOH5 NMke9mcPeUiKIYyidGxSgFOonMoBsToWofPr0l3lsb7MS8zBLbCmz6u/nrnxHJwD vKR1cJ4P9HEII6IK1vGuA57BPfJMtLKM0wkpLwgc+EP4CZlZqkbgsCmutXPeGCNi pru3jI75XbXEcGUws99YJH7xEBF2gJbuyDgQ5nm5RQbMME6d9FxNIP6dFMzd2yaH jkFXYbcyHZWtP+9q4cgWb3rGkCUQVM2tbfepZTnMbgfjaI/7OWXLjbtr9E8DgPiN wrox/kR+zCRvbwdcP3PYMHyFwas+7bjYo7LwNVXCVzg4tNno4Ti56Jr/vL8g84TH 0KfMCDZKCkn76asAHMPScmYRn+JQyq4MIlf6NmZDXt1vNbVo0bJ2gkhOBUkFfObs FrXoWkvdnIlqfTrwGK1tVDY0AbqqMAGzEfukRi8f9GzYx+pqmPzz1qVF/OeS3WSK ci/BqaBNL86u4VXJzcsXTC5T4qfkhNE3yWrg7z/hZHMsrnCrHplOY2q9uQKMpd4/ Jmsa6OhubtNiGSFeSDrm1hHuws26OkJRK5darfB+Uuk39nOJPbbORiDCdNPPfZRs XrTTjrTFIt4Y0MHf7xoC =8tXk -----END PGP SIGNATURE----- --1E1Oui4vdubnXi3o--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20160315061641.GI11339>