Re: changing the default ghostscript type

[Greg Troxel <gdt%lexort.com@localhost>]

Leonardo Taccari <leot%NetBSD.org@localhost> writes:

> Hello Greg,
>
> Greg Troxel writes:
>> I am contemplating changing the code so that in the absence of an
>> installed ghostscript package, the package will depend on the chosen
>> version, but that if either version is installed, the installed version
>> will be used.  With this setup, I believe that if someone installs
>> ghostcript-gpl (overding the vulnerable check), then ghostscript, even
>> with the new default option, will happiliy depend on it, that that
>> person can easily have what they want, even with binary packages.
>> [...]
>
> I think that's probably okay for packages that needs ghostscript
> via DEPENDS (e.g. if they just need `gs') but I am not sure if it
> can be problematic for packages that includes
> print/ghostscript/buildlink3.mk.
>
> Apart that the PKG_OPTIONS.gs_type will then not reflect the actual
> reality and leads to possible surprises (okay, `pkg_info
> ghostscript-{a,}gpl' will then reveal the truth :)).
>
> For these reasons I think it would be better to avoid `step 2'.

That's a good point that I had not considered.

I guess it's a tradeoff between users being able to install the
unmaintained GPL3 version (if that's what they want) with minimal
hassle, vs the problems caused by the unintended switch.   But I tend to
avoiding wrong things that will cause unknown future trouble.

  ./editors/TeXmacs/Makefile:.include "../../print/ghostscript/buildlink3.mk"
  ./fonts/ghostscript-cidfonts-ryumin/Makefile:.include "../../print/ghostscript/buildlink3.mk"
  ./graphics/GraphicsMagick/buildlink3.mk:.include "../../print/ghostscript/buildlink3.mk"
  ./graphics/GraphicsMagick/options.mk:.include "../../print/ghostscript/buildlink3.mk"
  ./graphics/ImageMagick/buildlink3.mk:.include "../../print/ghostscript/buildlink3.mk"
  ./graphics/ImageMagick/Makefile:.include "../../print/ghostscript/buildlink3.mk"
  ./graphics/camlimages/Makefile:.include "../../print/ghostscript/buildlink3.mk"
  ./graphics/graphviz/CVS/Base/options.mk:.    include "../../print/ghostscript/buildlink3.mk"
  ./graphics/graphviz/options.mk:.  include "../../print/ghostscript/buildlink3.mk"
  ./graphics/ImageMagick6/buildlink3.mk:.include "../../print/ghostscript/buildlink3.mk"
  ./graphics/ImageMagick6/Makefile:.include "../../print/ghostscript/buildlink3.mk"
  ./math/R-CGIwithR/Makefile:.include "../../print/ghostscript/buildlink3.mk"
  ./print/foomatic4-filters/Makefile:#.include "../../print/ghostscript/buildlink3.mk"
  ./print/libspectre/buildlink3.mk:.include "../../print/ghostscript/buildlink3.mk"
  ./print/libspectre/Makefile:.include "../../print/ghostscript/buildlink3.mk"
  ./print/cups-filters/Makefile:.include "../../print/ghostscript/buildlink3.mk"
  ./print/hplip/Makefile:.include "../../print/ghostscript/buildlink3.mk"

  ./wip/ghostscript-cidfonts-ryumin/Makefile:.include "../../print/ghostscript/buildlink3.mk"
  ./wip/tc-hasida-table/Makefile:.include "../../print/ghostscript/buildlink3.mk"