Subject: Re: CVS commit: pkgsrc/print/ghostscript
To: None <pkgsrc-changes@NetBSD.org>
From: None <david.sainty@dtsp.co.nz>
List: pkgsrc-changes
Date: 10/14/2007 15:26:55
Joerg Sonnenberger writes:

> On Thu, Oct 11, 2007 at 11:45:07AM +0000, David Sainty wrote:
> > I solve this case here by explicitly disabling CUPS support.  For
> > normal cases this should mean "No change", but in exotic cases it fixes
> > the build and install.
> 
> Does the CUPS support add other dependencies or just build the
> pstoraster binary? If it is the latter, I would strongly prefer to make
> that the default and be able to just drop the special handling for CUPS.

It would add a dependency on "cups".  Elsewhere, "cups" is often a
package option, so it might be a shame to pull it in here.  Making
"cups" a package option on "print/ghostscript" is easy of course, and
is one solution.

But there is another alternative that is possibly more correct, but
much more work.  It may makes more sense to have a
"print/ghostscript-cups" that depends on "print/cups-lib", and have
"print/cups" depend on both (possibly with a "ghostscript" package
option for "print/ghostscript-cups", but likely by default it would be
enabled).

The idea being that if you want "print/cups", getting
"print/ghostscript-cups" by default is probably a good thing.  But you
can't do that as it stands without getting a dependency loop.

But I also didn't look deeply into this, since I had another solution
that maintained the status quo.

Cheers,

Dave