review 5.0.0 → 5.3.0

data/lib/review/makerhelper.rb

@@ -8,7 +8,6 @@
 require 'pathname'
 require 'fileutils'
 require 'yaml'
-require 'shellwords'
 
 begin
   require 'cgi/escape'
@@ -18,12 +17,6 @@ end
 
 module ReVIEW
   module MakerHelper
-    # Return review/bin directory
-    def bindir
-      Pathname.new("#{Pathname.new(__FILE__).realpath.dirname}/../../bin").realpath
-    end
-    module_function :bindir
-
     def h(str)
       CGI.escapeHTML(str)
     end
@@ -51,6 +44,7 @@ module ReVIEW
       Dir.open(from_dir) do |dir|
         dir.each do |fname|
           next if fname =~ /^\./
+
           if FileTest.directory?("#{from_dir}/#{fname}")
             image_files += copy_images_to_dir("#{from_dir}/#{fname}", "#{to_dir}/#{fname}", options)
           else
@@ -59,6 +53,7 @@ module ReVIEW
             is_converted = false
             (options[:convert] || {}).each do |orig_type, conv_type|
               next unless /\.#{orig_type}$/ =~ fname
+
               is_converted = system("convert #{from_dir}/#{fname} #{to_dir}/#{fname}.#{conv_type}")
               image_files << "#{from_dir}/#{fname}.#{conv_type}"
             end
@@ -66,7 +61,11 @@ module ReVIEW
             exts = options[:exts] || %w[png gif jpg jpeg svg pdf eps ai tif psd]
             exts_str = exts.join('|')
             if !is_converted && fname =~ /\.(#{exts_str})$/i
-              FileUtils.cp("#{from_dir}/#{fname}", to_dir)
+              if options[:use_symlink]
+                FileUtils.ln_s(File.realpath("#{from_dir}/#{fname}"), to_dir)
+              else
+                FileUtils.cp("#{from_dir}/#{fname}", to_dir)
+              end
               image_files << "#{from_dir}/#{fname}"
             end
           end
@@ -76,203 +75,5 @@ module ReVIEW
       image_files
     end
     module_function :copy_images_to_dir
-
-    def cleanup_mathimg(path = '_review_math')
-      math_dir = "./#{@config['imagedir']}/#{path}"
-      if @config['imgmath'] && Dir.exist?(math_dir)
-        FileUtils.rm_rf(math_dir)
-      end
-    end
-
-    def default_imgmath_preamble
-      <<-EOB
-\\documentclass[uplatex,a3paper,landscape]{jsarticle}
-\\usepackage[deluxe,uplatex]{otf}
-\\usepackage[T1]{fontenc}
-\\usepackage{textcomp}
-\\usepackage{lmodern}
-\\usepackage[dvipdfmx]{graphicx}
-\\usepackage[dvipdfmx,table]{xcolor}
-\\usepackage[utf8]{inputenc}
-\\usepackage{ascmac}
-\\usepackage{float}
-\\usepackage{alltt}
-\\usepackage{amsmath}
-\\usepackage{amssymb}
-\\usepackage{amsfonts}
-\\usepackage{anyfontsize}
-\\usepackage{bm}
-\\pagestyle{empty}
-% \\setpaperwidth{1000mm}
-    EOB
-    end
-
-    def make_math_images(math_dir)
-      fontsize = @config['imgmath_options']['fontsize'].to_f
-      lineheight = @config['imgmath_options']['lineheight'].to_f
-
-      texsrc = default_imgmath_preamble
-      if @config['imgmath_options']['preamble_file'] && File.readable?(@config['imgmath_options']['preamble_file'])
-        texsrc = File.read(@config['imgmath_options']['preamble_file'])
-      end
-
-      texsrc << <<-EOB
-\\begin{document}
-\\fontsize{#{fontsize}}{#{lineheight}}\\selectfont
-\\input{__IMGMATH_BODY__}
-\\end{document}
-EOB
-
-      hashes = File.readlines(File.join(math_dir, '__IMGMATH_BODY__.map')).sort.uniq
-      File.write(File.join(math_dir, '__IMGMATH_BODY__.map'), hashes.join)
-
-      File.open(File.join(math_dir, '__IMGMATH_BODY__.tex'), 'w') do |f|
-        File.open(File.join(math_dir, '__IMGMATH_BODY__.map')) do |map|
-          map.each_line do |l|
-            l.chomp!
-            f.puts "% #{l}"
-            f.puts File.read(File.join(math_dir, "__IMGMATH_BODY__#{l}.tex"))
-            File.unlink(File.join(math_dir, "__IMGMATH_BODY__#{l}.tex"))
-            f.puts '\\clearpage'
-            f.puts
-          end
-        end
-      end
-
-      math_dir = File.realpath(math_dir)
-      Dir.mktmpdir do |tmpdir|
-        FileUtils.cp([File.join(math_dir, '__IMGMATH_BODY__.tex'),
-                      File.join(math_dir, '__IMGMATH_BODY__.map')],
-                     tmpdir)
-        tex_path = File.join(tmpdir, '__IMGMATH__.tex')
-        File.write(tex_path, texsrc)
-
-        begin
-          case @config['imgmath_options']['converter']
-          when 'pdfcrop'
-            make_math_images_pdfcrop(tmpdir, tex_path, math_dir)
-          when 'dvipng'
-            make_math_images_dvipng(tmpdir, tex_path, math_dir)
-          else
-            error "unknown math converter error. imgmath_options/converter parameter should be 'pdfcrop' or 'dvipng'."
-          end
-        rescue CompileError
-          FileUtils.cp([tex_path,
-                        File.join(File.dirname(tex_path), '__IMGMATH__.log')],
-                       math_dir)
-          error "LaTeX math compile error. See #{math_dir}/__IMGMATH__.log for details."
-        end
-      end
-      FileUtils.rm_f([File.join(math_dir, '__IMGMATH_BODY__.tex'),
-                      File.join(math_dir, '__IMGMATH_BODY__.map')])
-    end
-    module_function :make_math_images
-
-    def make_math_images_pdfcrop(dir, tex_path, math_dir)
-      # rubocop:disable Metrics/BlockLength
-      Dir.chdir(dir) do
-        dvi_path = '__IMGMATH__.dvi'
-        pdf_path = '__IMGMATH__.pdf'
-        out, status = Open3.capture2e(*[@config['texcommand'], @config['texoptions'].shellsplit, tex_path].flatten.compact)
-        if !status.success? || (!File.exist?(dvi_path) && !File.exist?(pdf_path))
-          raise CompileError
-        end
-        if File.exist?(dvi_path)
-          out, status = Open3.capture2e(*[@config['dvicommand'], @config['dvioptions'].shellsplit, dvi_path].flatten.compact)
-          if !status.success? || !File.exist?(pdf_path)
-            warn "error in #{@config['dvicommand']}. Error log:\n#{out}"
-            raise CompileError
-          end
-        end
-        args = @config['imgmath_options']['pdfcrop_cmd'].shellsplit
-        args.map! do |m|
-          m.sub('%i', pdf_path).
-            sub('%o', '__IMGMATH__pdfcrop.pdf')
-        end
-        out, status = Open3.capture2e(*args)
-        unless status.success?
-          warn "error in pdfcrop. Error log:\n#{out}"
-          raise CompileError
-        end
-        pdf_path = '__IMGMATH__pdfcrop.pdf'
-        pdf_path2 = pdf_path
-
-        File.open('__IMGMATH_BODY__.map') do |f|
-          page = 0
-          f.each_line do |key|
-            page += 1
-            key.chomp!
-            if File.exist?(File.join(math_dir, "_gen_#{key}.#{@config['imgmath_options']['format']}"))
-              # made already
-              next
-            end
-
-            if @config['imgmath_options']['extract_singlepage']
-              # if extract_singlepage = true, split each page
-              args = @config['imgmath_options']['pdfextract_cmd'].shellsplit
-
-              args.map! do |m|
-                m.sub('%i', pdf_path).
-                  sub('%o', "__IMGMATH__pdfcrop_p#{page}.pdf").
-                  sub('%O', "__IMGMATH__pdfcrop_p#{page}").
-                  sub('%p', page.to_s)
-              end
-              out, status = Open3.capture2e(*args)
-              unless status.success?
-                warn "error in pdf extracting. Error log:\n#{out}"
-                raise CompileError
-              end
-
-              pdf_path2 = "__IMGMATH__pdfcrop_p#{page}.pdf"
-            end
-
-            args = @config['imgmath_options']['pdfcrop_pixelize_cmd'].shellsplit
-            args.map! do |m|
-              m.sub('%i', pdf_path2).
-                sub('%t', @config['imgmath_options']['format']).
-                sub('%o', File.join(math_dir, "_gen_#{key}.#{@config['imgmath_options']['format']}")).
-                sub('%O', File.join(math_dir, "_gen_#{key}")).
-                sub('%p', page.to_s)
-            end
-            out, status = Open3.capture2e(*args)
-            unless status.success?
-              warn "error in pdf pixelizing. Error log:\n#{out}"
-              raise CompileError
-            end
-          end
-        end
-      end
-      # rubocop:enable Metrics/BlockLength
-    end
-
-    def make_math_images_dvipng(dir, tex_path, math_dir)
-      Dir.chdir(dir) do
-        dvi_path = '__IMGMATH__.dvi'
-        out, status = Open3.capture2e(*[@config['texcommand'], @config['texoptions'].shellsplit, tex_path].flatten.compact)
-        if !status.success? || !File.exist?(dvi_path)
-          raise CompileError
-        end
-
-        File.open('__IMGMATH_BODY__.map') do |f|
-          page = 0
-          f.each_line do |key|
-            page += 1
-            key.chomp!
-            args = @config['imgmath_options']['dvipng_cmd'].shellsplit
-            args.map! do |m|
-              m.sub('%i', dvi_path).
-                sub('%o', File.join(math_dir, "_gen_#{key}.#{@config['imgmath_options']['format']}")).
-                sub('%O', File.join(math_dir, "_gen_#{key}")).
-                sub('%p', page.to_s)
-            end
-            out, status = Open3.capture2e(*args)
-            unless status.success?
-              warn "error in dvipng. Error log:\n#{out}"
-              raise CompileError
-            end
-          end
-        end
-      end
-    end
   end
 end

data/lib/review/markdownbuilder.rb

@@ -1,4 +1,4 @@
-# Copyright (c) 2013-2019 KADO Masanori, Masayoshi Takahashi, Kenshi Muto
+# Copyright (c) 2013-2021 KADO Masanori, Masayoshi Takahashi, Kenshi Muto
 #
 # This program is free software.
 # You can distribute or modify this program under the terms of
@@ -18,6 +18,7 @@ module ReVIEW
     end
 
     def builder_init_file
+      super
       @noindent = nil
       @blank_seen = nil
       @ul_indent = 0
@@ -97,7 +98,7 @@ module ReVIEW
     end
 
     def ul_item_begin(lines)
-      puts '  ' * (@ul_indent - 1) + '* ' + join_lines_to_paragraph(lines)
+      puts ('  ' * (@ul_indent - 1)) + '* ' + join_lines_to_paragraph(lines)
     end
 
     def ul_item_end
@@ -224,6 +225,14 @@ module ReVIEW
       "<u>#{str}</u>"
     end
 
+    def inline_ins(str)
+      "<ins>#{str}</ins>"
+    end
+
+    def inline_del(str)
+      "~~#{str}~~"
+    end
+
     def image_image(id, caption, _metric)
       blank
       puts "![#{compile_inline(caption)}](#{@chapter.image(id).path.sub(%r{\A\./}, '')})"
@@ -237,13 +246,35 @@ module ReVIEW
     def inline_img(id)
       "#{I18n.t('image')}#{@chapter.image(id).number}"
     rescue KeyError
-      error "unknown image: #{id}"
+      app_error "unknown image: #{id}"
     end
 
     def inline_dtp(str)
       "<!-- DTP:#{str} -->"
     end
 
+    def inline_hd_chap(chap, id)
+      n = chap.headline_index.number(id)
+      if n.present? && chap.number && over_secnolevel?(n)
+        str = I18n.t('hd_quote', [n, compile_inline(chap.headline(id).caption)])
+      else
+        str = I18n.t('hd_quote_without_number', compile_inline(chap.headline(id).caption))
+      end
+      if @book.config['chapterlink']
+        if @chapter == chap
+          anchor = 'h' + n.tr('.', '-')
+          %Q(<a href="##{anchor}">#{str}</a>)
+        else
+          warn 'MARKDOWNBuilder does not support links to other chapters', location: location
+          str
+        end
+      else
+        str
+      end
+    rescue KeyError
+      app_error "unknown headline: #{id}"
+    end
+
     def indepimage(_lines, id, caption = '', _metric = nil)
       blank
       puts "![#{compile_inline(caption)}](#{@chapter.image(id).path.sub(%r{\A\./}, '')})"
@@ -330,6 +361,14 @@ module ReVIEW
       "[^#{id}]"
     end
 
+    def inline_endnote(id)
+      "<sup>#{I18n.t('html_endnote_refmark', @chapter.endnote(id).number)}</sup>"
+    end
+
+    def endnote_item(id)
+      puts "#{I18n.t('html_endnote_textmark', @chapter.endnote(id).number)}#{compile_inline(@chapter.endnote(id).content)}"
+    end
+
     def inline_br(_str)
       "\n"
     end
@@ -358,6 +397,7 @@ module ReVIEW
 
     def comment(lines, comment = nil)
       return unless @book.config['draft']
+
       lines ||= []
       unless comment.blank?
         lines.unshift(comment)
@@ -370,7 +410,7 @@ module ReVIEW
       begin
         "![](#{@chapter.image(id).path.sub(%r{\A\./}, '')})"
       rescue
-        warn "image not bound: #{id}"
+        warn "image not bound: #{id}", location: location
         %Q(<pre>missing image: #{id}</pre>)
       end
     end

data/lib/review/pdfmaker.rb

@@ -1,4 +1,4 @@
-# Copyright (c) 2010-2020 Kenshi Muto and Masayoshi Takahashi
+# Copyright (c) 2010-2021 Kenshi Muto and Masayoshi Takahashi
 #
 # This program is free software.
 # You can distribute or modify this program under the terms of
@@ -22,11 +22,15 @@ require 'review/yamlloader'
 require 'review/version'
 require 'review/makerhelper'
 require 'review/template'
+require 'review/latexbox'
+require 'review/call_hook'
+require 'review/loggable'
 
 module ReVIEW
   class PDFMaker
-    include FileUtils
     include ReVIEW::LaTeXUtils
+    include Loggable
+    include ReVIEW::CallHook
 
     attr_accessor :config, :basedir
 
@@ -35,13 +39,14 @@ module ReVIEW
       @logger = ReVIEW.logger
       @input_files = Hash.new { |h, key| h[key] = '' }
       @mastertex = '__REVIEW_BOOK__'
+      @compile_errors = nil
     end
 
     def system_with_info(*args)
       @logger.info args.join(' ')
       out, status = Open3.capture2e(*args)
       unless status.success?
-        @logger.error "execution error\n\nError log:\n" + out
+        error "execution error\n\nError log:\n#{out}"
       end
     end
 
@@ -49,19 +54,10 @@ module ReVIEW
       @logger.info args.join(' ')
       out, status = Open3.capture2e(*args)
       unless status.success?
-        error "failed to run command: #{args.join(' ')}\n\nError log:\n" + out
+        error! "failed to run command: #{args.join(' ')}\n\nError log:\n#{out}"
       end
     end
 
-    def error(msg)
-      @logger.error msg
-      exit 1
-    end
-
-    def warn(msg)
-      @logger.warn msg
-    end
-
     def pdf_filepath
       File.join(@basedir, @config['bookname'] + '.pdf')
     end
@@ -89,7 +85,7 @@ module ReVIEW
       if ignore_errors
         @logger.info 'compile error, but try to generate PDF file'
       else
-        error 'compile error, No PDF file output.'
+        error! 'compile error, No PDF file output.'
       end
     end
 
@@ -123,7 +119,7 @@ module ReVIEW
 
     def execute(*args)
       cmd_config, yamlfile = parse_opts(args)
-      error "#{yamlfile} not found." unless File.exist?(yamlfile)
+      error! "#{yamlfile} not found." unless File.exist?(yamlfile)
 
       @config = ReVIEW::Configure.create(maker: 'pdfmaker',
                                          yamlfile: yamlfile,
@@ -151,7 +147,8 @@ module ReVIEW
         generate_pdf
       rescue ApplicationError => e
         raise if @config['debug']
-        error(e.message)
+
+        error! e.message
       end
     end
 
@@ -198,7 +195,7 @@ module ReVIEW
       Dir.chdir(@path) do
         File.open("./#{@mastertex}.tex", 'wb') { |f| f.write template }
 
-        call_hook('hook_beforetexcompile')
+        call_hook('hook_beforetexcompile', Dir.pwd, @basedir, base_dir: @basedir)
 
         ## do compile
         if ENV['REVIEW_SAFE_MODE'].to_i & 4 > 0
@@ -213,7 +210,7 @@ module ReVIEW
           makeindex_dic = ReVIEW::Configure.values['pdfmaker']['makeindex_dic']
         else
           unless @config['texcommand'].present?
-            error "texcommand isn't defined."
+            error! "texcommand isn't defined."
           end
           texcommand = @config['texcommand']
           dvicommand = @config['dvicommand']
@@ -241,32 +238,32 @@ module ReVIEW
           system_or_raise(*[texcommand, texoptions, "#{@mastertex}.tex"].flatten.compact)
         end
 
-        call_hook('hook_beforemakeindex')
+        call_hook('hook_beforemakeindex', Dir.pwd, @basedir, base_dir: @basedir)
         if @config['pdfmaker']['makeindex'] && File.size?("#{@mastertex}.idx")
           system_or_raise(*[makeindex_command, makeindex_options, @mastertex].flatten.compact)
+          call_hook('hook_aftermakeindex', Dir.pwd, @basedir, base_dir: @basedir)
           system_or_raise(*[texcommand, texoptions, "#{@mastertex}.tex"].flatten.compact)
         end
-        call_hook('hook_aftermakeindex')
 
         system_or_raise(*[texcommand, texoptions, "#{@mastertex}.tex"].flatten.compact)
-        call_hook('hook_aftertexcompile')
+        call_hook('hook_aftertexcompile', Dir.pwd, @basedir, base_dir: @basedir)
 
         if File.exist?("#{@mastertex}.dvi") && dvicommand.present?
           system_or_raise(*[dvicommand, dvioptions, "#{@mastertex}.dvi"].flatten.compact)
-          call_hook('hook_afterdvipdf')
+          call_hook('hook_afterdvipdf', Dir.pwd, @basedir, base_dir: @basedir)
         end
       end
     end
 
     def generate_pdf
       remove_old_file
-      erb_config
       @path = build_path
       begin
         @compile_errors = nil
 
         book = ReVIEW::Book::Base.new(@basedir, config: @config)
         @converter = ReVIEW::Converter.new(book, ReVIEW::LATEXBuilder.new)
+        erb_config
 
         @input_files = make_input_files(book)
 
@@ -276,6 +273,9 @@ module ReVIEW
         @config['usepackage'] = ''
         @config['usepackage'] = "\\usepackage{#{@config['texstyle']}}" if @config['texstyle']
 
+        if @config['pdfmaker']['use_symlink']
+          logger.info 'use symlink'
+        end
         copy_images(@config['imagedir'], File.join(@path, @config['imagedir']))
         copy_sty(File.join(Dir.pwd, 'sty'), @path)
         copy_sty(File.join(Dir.pwd, 'sty'), @path, 'fd')
@@ -287,8 +287,9 @@ module ReVIEW
         build_pdf
 
         FileUtils.cp(File.join(@path, "#{@mastertex}.pdf"), pdf_filepath)
+        @logger.success("built #{File.basename(pdf_filepath)}")
       ensure
-        remove_entry_secure(@path) unless @config['debug']
+        FileUtils.remove_entry_secure(@path) unless @config['debug']
       end
     end
 
@@ -298,8 +299,8 @@ module ReVIEW
         @converter.convert(filename + '.re', File.join(@path, filename + '.tex'))
       rescue => e
         @compile_errors = true
-        warn "compile error in #{filename}.tex (#{e.class})"
-        warn e.message
+        error "compile error in #{filename}.tex (#{e.class})"
+        error e.message
       end
     end
 
@@ -307,16 +308,26 @@ module ReVIEW
     #
     def copy_images(from, to)
       return unless File.exist?(from)
+
       Dir.mkdir(to)
-      ReVIEW::MakerHelper.copy_images_to_dir(from, to)
+      if @config['pdfmaker']['use_symlink']
+        ReVIEW::MakerHelper.copy_images_to_dir(from, to, use_symlink: true)
+      else
+        ReVIEW::MakerHelper.copy_images_to_dir(from, to)
+      end
     end
 
     def make_custom_page(file)
+      if file.nil?
+        return nil
+      end
+
       file_sty = file.to_s.sub(/\.[^.]+\Z/, '.tex')
       if File.exist?(file_sty)
-        return File.read(file_sty)
+        File.read(file_sty)
+      else
+        warn "File #{file_sty} is not found."
       end
-      nil
     end
 
     def join_with_separator(value, sep)
@@ -435,6 +446,10 @@ module ReVIEW
           end
       end
 
+      if @config['coverimage'] && !File.exist?(File.join(@config['imagedir'], @config['coverimage']))
+        raise ReVIEW::ConfigError, "coverimage #{@config['coverimage']} is not found."
+      end
+
       @locale_latex = {}
       part_tuple = I18n.get('part').split(/%[A-Za-z]{1,3}/, 2)
       chapter_tuple = I18n.get('chapter').split(/%[A-Za-z]{1,3}/, 2)
@@ -445,36 +460,47 @@ module ReVIEW
       @locale_latex['postchaptername'] = chapter_tuple[1].to_s
       @locale_latex['preappendixname'] = appendix_tuple[0].to_s
       @locale_latex['postappendixname'] = appendix_tuple[1].to_s
-    end
 
-    def erb_content(file)
-      @texcompiler = File.basename(@config['texcommand'], '.*')
-      erb = ReVIEW::Template.load(file, '-')
-      @logger.debug("erb processes #{File.basename(file)}") if @config['debug']
-      erb.result(binding)
+      if @config['pdfmaker']['boxsetting']
+        begin
+          @boxsetting = ReVIEW::LaTeXBox.new.tcbox(@config)
+        rescue ReVIEW::ConfigError => e
+          error! e
+        end
+      end
     end
 
     def latex_config
-      result = erb_content(File.expand_path('./latex/config.erb', ReVIEW::Template::TEMPLATE_DIR))
+      result = ReVIEW::Template.generate(path: './latex/config.erb', mode: '-', binding: binding)
       local_config_file = File.join(@basedir, 'layouts', 'config-local.tex.erb')
       if File.exist?(local_config_file)
         result << "%% BEGIN: config-local.tex.erb\n"
-        result << erb_content(local_config_file)
+        result << ReVIEW::Template.generate(path: 'layouts/config-local.tex.erb', mode: '-', binding: binding, template_dir: @basedir)
         result << "%% END: config-local.tex.erb\n"
       end
       result
     end
 
     def template_content
-      template = File.expand_path('./latex/layout.tex.erb', ReVIEW::Template::TEMPLATE_DIR)
+      template_dir = ReVIEW::Template::TEMPLATE_DIR
       if @config.check_version('2', exception: false)
-        template = File.expand_path('./latex-compat2/layout.tex.erb', ReVIEW::Template::TEMPLATE_DIR)
+        template_path = './latex-compat2/layout.tex.erb'
+      else
+        template_path = './latex/layout.tex.erb'
       end
       layout_file = File.join(@basedir, 'layouts', 'layout.tex.erb')
       if File.exist?(layout_file)
-        template = layout_file
+        template_dir = @basedir
+        template_path = 'layouts/layout.tex.erb'
+      end
+      ReVIEW::Template.generate(path: template_path, mode: '-', binding: binding, template_dir: template_dir)
+    rescue => e
+      if defined?(e.full_message)
+        error! "template or configuration error: #{e.full_message(highlight: false)}"
+      else
+        # <= Ruby 2.4
+        error! "template or configuration error: #{e.message}"
       end
-      erb_content(template)
     end
 
     def copy_sty(dirname, copybase, extname = 'sty')
@@ -486,26 +512,19 @@ module ReVIEW
       Dir.open(dirname) do |dir|
         dir.sort.each do |fname|
           next unless File.extname(fname).downcase == '.' + extname
+
           FileUtils.mkdir_p(copybase) unless Dir.exist?(copybase)
           if extname == 'erb'
             File.open(File.join(copybase, fname.sub(/\.erb\Z/, '')), 'w') do |f|
               f.print erb_content(File.join(dirname, fname))
             end
+          elsif @config['pdfmaker']['use_symlink']
+            FileUtils.ln_s(File.join(dirname, fname), copybase)
           else
             FileUtils.cp(File.join(dirname, fname), copybase)
           end
         end
       end
     end
-
-    def call_hook(hookname)
-      return if !@config['pdfmaker'].is_a?(Hash) || @config['pdfmaker'][hookname].nil?
-      hook = File.absolute_path(@config['pdfmaker'][hookname], @basedir)
-      if ENV['REVIEW_SAFE_MODE'].to_i & 1 > 0
-        warn 'hook configuration is prohibited in safe mode. ignored.'
-      else
-        system_or_raise(hook, Dir.pwd, @basedir)
-      end
-    end
   end
 end