[debian-devel:17663] yatex (1.73+dfsg1-6) mentors package review comments

[Junichi Uekawa <dancer@xxxxxxxxxxxxx>]

(debian-devel にスレッドを移動しました。)

倉敷さん
こんにちは、

yatex パッケージ拝見しました。2点コメントがあります。


> > http://mentors.debian.net/cgi-bin/sponsor-pkglist?action=details;package=yatex
> > をみるとlintian warning がありますね。
> > 直した方がよいかも。
> 
> 先ほど、lintian clean にしたものを再度 put しました。
> よろしくお願いします。

1. 
ここらへんのpostinst/prerm にコメントアウトされているやつの意味がよくわからないです。
コメントアウトするくらいなら削除してよいと思います。

--- yatex-1.73+dfsg1/debian/postinst
+++ yatex-1.73+dfsg1/debian/postinst
@@ -17,21 +17,21 @@
 #             /usr/lib/emacsen-common/emacs-package-install $PACKAGE
 #      fi
 
-       if [ -x /usr/sbin/install-info ]
-         then
-           install-info --quiet --section "Emacs" --menuentry="YaTeX(en)" \
-             --description="Yet Another LaTeX mode(English)." \
-             /usr/info/yatexe.gz
-           install-info --quiet --section "Emacs" --menuentry="YaTeX(ja)" \
-             --description="Yet Another LaTeX mode(Japanese)." \
-             /usr/info/yatexj.gz
-           install-info --quiet --section "Emacs" --menuentry="YaHTML(en)" \
-             --description="Yet Another HTML mode(English)." \
-             /usr/info/yahtmle.gz
-           install-info --quiet --section "Emacs" --menuentry="YaHTML(ja)" \
-             --description="Yet Another HTML mode(Japanese)." \
-             /usr/info/yahtmlj.gz
-       fi
+#      if [ -x /usr/sbin/install-info ]
+#        then
+#          install-info --quiet --section "Emacs" --menuentry="YaTeX(en)" \
+#            --description="Yet Another LaTeX mode(English)." \
+#            /usr/info/yatexe.gz
+#          install-info --quiet --section "Emacs" --menuentry="YaTeX(ja)" \
+#            --description="Yet Another LaTeX mode(Japanese)." \
+#            /usr/info/yatexj.gz
+#          install-info --quiet --section "Emacs" --menuentry="YaHTML(en)" \
+#            --description="Yet Another HTML mode(English)." \
+#            /usr/info/yahtmle.gz
+#          install-info --quiet --section "Emacs" --menuentry="YaHTML(ja)" \
+#            --description="Yet Another HTML mode(Japanese)." \
+#            /usr/info/yahtmlj.gz
+#      fi
 
     ;;


2. 
yatex-init.el に追加された設定ですが、changelogに

+  * yatex-init.el: set YaTeX-inhibit-prefix-key to t. (Closes: #480160)
+                   set YaTeX-kanji-code to nil. (Closes: #504009)
とだけ書いてありますね。
それぞれどういう意味なのか、ぱっとみ意味がわからないので、changelog に書き下してみてはいかがでしょうか?



--- yatex-1.73+dfsg1/debian/yatex-init.el
+++ yatex-1.73+dfsg1/debian/yatex-init.el
@@ -105,6 +105,13 @@
 ;(setq yahtml-image-viewer "xv") ; upstream default - unavailable
 (setq yahtml-image-viewer "gimv") ; free alternative for Debian
 
+;; YaTeX kanji-code
+;;
+(setq YaTeX-kanji-code nil)
+
+;; YaTeX prefix-key
+(setq YaTeX-inhibit-prefix-letter t)
+
 
 
                    ))


上川
-- 
dancer@{netfort.gr.jp,debian.org}