Subject: pkg/21747: devel/SDL/ has redundant .include's
To: None <>
From: None <>
List: netbsd-bugs
Date: 06/01/2003 10:43:48
>Number:         21747
>Category:       pkg
>Synopsis:       devel/SDL/ has redundant .include's
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    pkg-manager
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Sun Jun 01 14:45:00 UTC 2003
>Originator:     Todd Vierling
>Release:        NetBSD 1.6.1_STABLE
	DUH.ORG:  Pointing out the obvious since 1994.
System: NetBSD 1.6.1_STABLE NetBSD 1.6.1_STABLE (TODD) #0: Fri May 30 15:13:58 EDT 2003 i386
Architecture: i386
Machine: i386

devel/SDL/ .include's all of SDL's own dependencies.  This is
wrong, because SDL already pulls these in (and dependencies of SDL don't
give a damn what dependencies it pulls in).

This is starting to look like a disturbing trend.  I've said this before
and I'll say it again here:  A PACKAGE SHOULD ONLY PULL IN THE
DEPENDENCIES IT NEEDS, AND NO MORE.  This has the added implication that a should, in almost all cases, refer ONLY to the package
directory in which that resides (and generic stuff in mk/).

Adding a bunch of .include's to a only complicates the build
and packaging process, and makes it very difficult for someone to pull out
a single dependency in a local pkgsrc tree.  (Case in point:  I have to
edit two places to take ESD support out of SDL -- when I should be able to
do this by editing only SDL's Makefile.)


RCS file: /pub/NetBSD-CVS/pkgsrc/devel/SDL/,v
retrieving revision 1.6
diff -u -r1.6
---	2003/05/02 11:54:18	1.6
+++	2003/06/01 14:39:02
@@ -19,15 +19,6 @@
 PTHREAD_OPTS+=		require
-.if defined(SDL_USE_NAS)
-.  include "../../audio/nas/"
-.include "../../audio/esound/"
-.include "../../graphics/MesaLib/"
-.include "../../graphics/glut/"
-.include "../../graphics/aalib-x11/"
-.include "../../mk/"
 SDL-buildlink: _BUILDLINK_USE