Backport #2572: Re: "require 'logge'" fails on ruby 1.8.7 from http://github.com/shyouhei/ruby

• Status: Closed
• Assignee: nahi (Hiroshi Nakamura)

[shyouhei (Shyouhei Urabe)]

=begin
卜部です。

r20345はほんとうにこれでいいんですかねえ? 落ちはしないだけで意味不明な感じにな
りそうなんですが...

MOROHASHI Kyosuke さんは書きました:

諸橋です。
いつもお世話になっております。

http://github.com/shyouhei/ruby

から取得したRubyの 、remotes/origin/ruby_1_8_7 ブランチをビルドして
loggerを require するとNoMethodErrorが発生します。

原因は、Logger::ProgName を設定する際にの csv/svn の $Id$ のキーワード
展開への依存だと思うのですが、これは、1.9ブランチと1.8ブランチでは
すでに修正されているようです。

これを1.8.7ブランチへバックポートしていただけないでしょうか?

1.9ブランチへの修正
http://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=rev&revision=20290

1.8ブランチへの修正
http://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=rev&revision=20345

とはいえ、1.8ブランチへの修正は、他の変更と一緒になってしまっているので
そのままでは取り込めないようにも思います。
どうすべきか、ご判断をお願いします。

ちなみに、私自身がやりたいこと自体は、下記でうかがったString#inspectの
修正を含む、Ruby 1.8.7が欲しいというモノです。

http://redmine.ruby-lang.org/issues/show/2569

自分でcherry-pickして使おうとしたところ、今回の問題に当たりました。

=end

[naruse (Yui NARUSE)]

• Status changed from Open to Assigned
• Assignee set to nahi (Hiroshi Nakamura)

=begin

=end

[moro (Kyosuke MOROHASHI)]

=begin
もろはしです。

2010年1月7日18:47 Urabe Shyouhei [e-mail]:

r20345はほんとうにこれでいいんですかねえ? 落ちはしないだけで意味不明な感じにな
りそうなんですが...

たしかにそうですね。
結局、空になってしまいそうです。
そういう意味では、1.9ブランチ向けのr20290のほうが良さそうですが、そちらを
バックポートするというのは、ポリシー的に大丈夫なのでしょうか?

--
諸橋 恭介(もろはし きょうすけ)

本を書きました!!
「Railsレシピブック」http://www.amazon.co.jp/dp/4797336625

mail : [e-mail]

はてな: http://d.hatena.ne.jp/moro/
twitter: http://twitter.com/moro

=end

[nahi (Hiroshi Nakamura)]

=begin
いまいち問題を把握してない可能性もありますが、gitが$Id$展開しないからコケるんだけどgitからcloneしてもそのまま動くようにしてくれよ、ということでいいですかね？

trunkやruby_1_8で直って、ということはないはずですが。

svn-git連携でうまいことできたりしないものなんでしょうか。

=end

[moro (Kyosuke MOROHASHI)]

=begin
もろはしです。

2010年1月8日16:19 NAKAMURA, Hiroshi [e-mail]:

いまいち問題を把握してない可能性もありますが、gitが$Id$展開しないからコケるんだけどgitからcloneしてもそのまま動くようにしてくれよ、ということでいいですかね？

はい。そうです。

trunkやruby_1_8で直って、ということはないはずですが。

trunkのほうはおなじような理由での修正だったみたいです。
"should handle the case that cvs/svn do not expand $Id keyword.
[ruby-core:19991]"

svn-git連携でうまいことできたりしないものなんでしょうか。

いちおう同様なキーワード展開の機能はあるようです。ただ、それぞれのscmで
$Id$の示すモノが違うというのがちょっと紛らわしいかな、とも思います。

http://progit.org/book/ja/ch7-2.html

このあたりを踏まえ、

• そもそもこれは修正対象か(githubのは自己責任でね、というスタンス)
• 直すとしてどう直すか?
  -- 別ブランチへのパッチ取り込み
  -- 上記.gitattributesやGitのフィルタを使うなど
  -- logger..rbの該当箇所だけ別途修正

など、誰に何をお願いすればよいのか悩んでいるため相談させていただきました。

=end

[nahi (Hiroshi Nakamura)]

=begin
なるほど、今気付いたんですが、trunk側は不自然なことになってますねえ。そのうちruby_1_8に合わせときます。

で、gitでどうするかですが、shyouheiさんに任せます。ruby_1_8に合わせとくと（偶然）エラーにはならなくなる（ProgNameは"/"になる）のでそれでもいいですし、他に手があればそれでもいいと思います。

=end

[nahi (Hiroshi Nakamura)]

• Assignee changed from nahi (Hiroshi Nakamura) to shyouhei (Shyouhei Urabe)

=begin

=end

[shyouhei (Shyouhei Urabe)]

=begin
悩み中。
=end

[nahi (Hiroshi Nakamura)]

• Status changed from Assigned to Closed
• Assignee changed from shyouhei (Shyouhei Urabe) to nahi (Hiroshi Nakamura)

r31806で、問題のタネごとなくしてしまいました。長らくご迷惑をおかけしました。

Index: lib/logger.rb

--- lib/logger.rb (revision 31805)
+++ lib/logger.rb (revision 31806)
@@ -170,8 +170,7 @@

class Logger
VERSION = "1.2.6"

• id, name, rev = %w$Id$
• ProgName = "#{name.chomp(",v")}/#{rev}"

• ProgName = "#{File.basename(FILE)}/#{VERSION}"

  class Error < RuntimeError; end
  class ShiftingError < Error; end