Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix an issue in the env-var option of CartonHelpers.Process #437

Merged
merged 1 commit into from
May 17, 2024

Conversation

omochi
Copy link
Contributor

@omochi omochi commented May 17, 2024

課題

CartonHelpers.Process の環境変数設定機能が壊れています。
これを介して env コマンドを実行すると、以下のようなおかしな文字列が得られます。

ProcessEnvironmentKey(value: "PATH")=/usr/local/bin:/usr/bin

期待される正しい挙動は

PATH=/usr/local/bin:/usr/bin

です。

内容

このパッチはこのバグを修正します。
また、期待した結果が得られるか確認する自動テストを追加します。

懸念

このソースコードは swift-tools-support-core から派生しているようです。
そのため、このプロジェクト独自の変更はメンテナンス性を低下させる恐れがあります。
すでに TSC自体が deprecated だからどうでもいいですかね。

@kateinoigakukun kateinoigakukun added the bug Something isn't working label May 17, 2024
@kateinoigakukun kateinoigakukun merged commit 848ae5a into swiftwasm:main May 17, 2024
3 of 4 checks passed
@kateinoigakukun kateinoigakukun changed the title CartonHelpers.Processの環境変数設定機能のバグを直す Fix an issue in the env-var option of CartonHelpers.Process May 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants