[pkgsrc/trunk]: pkgsrc/sysutils/u-boot-pinebook u-boot-pinebook: Fixes option...

[leot <leot%pkgsrc.org@localhost>]

details:   https://anonhg.NetBSD.org/pkgsrc/rev/ccaaf0d93457
branches:  trunk
changeset: 325083:ccaaf0d93457
user:      leot <leot%pkgsrc.org@localhost>
date:      Fri Nov 09 11:48:03 2018 +0000

description:
u-boot-pinebook: Fixes options handling and Python tool handling

- PKG_DEFAULT_OPTIONS should not be used in package Makefile-s,
  use PKG_SUGGESTED_OPTIONS instead.
- u-boot.mk already defines that Python is needed, no need to redefine it there.

Reported, tested and thanks to <martin>!

diffstat:

 sysutils/u-boot-pinebook/Makefile |  10 +++-------
 1 files changed, 3 insertions(+), 7 deletions(-)

diffs (29 lines):

diff -r 54e2bd204d58 -r ccaaf0d93457 sysutils/u-boot-pinebook/Makefile
--- a/sysutils/u-boot-pinebook/Makefile Fri Nov 09 11:22:13 2018 +0000
+++ b/sysutils/u-boot-pinebook/Makefile Fri Nov 09 11:48:03 2018 +0000
@@ -1,4 +1,4 @@
-# $NetBSD: Makefile,v 1.8 2018/11/08 19:41:30 jakllsch Exp $
+# $NetBSD: Makefile,v 1.9 2018/11/09 11:48:03 leot Exp $
 
 UBOOT_VERSION=         2018.11rc3
 UBOOT_TARGET=          pinebook
@@ -15,17 +15,13 @@
 DISTINFO_FILE=         ${.CURDIR}/../../sysutils/u-boot-pinebook/distinfo
 PATCHDIR=              ${.CURDIR}/../../sysutils/u-boot-pinebook/patches
 
-# Switch to EL1 before booting kernel (required for 32-bit support)
-PKG_DEFAULT_OPTIONS+=  el1
-
 post-extract:
        ${CP} ${PREFIX}/share/arm-trusted-firmware/sun50iw1p1/bl31.bin ${WRKSRC}
 
 post-build:
        ${CAT} ${WRKSRC}/spl/sunxi-spl.bin ${WRKSRC}/u-boot.itb > ${WRKSRC}/u-boot-sunxi-with-spl.bin
 
-PYTHON_FOR_BUILD_ONLY= yes
-.include "../../lang/python/tool.mk"
-
 .include "../../sysutils/arm-trusted-firmware-sun50iw1p1/buildlink3.mk"
 .include "../../sysutils/u-boot/u-boot-arm64.mk"
+# Switch to EL1 before booting kernel (required for 32-bit support)
+PKG_SUGGESTED_OPTIONS+=        el1