Skip to content

Conversation

@cacapon
Copy link
Owner

@cacapon cacapon commented May 8, 2025

fixed #87 , fixed #182 , fixed #190
対応が必要そうな箇所のexitステータスを返す処理を追加しました。

対応した箇所

  • echo: writeに失敗した時に1を返すように変更(確認難しい)
  • export: 第二引数の最初が-だった場合、問答無用で2とエラーメッセージを返すように
  • export: いずれかの引数が無効な文字をだった場合に1を返す

確認方法

  • exportはminishellで確認可能です。
    • $ export --hoge a=bexport 1=2 a=b 2=3 c=dのようなコマンド後にecho $?でご確認ください。
  • echoは書き込み権限がないファイルに対してecho > file.txtで本来確認可能なのですが、別のバグで確認できません。

対応相談

  • env,unsetはメッセージ無しの0を返すのがbash準拠らしいのですが、現在は一部条件でエラーメッセージ・と1を返す作りになっています。
    • このプルリクでは未対応なのですがこちらの対応どうしましょうか?

@cacapon cacapon requested a review from dayano74 May 8, 2025 09:40
@cacapon cacapon self-assigned this May 8, 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の検証も確認します。

@dayano74 dayano74 merged commit 8f0241e into main May 9, 2025
2 checks passed
@dayano74 dayano74 deleted the feature/182-builtin-exit-sts branch May 9, 2025 05:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants