ytnobody / Testament

This repository is old. Please see https://github.com/testament-testing-env/Testament
Other
7 stars 2 forks source link

未対応アーキテクチャのエラーハンドリング #42

Closed moznion closed 11 years ago

moznion commented 11 years ago

ディストリが配布していないアーキテクチャを指定された場合の、エラー (例外) ハンドリングを実装する。

あるいは、アーキテクチャのテーブルを作成する。

e.g.

x86_64 -> amd64

みたいな。

ytnobody commented 11 years ago

これ、現状だとDLできなくて死ぬだけなんだけど、とりあえずそこのエラーメッセージに

Maybe, remote server is down or, specified os and/or version is illegal.

って付け加えておけばオッケーです?

moznion commented 11 years ago

あれ、今って明示的にdie (croak?) してたんでしたっけ。 それだったらそれで良い感じがしますね

ytnobody commented 11 years ago

https://github.com/ytnobody/Testament/blob/master/lib/Testament/URLFetcher.pm#L19

このへんですね。

moznion commented 11 years ago

あ、本当だ とりあえずエラーメッセージはそれで良いと思います。 あとはアーキテクチャの対応テーブルですかねー

ytnobody commented 11 years ago

アーキテクチャの対応テーブル、Testament::Setup::*.pmのmirrors関数あたりで頑張るしかなさそう

moznion commented 11 years ago

そうですね。下手に万能にしようとすると詰む感じがありますね

ytnobody commented 11 years ago

個々のSetupでやればオッケーなので、このイシューとじます