From owner-freebsd-x11@FreeBSD.ORG Mon Jan 12 04:28:22 2015 Return-Path: Delivered-To: freebsd-x11@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 7E6B2D7D; Mon, 12 Jan 2015 04:28:22 +0000 (UTC) Received: from mail.soaustin.net (pancho.soaustin.net [76.74.250.40]) by mx1.freebsd.org (Postfix) with ESMTP id 63A29EEB; Mon, 12 Jan 2015 04:28:21 +0000 (UTC) Received: by mail.soaustin.net (Postfix, from userid 502) id 5E0B156083; Sun, 11 Jan 2015 22:28:21 -0600 (CST) Date: Sun, 11 Jan 2015 22:28:21 -0600 From: Mark Linimon To: freebsd-x11@FreeBSD.org Subject: creeping dependency on x11-drivers/xf86-video-ati in tier-2? Message-ID: <20150112042821.GA16943@lonesome.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.5.21 (2010-09-15) Cc: linimon@FreeBSD.org X-BeenThere: freebsd-x11@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: X11 on FreeBSD -- maintaining and support List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 12 Jan 2015 04:28:22 -0000 It looks like for !(i386/amd64) builds that an uncessary dependency on the above has crept in. In that driver's Makefile we see # No Radeon kernel driver on non-x86 and PC98. ONLY_FOR_ARCHS= i386 amd64 but it's still depended on by default in x11-drivers/xorg-drivers/Makefile . I think the following fix is needed -- but is it complete? Thanks. mcl Index: x11-drivers/xorg-drivers/Makefile =================================================================== --- x11-drivers/xorg-drivers/Makefile (revision 376802) +++ x11-drivers/xorg-drivers/Makefile (working copy) @@ -30,8 +30,7 @@ void \ wacom -VIDEO_DRIVERS= ati \ - mach64 \ +VIDEO_DRIVERS= mach64 \ nv \ r128 \ vesa \ @@ -60,7 +59,6 @@ OPTIONS_DEFAULT= MOUSE \ KEYBOARD \ - ATI \ MACH64 \ NV \ R128 \ @@ -73,8 +71,8 @@ . endfor .endfor -OPTIONS_DEFINE_i386= INTEL OPENCHROME -OPTIONS_DEFAULT_i386= INTEL OPENCHROME +OPTIONS_DEFINE_i386= ATI INTEL OPENCHROME +OPTIONS_DEFAULT_i386= ATI INTEL OPENCHROME OPTIONS_DEFINE_amd64:= ${OPTIONS_DEFINE_i386} OPTIONS_DEFAULT_amd64:= ${OPTIONS_DEFAULT_i386} @@ -85,6 +83,7 @@ OPTIONS_DEFINE_ia64= OPENCHROME OPTIONS_DEFAULT_ia64= OPENCHROME +ATI_DESC= Install ati video driver INTEL_DESC= Install intel video driver OPENCHROME_DESC= Install openchrome video driver SUNFFB_DESC= Install sunffb video driver @@ -96,7 +95,7 @@ # Manual add arch specific drivers so they be added to depend lines. .if ${ARCH}==i386 || ${ARCH}==amd64 -VIDEO_DRIVERS+= intel openchrome +VIDEO_DRIVERS+= ati intel openchrome .endif .if ${ARCH}==sparc64