8000 Ubuntu 14.04対応 by tkmtmkt · Pull Request #157 · alminium/alminium · GitHub
[go: up one dir, main page]

Skip to content

Ubuntu 14.04対応 - #157

Merged
mikoto20000 merged 7 commits into
alminium:devfrom
tkmtmkt:patch-8
Oct 8, 2014
Merged

mikoto20000 merged 7 commits into
alminium:devfrom
tkmtmkt:patch-8

Conversation

@tkmtmkt
@tkmtmkt tkmtmkt commented Sep 3, 2014
Copy link
Copy Markdown

【内容】

  • jenkins設定ファイルを整理
  • apache設定ファイルを2.4対応にした。
  • ssl、jenkinsの判定を設定ファイルのコメント有無で切り替えるようにした。
  • apache設定のセットアップ処理をUbuntu 14.04対応にした。

【動作確認】

  • インストール実行して error, fatal が出ないこと
  • 主要な画面を表示して log/production.log にエラーが出ないこと(既存のエラーを除く)

【確認環境】

  • CentOS 6.5 x86_64
  • Ubuntu 12.04.5 LTS x86_64
  • Ubuntu 14.04 LTS x86_64

【補足】

  • Ubuntu 12.04では/etc/apache2/conf.d に置いたファイルを読み込んでいたが、Ubuntu 14.04では読み込まなくなり、conf-available/conf-enabledの組み合わせに変わった。(/etc/apache2/apache.conf)
  • 12.04で/etc/apache2/conf.dにシンボリックリンクを置いていたWEBアプリケーションパッケージは、14.04では/etc/apache2/conf-availableにシンボリックリンクを置くように変わっている。

* Ubuntu 14.04では/etc/apache2/conf.dの内容が読み込まれない。
* passenger.confファイルの出力先を変更したため/etc/opt/alminiumを
  先に作成しておく必要がある。
* 以下の3つのディレクトリの作成処理を近くに集めた。
  * ALM_ETC_DIR=/etc/opt/alminium
  * ALM_VAR_DIR=/var/opt/alminium
  * INSTALL_DIR=/opt/alminium
@tkmtmkt
tkmtmkt commented Sep 4, 2014
Copy link
Copy Markdown
Author

ファイル配置の変更に合わせてserverspecも修正してみましたが、if文追加しただけの残念なコードになったのでコミットしませんでした。もう少し調べてからにします。

@mikoto20000
8000 Copy link
Copy Markdown
Member

了解しました。お待ちしています。

* 「install⇒プラグインファイルコピー⇒install」と処理していたので、
  「プラグインファイルコピー⇒install」のようにinstallを一回だけ
  実施するように変更した。
* `$RAKE`を定義して使用しているが、`bundle exec`で呼び出す箇所で
  使用するのは間違い。たぶん過去の遺物と思われるので修正した。

【補足】

redmineプラグインをインストールするのに、ファイルコピーしてから
`bundle install`実行しているが、別のやり方があるのではないか?
* apache2.2/apache2.4で処理を分ける必要なかった。
* ファイル配置変更に合わせてserverspecを修正した。
@tkmtmkt
tkmtmkt commented Sep 15, 2014
Copy link
Copy Markdown
Author

redmineインストール処理の修正はこのPRと別にすべきでしたが、気付いたのがpushした後だったのでこのままで。
serverspecはOS種別毎の処理を別ファイルにして、条件分岐して呼び出す形にすることを考えていましたが、所定のディレクトリにテストコードを置いたら呼び出されるので、余計な判定処理は行わずにOS固有のテストコードの場合にチェック処理を入れるだけにしました。

mikoto20000 added a commit that referenced this pull request Oct 8, 2014
@mikoto20000
mikoto20000 merged commit 0e76958 into alminium:dev Oct 8, 2014
@mikoto20000
Copy link
Copy Markdown
Member

遅くなりました、マージしました。すみません、、、、

@tkmtmkt
tkmtmkt deleted the patch-8 branch March 15, 2015 03:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

0