-
Notifications
You must be signed in to change notification settings - Fork 19
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
A user usually don't know where is TREMA_HOME when trema gem is used #19
Comments
trema run を修正したので、いまは TREMA_HOME の指定なしでいきなり show_topology が実行できるはずです。本のほうにはそれで書いてあるので、README.md の修正だけで OK っぽいですね。 @kazuyas 隊長、修正をおねがいしていいですか? 🍻 (ビール 2 杯) |
こんな感じで Trema を起動しているのですが、 sudo LOGGING_LEVEL=$TREMA_LOG_LEVEL TREMA_TMP=$TREMA_TMP_DIR そうすると、 ubuntu@ostack01:/opt/stack/trema/apps/topology (current)$ sudo ./show_topology -g と怒られました。 環境は以下です。 $ gem list | grep trema |
ありがとうございます。これは恐らく別件のバグですので、すみませんが別 issue を切ってもらえますか? |
#24 を作っておきました。 |
これ trema 本体の #241 とも関係しています。 いまだと、以下のように "" で囲まないとだめだった思います。 $ trema run "./show_topology -G" | graph-easy #241 はすぐ直りますか?直らないのであれば、"" 付きで修正しておきます。 |
gli を導入してオプションの位置付けを整理したので、たぶんできると思います。 |
trema/trema#241 に詳しく書きましたが、よくよく考えたら "" 無しだといろいろ問題あるので、コントローラにオプションを付けたい場合には必ず「"" あり」でおねがいします。というわけで README.md のほうよろしくお願いしまする。 |
https://github.com/trema/apps/blob/master/topology/README.md で、
トポロジー表示の例があるが、
$ TREMA_HOME=../../trema ./show_topology -G | graph-easy
gem で Trema をインストールした場合、ユーザは TREMA_HOME を意識しないはずであり、TREMA_HOME がどこかよく分からない。
The text was updated successfully, but these errors were encountered: