review-jlreqにおけるgentombow.sty/jsbook.cls

[m-shibata]

review-init --latex-template=review-jlreqでビルドすると、styディレクトリに「gentombow.sty」と「jsbook.cls」がコピーされますがこれは必要でしょうか？

A. 不要なもので、コピーしないようにするのが望ましい。
B. 不要ではあるが、何らかの理由によりわざとreview-jsbookと同様にコピーしている。
C. 必要なもの。

あって困るものでもないので、今のままでも特に問題となるわけではないのですが、A.のように不要なら削除してしまったほうが混乱がないかなと思い、報告させていただきました。

review/lib/review/init.rb

Lines 163 to 173 in c6a103c

	def generate_texmacro(dir)
	texmacrodir = File.join(dir, 'sty')
	FileUtils.mkdir_p(texmacrodir)
	tdir = File.join(@review_dir, 'templates/latex', @template)
	@logger.error "#{tdir} not found." unless File.exist?(tdir)
	FileUtils.cp(Dir.glob(File.join(tdir, '*.*')), texmacrodir)
	# provide jsbook from vendor/. current version is 2018/06/23
	FileUtils.cp(File.join(@review_dir, 'vendor/jsclasses/jsbook.cls'), File.join(texmacrodir, 'jsbook.cls'))
	# provide gentombow from vendor/. current version is 2018/08/30 v0.9j
	FileUtils.cp(File.join(@review_dir, 'vendor/gentombow/gentombow.sty'), File.join(texmacrodir, 'gentombow.sty'))
	end

もし修正するとなると、上記においてif @template == 'review-jsbook'のとき（もしくはif @template != 'review-jlreqのとき）にのみ、FileUtils.cpを呼ぶ形の変更になるかと思います。

[kmuto]

そうですね、==のときにcp、は自然なのでそうします。