New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support Plamo Linux #273

Merged
merged 1 commit into from Oct 13, 2013

Conversation

Projects
None yet
2 participants
@tenforward
Contributor

tenforward commented Oct 13, 2013

見よう見まねで Plamo Linux サポートを追加してみました.十分に理解してないのでマズい点も多いかもしれません.気になる所は

  • Plamo には service コマンドはありませんのでコマンドとして check_running は実装していません.backend で check_running 失敗したら check_process してるからいいのかなと思ったからですが,いかがでしょうか?
  • テストは debian ベースに check_running 関係の部分を削除しました.他に修正必要な部分あるのかも?
  • Plamo はデーモン起動の決まった一つの方法がないので check_enabled の実装も微妙ですが,最近は rc.* スクリプトに色々追加して起動させるよりは init.d にスクリプトを置く事が多いのでとりあえずそこをチェックしています.他と違って runlevel 毎に細かく起動するデーモンを起動する仕組みもないので,期待される動きにならない部分もあるかもしれません.
  • Debian/Ubuntu のように Slackware 系でまとめることも考えましたが,ちょっと簡単には無理そうですので Plamo のみで実装しました.
@mizzy

This comment has been minimized.

Show comment
Hide comment
@mizzy

mizzy Oct 13, 2013

Owner

ありがとうございます!問題なさそうですので、マージします。

Owner

mizzy commented Oct 13, 2013

ありがとうございます!問題なさそうですので、マージします。

mizzy added a commit that referenced this pull request Oct 13, 2013

@mizzy mizzy merged commit 456c8b9 into mizzy:master Oct 13, 2013

1 check passed

default The Travis CI build passed
Details

@tenforward tenforward deleted the tenforward:support_plamo branch Oct 13, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment