プロジェクト

全般

プロフィール

Bug(バグ) #2397

プロジェクトルートに同梱されているsymfonyファイルを誤って更新している

Yuya Watanabe12年以上前に追加. 8年以上前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2011-09-08
期日:
進捗率:

100%

3.6 で発生するか:
Unknown (未調査)
3.8 で発生するか:
Unknown (未調査)

説明

内容

#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ファイルを適切なものに置き換える必要がある.

確認方法

  1. 空のプロジェクト内でsymfony-1.4.3のプロジェクトを作成タスクを行う
  2. 生成されたプロジェクトルートのsymfonyファイルを確認
    1. #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 の問題も解決できると思われる.


関連するチケット

関連している OpenPNE 3 - Bug(バグ) #1334: postfixでsymfonyコマンドの実行が失敗する New(新規) 2010-07-13
関連している OpenPNE 3 - Enhancement(機能追加・改善) #673: update symfony to 1.4.3 (同梱するsymfonyのバージョンを1.4.3に上げる) Fixed(完了) 2010-02-17
関連している OpenPNE 3 - Backport(バックポート) #1466: update symfony to 1.4.3 (同梱のsymfonyのバージョンを1.4.3へ上げる) Fixed(完了) 2010-07-02
関連している OpenPNE 3 - Enhancement(機能追加・改善) #2316: update symfony to 1.4.13 (同梱のsymfonyのバージョンを1.4.13へ上げる) Fixed(完了) 2011-07-29 2011-08-05
関連している OpenPNE 3 - Backport(バックポート) #2402: プロジェクトルートに同梱されているsymfonyファイルを誤って更新している Fixed(完了) 2011-09-13

関係しているリビジョン

リビジョン 0734d849 (差分)
Yuya Watanabe12年以上前に追加

(fixes #2397) replace symfony file

履歴

#1 Yuya Watanabe12年以上前に更新

  • 説明 を更新 (diff)

#2 Yuya Watanabe12年以上前に更新

  • 説明 を更新 (diff)

#3 Yuya Watanabe12年以上前に更新

  • 説明 を更新 (diff)

#4 Yuya Watanabe12年以上前に更新

確認方法

http://redmine.openpne.jp/issues/1334#note-5 より抜粋および追加.

  1. symfonyのプロジェクト生成タスクを実行
  2. OpenPNE3のソースコードを取得
  3. 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 Yuya Watanabe12年以上前に更新

  • ステータスNew(新規) から Accepted(着手) に変更

#6 Yuya Watanabe12年以上前に更新

OpenPNE3.7.0-devに同梱されているsymfony-1.4.6で生成されたプロジェクトと比較をし,OpenPNE側で手を加えられていない部分としてsymfonyファイル以外に差異が存在してないことを確認しました.symfonyファイルのみを差し替えます.

#7 wa ta12年以上前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

更新履歴 0734d849b39112b4c325df965885776de5c74fbc で適用されました。

#8 Yuya Watanabe12年以上前に更新

  • 説明 を更新 (diff)

#9 Kousuke Ebihara12年以上前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

#10 Kousuke Ebihara12年以上前に更新

  • 3.6 で発生するかYes にセット
  • 360対象RC1 にセット

#11 Shouta Kashiwagi約12年前に更新

  • ステータスPending Testing(テスト待ち) から Fixed(完了) に変更
  • 進捗率70 から 100 に変更
  • 3.6 で発生するかYes から Unknown (未調査) に変更
  • 3.4 で発生するかUnknown (未調査) にセット

テストOKです.

#12 kaoru n8年以上前に更新

  • 3.8 で発生するかUnknown (未調査) にセット

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