Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

isshue #6 残り #9

Open
yutopia opened this issue May 8, 2016 · 7 comments
Open

isshue #6 残り #9

yutopia opened this issue May 8, 2016 · 7 comments

Comments

@yutopia
Copy link
Contributor

yutopia commented May 8, 2016

#6 対応の #7 で棚上げした ruby ツールについて。

  • dic-it2skk.rb 動作不良?
  • ctdicconv.rb 該当辞書ファイルがみつからない
  • chasen2skk.rb 他の環境で動作確認の必要あり?
@kyohsuke
Copy link
Member

kyohsuke commented May 9, 2016

  • dic-it2skk.rb 動作不良?

dic-it2skk.rb は辞書を見つけられずコードをそのまま 2.2.5 対応かけただけなので、
辞書を探していただいた今ならこちらで確認できます。
これは直します。

  • ctdicconv.rb 該当辞書ファイルがみつからない

これ見つからないです、ほんと見つからないです。

  • chasen2skk.rb 他の環境で動作確認の必要あり?

これが一番頭の痛い問題ですね。
元々のコードが euc-jp 決め打ちの状態になっていて、
その形で euc-jp を食わせて euc-jp の skk-jisyo 形式を吐き出すのは確認したのですが、
先のお話であったように sjis も utf8 も環境によっては吐き出すので、
その部分をどうするかが難しいです。

@yutopia
Copy link
Contributor Author

yutopia commented May 9, 2016

コーディングについてシンプルな方法として、euc-jp -> euc-jp をデフォにしておいて、他のコーディングについては作業者がオプションで明示的に指示するようにするとか考えていたのですが。
つまり、オプション無しのとき euc-jp -> euc-jp
オプション -f (変換元)、 -t (変換先) とか。

@hattya
Copy link

hattya commented May 11, 2016

ctdicconv.rbSKK-JISYO.china_taiwan を生成するために使われています。

http://openlab.ring.gr.jp/skk/skk/dic/Makefile

@yutopia
Copy link
Contributor Author

yutopia commented May 11, 2016

@hattya ありがとうございます。
http://openlab.ring.gr.jp/skk/dev-ja.html の方法で cvs 経由で skk/dic から取って来れるということですね。

ついでに調べてみたのですが、openlab の ruby は現状では 2.1.8 みたいです。cron で定期的に make されてるのかなぁ。
配布辞書のメンテに skktool が使われてるのは恥かしながら知りませんでした。
ということは、skktools は github の更新内容を openlab に反映しないといけないということですね。

@yutopia
Copy link
Contributor Author

yutopia commented May 11, 2016

skk/skktools は今は削除されているので、cron が動作していたとしても skktools は更新されていません。
上のMakefile の中の ../tools ディレクトリをのぞいてみたのですが、ruby ファイルがなかったので、上記の Makefile でメンテされてるはずの辞書の一部がアップデートされてない可能性があるみたいです。
もしかして、これって大きな問題なんでは。。
新たに issue 立てときます。MLにも流さないといけないか。

tats added a commit that referenced this issue May 13, 2016
This fixes error of `make SKK-JISYO.china_taiwan` in skk/dic.
cf. #9
@tats
Copy link
Member

tats commented May 13, 2016

skk/dicでのmake SKK-JISYO.china_taiwanで下記のようにエラーになる
のを修正しました。cf. 8606481

$ make SKK-JISYO.china_taiwan TOOLS_DIR=~/tmp/skktools
ruby -I /home/tats/tmp/skktools/filters /home/tats/tmp/skktools/convert2skk/ctdicconv.rb csv/china_taiwan.csv > SKK-JISYO.tmp
/home/tats/tmp/skktools/convert2skk/ctdicconv.rb:108:in `tr!': incompatible character encodings: EUC-JIS-2004 and UTF-8 (Encoding::CompatibilityError)
        from /home/tats/tmp/skktools/convert2skk/ctdicconv.rb:108:in `block in <main>'
        from /home/tats/tmp/skktools/convert2skk/ctdicconv.rb:66:in `foreach'
        from /home/tats/tmp/skktools/convert2skk/ctdicconv.rb:66:in `<main>'
Makefile:64: recipe for target 'SKK-JISYO.china_taiwan' failed
make: *** [SKK-JISYO.china_taiwan] Error 1

@yutopia
Copy link
Contributor Author

yutopia commented May 14, 2016

@tats ありがとうございます。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants