Bug(バグ) #2397
プロジェクトルートに同梱されているsymfonyファイルを誤って更新している
100%
Description
内容¶
#673 「update symfony to 1.4.3 (同梱するsymfonyのバージョンを1.4.3に上げる)」によりsymfonyのバージョンアップが行われたが,この変更によってプロジェクトルートに存在するsymfonyファイルを更新されている.
https://github.com/openpne/OpenPNE3/commit/02b9af44d8ef1679d4d618b4d7e3d9637649faab#diff-107
しかし, http://redmine.openpne.jp/issues/1334#note-5 の調査結果により本来更新する必要がないはずが更新が行われていると考えられる.
このsymfonyファイルを適切なものに置き換える必要がある.
確認方法¶
- 空のプロジェクト内でsymfony-1.4.3のプロジェクトを作成タスクを行う
- 生成されたプロジェクトルートのsymfonyファイルを確認
- #673 における変更されたプロジェクトルートのsymfonyファイルに差異が存在していることを確認
原因¶
#623 によって誤った変更が行われている.具体的には,lib/vendor/symfony/data/bin/symfonyとプロジェクトルートに生成されるsymfonyは別であるが,lib/vendor/symfony/data/bin/symfonyのものがOpenPNEのプロジェクトルートに設置されている.
修正方針¶
対象ファイルを以下の内容のものに置き換える.
#!/usr/bin/env php <?php /* * This file is part of the symfony package. * (c) Fabien Potencier <fabien.potencier@symfony-project.com> * * For the full copyright and license information, please view the LICENSE * file that was distributed with this source code. */ chdir(dirname(__FILE__)); require_once(dirname(__FILE__).'/config/ProjectConfiguration.class.php'); include(sfCoreAutoload::getInstance()->getBaseDir().'/command/cli.php');
また,その他ファイルについても誤った更新が行われているかどうかを確認する.
確認バージョン¶
OpenPNE master, 3.6beta13, 3.4.12.1
ただし3.4.12.1は #1466 の修正により変更が加えられている.
補足¶
本チケットの修正により #1334 の問題も解決できると思われる.
Related issues
Associated revisions
(fixes #2397) replace symfony file
History
#1
Updated by Yuya Watanabe over 11 years ago
- Description updated (diff)
#2
Updated by Yuya Watanabe over 11 years ago
- Description updated (diff)
#3
Updated by Yuya Watanabe over 11 years ago
- Description updated (diff)
#4
Updated by Yuya Watanabe over 11 years ago
確認方法¶
http://redmine.openpne.jp/issues/1334#note-5 より抜粋および追加.
- symfonyのプロジェクト生成タスクを実行
- OpenPNE3のソースコードを取得
- 1および2において得られたsymfonyファイルの差異を確認
mkdir project cd project mkdir lib mkdir lib/vendor cd lib/vendor wget http://www.symfony-project.org/get/symfony-1.4.3.zip unzip symfony-1.4.3.zip ln -s symfony-1.4.3 symfony cd ../.. ./lib/vendor/symfony/data/bin/symfony generate:project project cd .. git clone https://github.com/openpne/OpenPNE3.git diff project/symfony OpenPNE3/symfony
出力結果
12,14c12,37 < chdir(dirname(__FILE__)); < require_once(dirname(__FILE__).'/config/ProjectConfiguration.class.php'); < include(sfCoreAutoload::getInstance()->getBaseDir().'/command/cli.php'); --- > // project exists? > if (file_exists('config/ProjectConfiguration.class.php')) > { > require_once('config/ProjectConfiguration.class.php'); > $dir = sfCoreAutoload::getInstance()->getBaseDir(); > } > else > { > if (is_readable(dirname(__FILE__).'/../../lib/autoload/sfCoreAutoload.class.php')) > { > // SVN > $dir = realpath(dirname(__FILE__).'/../../lib'); > } > else > { > // PEAR > $dir = '@PEAR-DIR@/symfony'; > > if (!is_dir($dir)) > { > throw new Exception('Unable to find symfony libraries'); > } > } > } > > include($dir.'/command/cli.php');
#5
Updated by Yuya Watanabe over 11 years ago
- Status changed from New(新規) to Accepted(着手)
#6
Updated by Yuya Watanabe over 11 years ago
OpenPNE3.7.0-devに同梱されているsymfony-1.4.6で生成されたプロジェクトと比較をし,OpenPNE側で手を加えられていない部分としてsymfonyファイル以外に差異が存在してないことを確認しました.symfonyファイルのみを差し替えます.
#7
Updated by wa ta over 11 years ago
- Status changed from Accepted(着手) to Pending Review(レビュー待ち)
- % Done changed from 0 to 50
更新履歴 0734d849b39112b4c325df965885776de5c74fbc で適用されました。
#8
Updated by Yuya Watanabe over 11 years ago
- Description updated (diff)
#9
Updated by Kousuke Ebihara over 11 years ago
- Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
- % Done changed from 50 to 70
#10
Updated by Kousuke Ebihara over 11 years ago
- 3.6 で発生するか set to Yes
- 360対象 set to RC1
#11
Updated by Shouta Kashiwagi almost 11 years ago
- Status changed from Pending Testing(テスト待ち) to Fixed(完了)
- % Done changed from 70 to 100
- 3.6 で発生するか changed from Yes to Unknown (未調査)
- 3.4 で発生するか set to Unknown (未調査)
テストOKです.
#12
Updated by kaoru n over 7 years ago
- 3.8 で発生するか set to Unknown (未調査)