プロジェクト

全般

プロフィール

操作
リンクをコピー

バグ #423

完了

config.rbのバージョンのマッチが上手く動いていない

コスモ 虹川 さんがほぼ13年前に追加. 12年以上前に更新.

ステータス:
終了
優先度:
通常
担当者:
コスモ 虹川
対象バージョン:
0.1
プラグイン名:
ブランチ:
クラッシュする:

説明

Ubuntu11.10でconfig.rbのsvn infoからリビジョンを引っ張る機能が上手く動いていないようです。

VERSION = [0,1,0, ((/Last Changed Rev\s*:\s*(\d+)/.match(`sh -c 'LANG=C svn info ../'`)[1] || REVISION).to_i rescue REVISION)]

では上手く引っ張って来れていないようです。
VERSION = [0,1,0, ((/Last Changed Rev\s*:\s*(\d+)/.match(`sh -c 'LC_ALL=C svn info ../'`)[1] || REVISION).to_i rescue REVISION)]

こちらだと引っ張れるようです
操作
リンクをコピー
 #1

toshi_a 初音 さんがほぼ13年前に更新

  • ステータス を 新規 から 実装待ち に変更
  • 対象バージョン を 0.0.4 にセット

言われてみれば。0.0.4から修正ということにします。

操作
リンクをコピー
 #2

toshi_a 初音 さんがほぼ13年前に更新

  • ステータス を 実装待ち から レビュー待ち に変更

修正しました。確認お願いします。

操作
リンクをコピー
 #3

toshi_a 初音 さんがほぼ13年前に更新

  • 担当者 を toshi_a 初音 から コスモ 虹川 に変更
操作
リンクをコピー
 #4

コスモ 虹川 さんがほぼ13年前に更新

  • ステータス を レビュー待ち から 解決 に変更

確認しました。Rev.681にて正常に動いています。

操作
リンクをコピー
 #5

Izumi Tsutsui さんが12年以上前に更新

bugreport.rb にも env LANG=C svn info があるようですがそちらも同様に修正必要?

操作
リンクをコピー
 #6

コスモ 虹川 さんが12年以上前に更新

  • ステータス を 解決 から まだダメ に変更

Tsutsuiさんの言う通り

bugreport.rb にも env LANG=C svn info

これも動いてないですね。

open('|env LANG=C svn info').read.match(/Revision\s*:\s*(\d+)/)[1]

から
open('|env LC_ALL=C svn info').read.match(/Revision\s*:\s*(\d+)/)[1]

としないとロケールが正しくなりませんね

操作
リンクをコピー
 #7

toshi_a 初音 さんが12年以上前に更新

  • ステータス を まだダメ から レビュー待ち に変更
  • 対象バージョン を 0.0.4 から 0.1 に変更

修正しました。
ただし、0.0.4はサポートを終了したので、0.1ブランチにのみ適用しました。確認お願いします。

操作
リンクをコピー
 #8

toshi_a 初音 さんが12年以上前に更新

  • ステータス を レビュー待ち から 終了 に変更
操作
リンクをコピー

他の形式にエクスポート: Atom PDF