tDiary

ただただしです。

2008/11/27 Kazuhiro NISHIYAMA <zn@...>:
> 詳細は http://znz.s1.xrea.com/t/?date=20081127#p01 に書いたのですが、
> antirefspamのcomment_filterの中で例外が発生する状況になっていて、
> TDiary::TDiaryComment::newでのdefaultioのLOCK_EXが残ってしまったまま
> TDiary::TDiaryLatest::newで再度同じファイルを開いてロックしようとして
> デッドロックになってしまうようです。

defaultio.rbのこの部分はちょっと汚いので、これを機に整理しよう
と思い、パッチを検討してみたんですが、これ、かなり古いバージョン
に見えます。2.2系だとは思いますが、いつのファイル向けのパッチ
でしょう?

-- 
♪ツッコミは、短く鋭く愛を込めて。
ただただし <http://tdtds.jp/>
※メールアドレスが変わりました※

西山和広です。

At Fri, 28 Nov 2008 23:16:08 +0900,
TADA Tadashi wrote:
> 
> defaultio.rbのこの部分はちょっと汚いので、これを機に整理しよう
> と思い、パッチを検討してみたんですが、これ、かなり古いバージョン
> に見えます。2.2系だとは思いますが、いつのファイル向けのパッチ
> でしょう?

Debianパッケージでインストールしたものなので、2.0.2+20060303-5です。

一応trunkの方もopenのところが同じような書き方になっているのは
確認していたのですが、closeはensureになっているのは見落として
いました。


-- 
|ZnZ(ゼット エヌ ゼット)
|西山和広(Kazuhiro NISHIYAMA)

ただただしです。

2008/11/29 Kazuhiro NISHIYAMA <zn@...>:
> Debianパッケージでインストールしたものなので、2.0.2+20060303-5です。
>
> 一応trunkの方もopenのところが同じような書き方になっているのは
> 確認していたのですが、closeはensureになっているのは見落として
> いました。

なるほど。じゃあ、2.2と2.3では、今回報告された問題は発生しない
と考えていいですかね。

ブロックにしたいとは思うんだけど、いろいろ制約があって……。

-- 
♪ツッコミは、短く鋭く愛を込めて。
ただただし <http://tdtds.jp/>
※メールアドレスが変わりました※

ただただしです。

2008/11/26 Keisuke Nomura <knomura.1394@...>:
> 自分の環境でposttdiary.rbが動かなかったため、調査しました。
>
> 環境は、
> Generated by tDiary version 2.3.1
> Powered by Ruby version 1.8.5
> な感じです。
>
> 結論としては、GetoptLongのパラメタが凍結されていて、
> それを変更しようとしているのがまずいようです。

なるほど。オプションは外から来たものだからfreezeされるように
なってたのかな……?

> GetoptLongのメンテナンスの状況がよくわからないなーというのと、
> optparseを使った方がいいのかなーというところです。

1.9にも入っているので、問題ないでしょう。オプション処理なん
てそうそう変わるものじゃないし。コミットしてもらえますか?

-- 
♪ツッコミは、短く鋭く愛を込めて。
ただただし <http://tdtds.jp/>
※メールアドレスが変わりました※

野村です。

>> 結論としては、GetoptLongのパラメタが凍結されていて、
>> それを変更しようとしているのがまずいようです。
>
> なるほど。オプションは外から来たものだからfreezeされるように
> なってたのかな……?
>

結局、これがどのバージョンからそうなっていたのかはわかりませんでしたが、
コミットしました。

ただただしです。

2008/11/16 SHIBATA Hiroshi <shibata.hiroshi@...>:
> ruby1.9対応をしている途中に見つけたのですがtDiaryの
> index.rbとupdate.rbの先頭にある$defoutはruby1.8では
> すでにobsoleteになっているそうです。

あー、そういえば残ってたか……。

> ruby1.9対応をしていくうえで$defoutは$stdoutに今から置き換えて
> しまおうと思いますが、問題ないでしょうか?

良いと思います。

-- 
♪ツッコミは、短く鋭く愛を込めて。
ただただし <http://tdtds.jp/>
※メールアドレスが変わりました※

すのこです。
    |     wrote : "TADA Tadashi" <t@...>
    |   subject : Re: [tDiary-devel] 2.3.1リリースしました
    |Message-Id : <f7dffd10811120036p6a9ce012weca379570df79079@...>

> キャッシュを使わないようにすると、毎回*.tdcファイルからツッコミを
> 読み込むことになるんですが、そのときにtb-show.rbが上書きしている
> 機能を使ってしまうのが原因でした。でもここは2.2から変わってないの
> で、なんか変なんですけどね……。
> 
> というわけで、tb-show.rbを最新にして様子を見てもらえますか?

trunkにして試してみました。
ツッコミを入れてもTBは問題なく表示されたままになしました。

tb-show.rb の修正箇所を見てみましたが、確かに、これって、2.2 
でも起きても良いような気がしますね。
で、version 2.2.1.20080317 で試してみたのですが、全然問題な
いですねぇ(^^;) 不思議です。

とりあえず、2.3.1 で私の環境で発生した不具合は、これで全て解
決しました。

感覚として、2.3.1 はかなり安定しているように感じます。

ありがとうございました。

------------------------------------------------------------
                           sunoko
                 mailto:sunoko@...

こんにちは　不断です

報告するのをすっかり忘れていました。

On Sun, 2 Nov 2008 08:47:05 +0900,
In article <11f7f3e60811011647w5a71a922o3f63e427e1a7e99a@...>,
"KOSAKA Takashi" <takosaka@...> wrote:
> Plugin error in '00default.rb'.
> private method `scan' called for nil:NilClass
> (plugin/00default.rb):20:in `load_plugin'
(snip)
> http://www.nanashinonozomi.com/tdiary/20081031.html#p02

この問題ですが、件のエントリを書いてから現在まで、状況に変化
はありません。

環境は、

tDiary 2.3.1
Ruby 1.8.6-p287
Apache 1.3.39 (Unix)
さくらのレンタルサーバ

です。

-- 
不断
mailto:joker@...

こんにちは　不断です

tdiary/plugin/00default.rb の

19:		when 'month'
20:			Time::local( *@cgi.params['date'][0].scan(/^(\d{4})(\d\d)/ ).flatten )

が問題になっているようなので、ここを通らないようにすればいい
のではないか、ということで場当たり的な対応をしてみました。
また、一部、 EUC-JP で処理をするようになっている部分が残って
いたようなので、合わせて修正してみました。

とりあえず、これで手元の環境では、エラーは出なくなっているよ
うですが、適切な修正かどうかは分かりません(^^;

-- 
不断
mailto:joker@...

こさかです．

> とりあえず、これで手元の環境では、エラーは出なくなっているよ
> うですが、適切な修正かどうかは分かりません(^^;

不断さんのパッチで，エラーが出なくなりました．
助かりました．ありがとうございました．

ちなみに，私も「さくらのレンタルサーバ」と
同環境です．

-- 
takosaka@...

おはようございます　不断です

On Sun, 2 Nov 2008 20:37:28 +0900,
In article <11f7f3e60811020337u503a8215p4e1f2339da2fc29d@...>,
"KOSAKA Takashi" <takosaka@...> wrote:
>>とりあえず、これで手元の環境では、エラーは出なくなっているよ
>不断さんのパッチで，エラーが出なくなりました．

お役に立てたようで何よりですが、わたしの怪しいパッ
チを適用して実稼働環境で運用するのは危険性が…。

開発陣の正式対応を待って頂いた方が良いのではないか
と思います(^^;;; 

-- 
不断
mailto:joker@...

柴田です。

文字コードの部分は不断さんの修正は必要ですので
先ほどコミットしておきました。

modeの部分は消してしまっても動くような気がするので
引数を消して試していただけないでしょうか?
動かない場合はnilを渡す方法を入れておこうと思います。

# 手元に動作確認ができる環境がないもので…。

-- 
SHIBATA Hiroshi e-mail:shibata.hiroshi@...
URL:http://www.hsbt.org/

こんばんは　不断です
# 引用の順序が前後しています。

On Mon, 3 Nov 2008 16:05:00 +0900,
In article <f7aeca990811022305y27a1da6jbf6ec9c0b1cb2ea@...>,
"SHIBATA Hiroshi" <shibata.hiroshi@...> wrote:
> modeの部分は消してしまっても動くような気がするので
> 引数を消して試していただけないでしょうか?
> 動かない場合はnilを渡す方法を入れておこうと思います。

手元で試した限りでは、404行目の「'mode' => 'month',」
について、

  ・「'mode' => '',」
  ・404行目をコメントアウト

のいずれの場合とも、特に問題なく動いているように見
えます。

> 文字コードの部分は不断さんの修正は必要ですので
> 先ほどコミットしておきました。

ありがとうございます。

ところで、

389: def tdiary2text(html)
390:  apply_tdiary_plugins(html).gsub(%r[<[^>]*>]em, '').slice(/\A.{0,120}/mu)
391: end

の gsub メソッドにおける正規表現部について、修正さ
れていないようですが、この部分の修正は不要だったと
いうことで OK でしょうか?

-- 
不断
mailto:joker@...

柴田です。

確認ありがとうございます。
contribを直しておきました。

> ところで、
>
> 389: def tdiary2text(html)
> 390:  apply_tdiary_plugins(html).gsub(%r[<[^>]*>]em, '').slice(/\A.{0,120}/mu)
> 391: end
>
> の gsub メソッドにおける正規表現部について、修正さ
> れていないようですが、この部分の修正は不要だったと
> いうことで OK でしょうか?

ただの見落としでしたので、こちらも直しました。

-- 
SHIBATA Hiroshi e-mail:shibata.hiroshi@...
URL:http://www.hsbt.org/

こんばんは　不断です

On Mon, 3 Nov 2008 18:51:30 +0900,
In article <f7aeca990811030151n5a7f65f7gf8f7ac5cb7a66492@...>,
"SHIBATA Hiroshi" <shibata.hiroshi@...> wrote:
> > 390:  apply_tdiary_plugins(html).gsub(%r[<[^>]*>]em, '').slice(/\A.{0,120}/mu)
> ただの見落としでしたので、こちらも直しました。

ありがとうございます。
こちらも合わせて修正を確認させて頂きました。

-- 
不断
mailto:joker@...