Project

General

Profile

Actions

バグ #423

closed

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

Added by コスモ 虹川 over 9 years ago. Updated over 9 years ago.

Status:
終了
Priority:
通常
Target version:
プラグイン名:
ブランチ:
クラッシュする:

Description

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)]

こちらだと引っ張れるようです
Actions #1

Updated by toshi_a 初音 over 9 years ago

  • Status changed from 新規 to 実装待ち
  • Target version set to 0.0.4

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

Actions #2

Updated by toshi_a 初音 over 9 years ago

  • Status changed from 実装待ち to レビュー待ち

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

Actions #3

Updated by toshi_a 初音 over 9 years ago

  • Assignee changed from toshi_a 初音 to コスモ 虹川
Actions #4

Updated by コスモ 虹川 over 9 years ago

  • Status changed from レビュー待ち to 解決

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

Actions #5

Updated by Izumi Tsutsui over 9 years ago

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

Actions #6

Updated by コスモ 虹川 over 9 years ago

  • Status changed from 解決 to まだダメ

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]

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

Actions #7

Updated by toshi_a 初音 over 9 years ago

  • Status changed from まだダメ to レビュー待ち
  • Target version changed from 0.0.4 to 0.1

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

Actions #8

Updated by toshi_a 初音 over 9 years ago

  • Status changed from レビュー待ち to 終了
Actions

Also available in: Atom PDF