Drop NEWPDF=false switch for recent Ghostscript (#288)

CHANGELOG.md

@@ -18,6 +18,7 @@ config :chromic_pdf, chrome_version: "Google Chrome 120.0.6099.71"
 
 ### Changed
 
+- Drop `-dNEWPDF=false` switch from Ghostscript >= 10.2. The old PDF interpreter has been removed and using the switch results in a warning now.
 - Split `Chromic.Template.styles/1` into `page_styles/1` and `header_footer_styles/1`, and trim the stylesheets.
 - Cookies set via `:set_cookie` are now `httpOnly: true` by default.
 

lib/chromic_pdf/pdfa/ghostscript_runner.ex

@@ -29,7 +29,7 @@ defmodule ChromicPDF.GhostscriptRunner do
 
   @ghostscript_bin "gs"
   @ghostscript_safer_version [9, 28]
-  @ghostscript_new_interpreter_version [9, 56]
+  @ghostscript_new_interpreter_version {[9, 56], [10, 2]}
 
   @spec run_postscript(binary(), binary()) :: binary()
   def run_postscript(pdf_path, ps_path) do
@@ -116,7 +116,10 @@ defmodule ChromicPDF.GhostscriptRunner do
   end
 
   defp maybe_disable_new_interpreter do
-    if semver_compare(ghostscript_version(), @ghostscript_new_interpreter_version) in [:eq, :gt] do
+    {bad, good} = @ghostscript_new_interpreter_version
+
+    if semver_compare(ghostscript_version(), bad) in [:eq, :gt] &&
+         semver_compare(ghostscript_version(), good) == :lt do
       # We get segmentation faults with the new intepreter (see https://github.com/bitcrowd/chromic_pdf/issues/153):
       #
       # /usr/bin/gs exited with status 139!