Skip to content

Conversation

@cacapon
Copy link
Owner

@cacapon cacapon commented May 9, 2025

fixed #231

元々echo $'SHELL'のようなコマンドで無限ループになっていた件ですが、
echo $のような$のみでも同様に無限ループに陥ることが判明したので対応しました。

確認方法

  • minishell起動後echo $などで$だけ表示され、次のプロンプトが正常に入力できることを確認してください。

備考

  • echo $'SHELL'の場合SHELLと出るのがBash準拠なのですが、minishellだと$SHELLとなります。
    • こちらの対応についてはWANTとさせてください。

@cacapon cacapon requested a review from dayano74 May 9, 2025 07:35
@cacapon cacapon self-assigned this May 9, 2025
Copy link
Collaborator

@dayano74 dayano74 left a comment

Choose a reason for hiding this comment

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

確認方法にしたがって確認しました。現在の求めている出力だったので、問題ないと思います。echo $'SHELL'のwant把握しました。

@dayano74 dayano74 merged commit ab8e50b into main May 9, 2025
2 checks passed
@dayano74 dayano74 deleted the feature/231-fix-dollar branch May 9, 2025 07:59
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.

$'SHELL\'でクラッシュ

3 participants