Skip to content

Conversation

@cacapon
Copy link
Owner

@cacapon cacapon commented Apr 29, 2025

fixed #89
パーサーで$?が展開するように対応しました。

./test_parser.out "cat \"\$?\" \$?\$SHELL\$SHELL"
        cmd->type=0
        cmd->argc=3
        cmd->argv=[cat,"0",0/bin/zsh/bin/zsh,NULL]

@cacapon cacapon requested a review from dayano74 April 29, 2025 03:59
@cacapon cacapon self-assigned this Apr 29, 2025
@dayano74
Copy link
Collaborator

./test_tokenizer.out で確認できますか?

@cacapon
Copy link
Owner Author

cacapon commented Apr 29, 2025

./test_tokenizer.out で確認できますか?

口頭ではちらっと行ったかもですが、./test_parser.outで確認できます。

@cacapon cacapon changed the title Feature/89 status expand 終了ステータスを展開できるように変更 Apr 29, 2025
@dayano74
Copy link
Collaborator

dayano74 commented May 1, 2025

終了ステータスの確認。$?が0のときは確認できたが、0以外のときの出力を確認できていないため。

@dayano74 dayano74 mentioned this pull request May 1, 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.

$?が0のときは確認できました。それ以外の確認はイシュー立てたのでそっちで解決しましょう。

@dayano74 dayano74 merged commit 9561a83 into main May 1, 2025
1 check passed
@dayano74 dayano74 deleted the feature/89-status-expand branch May 1, 2025 04:12
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.

$?に最後に実行したフォアグラウンドのパイプラインの終了ステータスを格納する必要があります。

3 participants