Re: print/ghostscript - "cups" option shouldn't be default?

["D'Arcy J.M. Cain" <darcy%NetBSD.org@localhost>]

On Fri, 20 May 2011 17:56:09 -0300
Silas Silva <silasdb%gmail.com@localhost> wrote:
> It seems it is a real dependency.  I have just removed the line that
> defined depend and it stopped at an error trying to include some cups
> headers.

Really?  I just tried the following patch after making sure that cups
wasn't installed on that machine and it compiled.  I don't know if it
would actually work with cups though.  I don't know enough about the
intricacies of either program.

Index: Makefile
===================================================================
RCS file: /cvsroot/pkgsrc/print/ghostscript/Makefile,v
retrieving revision 1.83
diff -u -r1.83 Makefile
--- Makefile    13 Jan 2011 13:36:06 -0000      1.83
+++ Makefile    21 May 2011 02:48:43 -0000
@@ -28,6 +28,7 @@
 BUILD_TARGET=          so
 INSTALL_TARGET=                soinstall
 CONFIGURE_ARGS+=       -with-omni=no   # otherwise pulls in libstdc++
+CONFIGURE_ARGS+=    --enable-cups
 
 REPLACE_PERL+=         lib/fixmswrd.pl
 
Index: options.mk
===================================================================
RCS file: /cvsroot/pkgsrc/print/ghostscript/options.mk,v
retrieving revision 1.11
diff -u -r1.11 options.mk
--- options.mk  23 Dec 2010 11:44:54 -0000      1.11
+++ options.mk  21 May 2011 02:48:43 -0000
@@ -15,28 +15,6 @@
 CONFIGURE_ARGS+=       --without-x
 .endif
 
-PLIST_VARS+=           cups
-.if !empty(PKG_OPTIONS:Mcups)
-CONFIGURE_ARGS+=       --enable-cups
-PLIST.cups=            yes
-INSTALL_TARGET+=       install-cups
-
-CUPS_CONFDIR?= ${PKG_SYSCONFBASEDIR}/cups
-CUPS_EGDIR=    ${PREFIX}/share/examples/cups
-CONF_FILES+=   ${CUPS_EGDIR}/pstoraster.convs
${CUPS_CONFDIR}/pstoraster.convs -
-SUBST_CLASSES+=                cupsetc
-SUBST_STAGE.cupsetc=   post-extract
-SUBST_MESSAGE.cupsetc= Fixing CUPS etc directory path to install as
example -SUBST_FILES.cupsetc=   cups/cups.mak
-SUBST_SED.cupsetc=     -e 's|$$(CUPSSERVERROOT)|${CUPS_EGDIR}|g'
-
-.include "../../print/cups/buildlink3.mk"
-.include "../../mk/jpeg.buildlink3.mk"
-.else
-CONFIGURE_ARGS+=       --disable-cups
-.endif
-
 .if !empty(PKG_OPTIONS:Mdebug)
 CONFIGURE_ARGS+=       --enable-debug
 .endif


-- 
D'Arcy J.M. Cain <darcy%NetBSD.org@localhost>
http://www.NetBSD.org/