[Freewnn-users 184] CVS版FreeWnnに対するpatchを作りました (was: 1xx さんを開発メンバーに加えました)

Back to archive index

1xx itsan****@gmail*****
2013年 9月 2日 (月) 12:28:48 JST


2013年9月1日 7:14 1xx <itsan****@gmail*****>:
> 2013年8月31日 22:40 Tomoki AONO <aono****@cc*****>:
>> <CANW2+ivBWJf6TunBQH7us7EXDPdK3J02wRRb_ZACPPm23gdr_g****@mail*****>の記事において
>> itsan****@gmail*****さんは書きました。
>
>>>> > cvs -z3 -d:ext:itisa****@cvs*****:/cvsroot/freewnn co FreeWnn
>>>> Permission denied (publickey).
>>>> cvs [checkout aborted]: end of file from server (consult above messages if any)
>>>> というerrorが出ております。

>>
>> もしかして、sshの公開鍵ファイルを作成していないか、あるいは
>> Web UIでご自身の公開鍵(ex. ~/.ssh/id_rsa.pub)を登録していな
>> いということはないでしょうか。試しにshell.sourceforge.jpで
>> ssh -vしてみたところでは登録してない状態だと1xxさんのような
>> ログになりました。
>> #その後公開鍵を登録して接続できましたが、その際はWebUIの不
>> #可解な挙動はありませんでした。
>>
>> もし登録されてないのであれば、以下の説明を参考にしてくださ
>> い。違ってたらすみません。
>> http://sourceforge.jp/docs/%E5%85%AC%E9%96%8B%E9%8D%B5%E3%81%AE%E4%BD%9C%E6%88%90%E3%81%A8%E7%99%BB%E9%8C%B2
>
> いえ、Webからの公開鍵の登録は実行したのですが、
> それが何時まで経っても反映されないような挙動をしています。
>
> この鍵を使ってGitHubや別のserverにはlogin出来ているので、
> 鍵が壊れているのではないと思うのですが、原因が判りません。
> sourceforge.jpには連絡済みですので、回答を待ちたいと思います。
>
> http://sourceforge.jp/ticket/browse.php?group_id=1&tid=31956
>
> 私の単純missなら良いのですが。

相変わらずSSH接続できない状態です。
土日を挟んだからかsourceforge.jpからはまだ回答がありません。

そこで、openSUSEのFreeWnnに当てたpatchの内、
directory構成など
openSUSE固有の事情があるものを除いたpatch群を作り直しました。
#といっても、patchが当たらなかったのは1ヶ所だけでしたが。
CVSからcheckoutした現行のソースに対して適応できます。
このpatchを適応したCVS版FreeWnnは
openSUSE13.1 Milestone 4 64bit環境上で動作しております。

https://dl.dropboxusercontent.com/u/86335040/FreeWnn/patch.tar.gz
からdownloadできます。
また、tarballの中にpatchを一気に当てるshell scriptを入れました。
patchを当てる順番はshell scriptを参照していただければ判ると思います。

FreeWnn projectのdirectoryの隣にpatchを展開します。
例えば
~/src/FreeWnn/
がFreewnn projectのdirectoryだとすると、
cd ~/src
tar xvzf patch.tar.gz
で
~/src/patch
というdirectoryができます。
cd ~/src/FreeWnn
でcurrent directoryを変更して
../patch/patch_all1.sh
とshell scriptを実行すれば適応できます。

皆さんで試していただいていただけると幸いです。 >all

また、よろしければCVSのtrunkにmergeしてください。 >青野さん


これらのpatchの基になった
openSUSE固有のpatchは
https://build.opensuse.org/package/show/M17N/fwnn
から取得できます。

今回の作業でopenSUSEのpatchを見直しました。
fwnn.spec
https://build.opensuse.org/package/view_file/M17N/fwnn/fwnn.spec?expand=1
の中で、
Wnn/jserver/kaiseki.h を
Wnn/include/ に移動する作業をしていましたが、
今回の見直しで、この作業が必要ないことが判りましたので、
やめています。

石川さんの修正source
https://bitbucket.org/zephyrus00jp/freewnn-1.1.1-a022-debian-0827
も取り込もうかと思ったのですが、
pointerのcastが怖くて取り込めませんでした。

以下、patchを当てる際に作ったgit logです。
ChangeLogの叩き台にしていただければ幸いです。

-------- BEGIN --------
commit 414a238a04a2a67a0b11cc0695e28d546db0d4d3
Author: Mitsu <bkbin****@rinku*****>
Date:   Mon Sep 2 10:25:42 2013 +0900

    Patched: FreeWnn-w_char_p.patch

    found type missmatches in main() .
    Old code gave "" to js_dic_file_create(),
    but js_dic_file_create()'s argument has a w_char * type .
    So added w_char W_CHAR_NUL_STR[] = { 0 };
    and gave it to js_dic_file_create() .

    modified:   Wnn/jutil/wdreg.c

commit e5814d8465acaa8705adf5a74fc8a9c9aa929925
Author: Mitsu <bkbin****@rinku*****>
Date:   Mon Sep 2 10:18:40 2013 +0900

    Patched: FreeWnn-prototypes_3-Makefile.in.patch

    Added header files .
    So changed Makefile.in .

    modified:   cWnn/include/Makefile.in
    modified:   cWnn/jserver/Makefile.in
    modified:   kWnn/include/Makefile.in

commit b637e6300c0c5780025a546c0145280fadd03ed6
Author: Mitsu <bkbin****@rinku*****>
Date:   Mon Sep 2 09:54:58 2013 +0900

    Patched: FreeWnn-prototypes_3-excludeMakefile.in.patch

    Date: 2013-08-17 14:07:45 +0900
    Subject: input: prototype declarations for reducing compilation warnings
    References: http://lists.opensuse.org/opensuse-ja/2013-08/msg00035.html
    Upstream: sent

    1. prototype declarations for reducing compilation warnings
    2. I found a bug in make_hinsi_list() at Wnn/jserver/hinsi_list.c,
      then fixed it .
    3. 2013-09-01 exclude patches of Makefile.in

    new file:   Wnn/include/conv.h
    new file:   Wnn/include/etc.h
    new file:   Wnn/include/jlib_hide.h
    new file:   Wnn/include/jutil.h
    new file:   Wnn/include/romkan.h
    modified:   Wnn/conv/cvt_key.c
    modified:   Wnn/conv/cvt_read.c
    modified:   Wnn/etc/bdic.c
    modified:   Wnn/etc/dic_atojis.c
    modified:   Wnn/etc/gethinsi.c
    modified:   Wnn/etc/hindo.c
    modified:   Wnn/etc/msg.c
    modified:   Wnn/etc/pwd.c
    modified:   Wnn/etc/revdic.c
    modified:   Wnn/etc/sstrings.c
    modified:   Wnn/etc/xutoj.c
    modified:   Wnn/etc/yincoding.c
    modified:   Wnn/include/jdata.h
    modified:   Wnn/include/jllib.h
    modified:   Wnn/include/jslib.h
    modified:   Wnn/include/msg.h
    modified:   Wnn/jlib.V3/jlv3.c
    modified:   Wnn/jlib/jl.c
    modified:   Wnn/jlib/js.c
    modified:   Wnn/jserver/daibn_kai.c
    modified:   Wnn/jserver/de.c
    modified:   Wnn/jserver/de_header.h
    modified:   Wnn/jserver/do_dic_env.c
    modified:   Wnn/jserver/do_filecom.c
    modified:   Wnn/jserver/do_henkan.c
    modified:   Wnn/jserver/do_henkan1.c
    modified:   Wnn/jserver/do_hindo_s.c
    modified:   Wnn/jserver/fzk.c
    modified:   Wnn/jserver/get_kaiarea.c
    modified:   Wnn/jserver/hinsi_list.c
    modified:   Wnn/jserver/jbiki.c
    modified:   Wnn/jserver/jbiki_b.c
    modified:   Wnn/jserver/jikouho.c
    modified:   Wnn/jserver/jikouho_d.c
    modified:   Wnn/jserver/jishoop.c
    modified:   Wnn/jserver/jishosub.c
    modified:   Wnn/jserver/mknode0.c
    modified:   Wnn/jserver/mknode1.c
    modified:   Wnn/jserver/mknode2.c
    modified:   Wnn/jserver/rd_jishoop.c
    modified:   Wnn/jserver/readfile.c
    modified:   Wnn/jserver/sisheng.c
    modified:   Wnn/jserver/w_string.c
    modified:   Wnn/jutil/atod.c
    modified:   Wnn/jutil/atof.c
    modified:   Wnn/jutil/atorev.c
    modified:   Wnn/jutil/dic_head.c
    modified:   Wnn/jutil/dtoa.c
    modified:   Wnn/jutil/oldatonewa.c
    modified:   Wnn/jutil/ujisf.c
    modified:   Wnn/jutil/wddel.c
    modified:   Wnn/jutil/wdreg.c
    modified:   Wnn/jutil/wnnkill.c
    modified:   Wnn/jutil/wnnstat.c
    modified:   Wnn/jutil/wnntouch.c
    modified:   Wnn/romkan/rk_bltinfn.c
    modified:   Wnn/uum/basic_op.c
    modified:   Wnn/uum/header.c
    modified:   Wnn/uum/history.c
    modified:   Wnn/uum/inspect.c
    modified:   Wnn/uum/jhlp.c
    modified:   Wnn/uum/jikouho.c
    modified:   Wnn/uum/jis_in.c
    modified:   Wnn/uum/jutil.c
    modified:   Wnn/uum/key_bind.c
    modified:   Wnn/uum/kuten.c
    modified:   Wnn/uum/printf.c
    modified:   Wnn/uum/prologue.c
    modified:   Wnn/uum/screen.c
    modified:   Wnn/uum/sdefine.h
    modified:   Wnn/uum/select_ele.c
    modified:   Wnn/uum/sheader.h
    modified:   Wnn/uum/termio.c
    modified:   Wnn/uum/touroku.c
    modified:   Wnn/uum/ttyfdslot.c
    modified:   Wnn/uum/uif.c
    modified:   Wnn/uum/wnnrc_op.c

commit c72c79757dfa34c0167dc0a14a930a862f700d5f
Author: Mitsu <bkbin****@rinku*****>
Date:   Mon Sep 2 09:40:37 2013 +0900

    Patched: FreeWnn-prototypes_msg.patch

    Date: 2013-08-09 09:51:15 +0900
    Subject: input: prototype declarations for bugs about error messages .
    References: http://lists.opensuse.org/opensuse-ja/2013-08/msg00014.html
    Upstream: sent

    Bugfix: In openSUSE 13.1 Milestone 3,
    uum could not get error messages .
    This issue's cause is that we forgot prototype declaration
    of msg_get() in Wnn/include/msg.h .
    Then I added "FreeWnn-prototypes_msg.patch" as %Patch15 .
    http://lists.opensuse.org/opensuse-ja/2013-08/msg00014.html
    http://sourceforge.jp/projects/freewnn/lists/archive/users/2013-August/000140.html

    modified:   Wnn/include/msg.h

commit 4dc56672245126fc0d3bec1c62f136fe1eab3a17
Author: Mitsu <bkbin****@rinku*****>
Date:   Mon Sep 2 09:26:04 2013 +0900

    Patched: FreeWnn-prototypes.patch

    add many prototype declarations .

    modified:   Wnn/include/jllib.h
    modified:   Wnn/include/jslib.h
    modified:   Wnn/include/wnn_string.h
    modified:   Wnn/jlib/jl.c
    modified:   Wnn/jserver/error.c
    modified:   Wnn/jutil/atod.c
    modified:   Wnn/romkan/rk_bltinfn.c
    modified:   Wnn/romkan/rk_fundecl.h
    modified:   Wnn/romkan/rk_main.c
    modified:   Wnn/romkan/rk_modread.c
    modified:   Wnn/romkan/rk_read.c
    modified:   Xwnmo/xwnmo/do_socket.c

commit bb7ba4e3f096e1c976f2409dbcefa3c1022ed865
Author: Mitsu <bkbin****@rinku*****>
Date:   Mon Sep 2 09:10:23 2013 +0900

    Patched: FreeWnn-include.patch

    added header files <string.h> and <unistd.h> .

    modified:   PubdicPlus/ctopd.c
    modified:   PubdicPlus/wtopd.c

commit 7f97794d9fed3a9ffc6ca374883c2ac9db8a8d2b
Author: Mitsu <bkbin****@rinku*****>
Date:   Mon Sep 2 08:56:05 2013 +0900

    Patched: FreeWnn-warnings.patch

    fixed casts in jd_dicadd() .

    modified:   Wnn/jlib.V3/jlv3.c

commit 4081a7bc856473b03b4dbfd651c9e01f5dcb9f01
Author: Mitsu <bkbin****@rinku*****>
Date:   Mon Sep 2 08:44:29 2013 +0900

    Patched: FreeWnn-s390x.patch

    changed some types of variables .

    modified:   Wnn/jserver/error.c
    modified:   Xwnmo/xwnmo/do_socket.c

commit ba0a8a9c61bbf9685189f01d59928b95da204ced
Author: Mitsu <bkbin****@rinku*****>
Date:   Mon Sep 2 08:14:32 2013 +0900

    Patched: FreeWnn-1.1.1-a022.dif - modified some definication .

    modified definication of message_out() .
    modified argument's types
      jl_connect_lang(), jl_open_lang(), create_pwd_file(),
      jl_set_env_wnnrc(), jl_set_env_wnnrc1(), create_file(),
      make_dir_rec1(), make_dir1(), call_error_handler(),
      message_out() ...
    fixed casts .

    modified:   Wnn/jlib/jl.c
    modified:   Wnn/jlib/js.c

commit d7c8b95ceeefae9c25111ebbab43cbe6322e64f2
Author: Mitsu <bkbin****@rinku*****>
Date:   Mon Sep 2 08:05:43 2013 +0900

    Patched: FreeWnn-sighandler.patch - signal handling .

    Added signal handler .
    Server's exit status changed .

    modified:   Wnn/jserver/de.c

commit 859c4b47cac9cc1bd51948d7867fa166077afd53
Author: Mitsu <bkbin****@rinku*****>
Date:   Sun Sep 1 09:26:42 2013 +0900

    First version FreeWnn-1.1.1-a022
-------- END --------

>
> ただ現状cvs serverに繋がらないこともあり、
> どういう運用が妥当か、模索する必要がありますね。

>
>>
>>>> > - FreeWnn-trad_cpp.patchはもういらないと思います。(やってい
>>>> >   ることは同じはずです。)ただ、Debian本家のパッケージで失敗
>>>> >   しているので並列makeが動くような修正を加える予定です。
>
>>
>> 念のためopenSUSEでお試しになってください(多分支障はないと思
>> いますが)。
>
> その予定です。

>>
>>>> > - できればcommitされる際は機能別に分割してもらえると検証も
>>>> >   しやすいのでありがたいです。(ex. 明らかなバグ、NULLと0の
>>>> >   入れ替え、各種プロトタイプ宣言など)
>>>>
>>>> patchの当て方が五月雨式になっていて申し訳ありません。
>>>> patchが歴史を背負ってしまっているのと、
>>>> 私自身が不慣れなことで、あのような形になってしまいました。
>>
>> できればお願いします。(別ブランチで試されるなら適宜こちらで
>> 取捨選択はできるかな…?)
>
> やってみます。

すいません。
patchを機能毎に分けようと試みたのですが、
prototype宣言などに一旦変更をかけてしまうと
大きくsourceを修正するまでcompileが通らないことが判明しましたので、
機能別分解は断念しました。

-- 
1xx <ItSAN****@gmail*****> <https://twitter.com/ItSANgo>
<http://d.hatena.ne.jp/Itisango/>




freewnn-users メーリングリストの案内
Back to archive index