Skip to content

Windows環境でwhere.exeを使用するよう変更 - #2

Open
bitpaname wants to merge 1 commit into
key-moon:mainfrom
bitpaname:win-tool-exists
Open

Windows環境でwhere.exeを使用するよう変更#2
bitpaname wants to merge 1 commit into
key-moon:mainfrom
bitpaname:win-tool-exists

Conversation

@bitpaname

Copy link
Copy Markdown

tool_existsでツールの導入状況を確認する際、非WSLのWindowsではwhichコマンドが使用できず「エラー: 対応するツールがインストールされていません」となっていたため、sys.platformでWindows環境を検出しwhere.exeによる存在判定を行うよう変更を加えました。ご確認頂ければ幸いです。

@weirdo0z

Copy link
Copy Markdown

横から失礼いたします。
私の方では標準ライブラリであるshutil.which()を使うように変更をしようとしていたのですが、
こちらの方法はいかがですか?
実装は以下のようになるかと思います。

def tool_exists(cls) -> bool:
    """ツールがシステムに存在するかを確認する"""
    try:
        return shutil.which(cls.name) is not None
    except Exception:
        return False

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