Skip to content

feat: PJ_ROOT_DIR 環境変数の追加 & PJ_NAME 環境変数の削除#620

Merged
laysakura merged 9 commits intomainfrom
feature/PJ_ROOT_DIR-envvar
May 27, 2021
Merged

feat: PJ_ROOT_DIR 環境変数の追加 & PJ_NAME 環境変数の削除#620
laysakura merged 9 commits intomainfrom
feature/PJ_ROOT_DIR-envvar

Conversation

@laysakura
Copy link
Copy Markdown
Contributor

@laysakura laysakura commented May 27, 2021

Issueへのリンク

(なし)

やったこと

CDWが ~/anonify/** にないときに、EnclaveDir::new() で作成されるディレクトリが /.anonify/ になりがち( PJ_NAME="anonify" の場合)。
CWDに依存せずにEnclaveDirのpathを決定するために、PJ_ROOT_DIR環境変数を追加。

それに伴い、このrepoからは PJ_NAME 環境変数を削除(PJ_ROOT_DIR 設定にしか使われていない)。

ただし他のrepoでもPJ_NAMEを使っているので、いまは併存期間とし、PJ_ROOT_DIR の決定アルゴリズムをこうする:

  1. PJ_ROOT_DIR 環境変数があればそれにする
  2. なくて、 PJ_NAME があれば、CWDからディレクトリを上に辿っていき、PJ_NAMEにマッチするか / に至るまで探索し、見つかったディレクトリにする(既存アルゴリズム)
  3. PJ_NAME もなければ、HOMEディレクトリにする

やらないこと

(なし)

動作検証

  • CDW が /home/anonify-dev/.pgx/data-13
  • PJ_ROOT_DIR="/home/anonify-dev/"

の場合に EnclaveDir::new() し、 /home/anonify-dev/.anonify/ ディレクトリができることを確認。

参考

(なし)

@laysakura laysakura force-pushed the feature/PJ_ROOT_DIR-envvar branch from 06272f3 to b8cc450 Compare May 27, 2021 07:19
@laysakura laysakura changed the title feat: PJ_ROOT_DIR 環境変数の追加 feat: PJ_ROOT_DIR 環境変数の追加 & PJ_NAME 環境変数の削除 May 27, 2021
#!/bin/bash

set -e
set -ex
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CIが落ちたときにどの箇所かわかりやすくするためにサクッとつけてしまいました。

@laysakura
Copy link
Copy Markdown
Contributor Author

https://github.com/LayerXcom/anonify-contracts/search?q=PJ_NAME
こっち側で PJ_NAME 使ってるからCI落ちてる模様 😞

@osuketh PJ_NAMEも同時に消しちゃおうと話しましたが、やはり併存期間必要そうです。

  • このrepoではPJ_NAMEを定義・使用する箇所はなくす
  • PJ_ROOT_DIR の決定アルゴリズムをこうする:
    1. PJ_ROOT_DIR 環境変数があればそれにする
    2. なくて、 PJ_NAME があれば、CWDからディレクトリを上に辿っていき、PJ_NAMEにマッチするか / に至るまで探索し、見つかったディレクトリにする(既存アルゴリズム)
    3. PJ_NAME もなければ、HOMEディレクトリにする

としようと思います。どうでしょうか?

@osuketh
Copy link
Copy Markdown
Member

osuketh commented May 27, 2021

あー..あざます!そいつで行きましょう 👍

@laysakura laysakura force-pushed the feature/PJ_ROOT_DIR-envvar branch from ba51d40 to ec07063 Compare May 27, 2021 11:14
@laysakura laysakura requested a review from osuketh May 27, 2021 11:48
@laysakura laysakura marked this pull request as ready for review May 27, 2021 11:48
@laysakura laysakura merged commit 8584a0e into main May 27, 2021
@laysakura laysakura deleted the feature/PJ_ROOT_DIR-envvar branch May 27, 2021 11:54
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