Add support for reading env from named pipes

[Bortnyak]

Type of Change

• Bug fix
• New feature
• Breaking change
• Documentation update

Motivation and Context

This is a fix for issue#956

FileManager.default.contents(atPath:) returns nil for named pipes (FIFOs)
and process substitutions like /dev/fd/XX because:

1. It expects regular files with a known size
2. Named pipes are stream-based and block until data arrives

Solution

Use FileHandle(forReadingFrom:) instead, which:

• Properly handles blocking I/O
• Works with named pipes, process substitutions, and regular files (mentioned in the doc)

References

• https://stackoverflow.com/questions/60747062/ios-named-pipes-works-in-objc-does-not-work-in-swift-same-code

Testing

• Tested locally
• Added/updated tests
• Added/updated docs

Result

-> % ./bin/container run -it --rm --env-file <(printf "VAR1=value1\nVAR2=value2\n") ubuntu:latest
Warning! Running debug build. Performance may be degraded.
root@725da974-da43-47cd-b86e-745265317ed9:/# echo $VAR2
value2

[Bortnyak]

@dcantah Thank you for the review. I've added the requested changes. Please take a look.

[dcantah]

This LGTM, but I'd like an integration test for this (e.g. try and spawn a container doing what's in the issue and verifying it works). Can be added to TestCLIRunCommand.swift

[dcantah]

You'll need to make fmt and re-push. Also it's complaining that the commits aren't signed (ssh, or gpg will work)

[Bortnyak]

You'll need to make fmt and re-push. Also it's complaining that the commits aren't signed (ssh, or gpg will work)

Done

[dcantah]

@Bortnyak Still says some of these don't have signatures. Can you try squashing your commits down to one, signing that, and then pushing?

[dcantah]

@Bortnyak I think the rebase was screwed up a bit, there's 32 files changed. Unrelated, but it also seems whatever email you're using for the commits is not assigned to your github account.

[Bortnyak]

@dcantah I'm sorry, I messed things up while switching between laptops.
I've opened a new PR - #974, let's close this one.

[dcantah]

Fixed via #974

[Bortnyak]

Type of Change

• Bug fix
• New feature
• Breaking change
• Documentation update

Motivation and Context

This is a fix for issue#956

FileManager.default.contents(atPath:) returns nil for named pipes (FIFOs)
and process substitutions like /dev/fd/XX because:

1. It expects regular files with a known size
2. Named pipes are stream-based and block until data arrives

Solution

Use FileHandle(forReadingFrom:) instead, which:

• Properly handles blocking I/O
• Works with named pipes, process substitutions, and regular files (mentioned in the doc)

References

• https://stackoverflow.com/questions/60747062/ios-named-pipes-works-in-objc-does-not-work-in-swift-same-code

Testing

• Tested locally
• Added/updated tests
• Added/updated docs

Result

-> % ./bin/container run -it --rm --env-file <(printf "VAR1=value1\nVAR2=value2\n") ubuntu:latest
Warning! Running debug build. Performance may be degraded.
root@725da974-da43-47cd-b86e-745265317ed9:/# echo $VAR2
value2

[Bortnyak]

@dcantah Thank you for the review. I've added the requested changes. Please take a look.

[dcantah]

This LGTM, but I'd like an integration test for this (e.g. try and spawn a container doing what's in the issue and verifying it works). Can be added to TestCLIRunCommand.swift

[dcantah]

You'll need to make fmt and re-push. Also it's complaining that the commits aren't signed (ssh, or gpg will work)

[Bortnyak]

You'll need to make fmt and re-push. Also it's complaining that the commits aren't signed (ssh, or gpg will work)

Done

[dcantah]

@Bortnyak Still says some of these don't have signatures. Can you try squashing your commits down to one, signing that, and then pushing?

[dcantah]

@Bortnyak I think the rebase was screwed up a bit, there's 32 files changed. Unrelated, but it also seems whatever email you're using for the commits is not assigned to your github account.

[Bortnyak]

@dcantah I'm sorry, I messed things up while switching between laptops.
I've opened a new PR - #974, let's close this one.

[dcantah]

Fixed via #974

[Bortnyak]

Type of Change

• Bug fix
• New feature
• Breaking change
• Documentation update

Motivation and Context

This is a fix for issue#956

FileManager.default.contents(atPath:) returns nil for named pipes (FIFOs)
and process substitutions like /dev/fd/XX because:

1. It expects regular files with a known size
2. Named pipes are stream-based and block until data arrives

Solution

Use FileHandle(forReadingFrom:) instead, which:

• Properly handles blocking I/O
• Works with named pipes, process substitutions, and regular files (mentioned in the doc)

References

• https://stackoverflow.com/questions/60747062/ios-named-pipes-works-in-objc-does-not-work-in-swift-same-code

Testing

• Tested locally
• Added/updated tests
• Added/updated docs

Result

-> % ./bin/container run -it --rm --env-file <(printf "VAR1=value1\nVAR2=value2\n") ubuntu:latest
Warning! Running debug build. Performance may be degraded.
root@725da974-da43-47cd-b86e-745265317ed9:/# echo $VAR2
value2

[Bortnyak]

@dcantah Thank you for the review. I've added the requested changes. Please take a look.

[dcantah]

This LGTM, but I'd like an integration test for this (e.g. try and spawn a container doing what's in the issue and verifying it works). Can be added to TestCLIRunCommand.swift

[dcantah]

You'll need to make fmt and re-push. Also it's complaining that the commits aren't signed (ssh, or gpg will work)

[Bortnyak]

You'll need to make fmt and re-push. Also it's complaining that the commits aren't signed (ssh, or gpg will work)

Done

[dcantah]

@Bortnyak Still says some of these don't have signatures. Can you try squashing your commits down to one, signing that, and then pushing?

[dcantah]

@Bortnyak I think the rebase was screwed up a bit, there's 32 files changed. Unrelated, but it also seems whatever email you're using for the commits is not assigned to your github account.

[Bortnyak]

@dcantah I'm sorry, I messed things up while switching between laptops.
I've opened a new PR - #974, let's close this one.

[dcantah]

Fixed via #974

[Bortnyak]

Type of Change

• Bug fix
• New feature
• Breaking change
• Documentation update

Motivation and Context

This is a fix for issue#956

FileManager.default.contents(atPath:) returns nil for named pipes (FIFOs)
and process substitutions like /dev/fd/XX because:

1. It expects regular files with a known size
2. Named pipes are stream-based and block until data arrives

Solution

Use FileHandle(forReadingFrom:) instead, which:

• Properly handles blocking I/O
• Works with named pipes, process substitutions, and regular files (mentioned in the doc)

References

• https://stackoverflow.com/questions/60747062/ios-named-pipes-works-in-objc-does-not-work-in-swift-same-code

Testing

• Tested locally
• Added/updated tests
• Added/updated docs

Result

-> % ./bin/container run -it --rm --env-file <(printf "VAR1=value1\nVAR2=value2\n") ubuntu:latest
Warning! Running debug build. Performance may be degraded.
root@725da974-da43-47cd-b86e-745265317ed9:/# echo $VAR2
value2

[Bortnyak]

@dcantah Thank you for the review. I've added the requested changes. Please take a look.

[dcantah]

This LGTM, but I'd like an integration test for this (e.g. try and spawn a container doing what's in the issue and verifying it works). Can be added to TestCLIRunCommand.swift

[dcantah]

You'll need to make fmt and re-push. Also it's complaining that the commits aren't signed (ssh, or gpg will work)

[Bortnyak]

You'll need to make fmt and re-push. Also it's complaining that the commits aren't signed (ssh, or gpg will work)

Done

[dcantah]

@Bortnyak Still says some of these don't have signatures. Can you try squashing your commits down to one, signing that, and then pushing?

[dcantah]

@Bortnyak I think the rebase was screwed up a bit, there's 32 files changed. Unrelated, but it also seems whatever email you're using for the commits is not assigned to your github account.

[Bortnyak]

@dcantah I'm sorry, I messed things up while switching between laptops.
I've opened a new PR - #974, let's close this one.

[dcantah]

Fixed via #974

[Bortnyak]

Type of Change

• Bug fix
• New feature
• Breaking change
• Documentation update

Motivation and Context

This is a fix for issue#956

FileManager.default.contents(atPath:) returns nil for named pipes (FIFOs)
and process substitutions like /dev/fd/XX because:

1. It expects regular files with a known size
2. Named pipes are stream-based and block until data arrives

Solution

Use FileHandle(forReadingFrom:) instead, which:

• Properly handles blocking I/O
• Works with named pipes, process substitutions, and regular files (mentioned in the doc)

References

• https://stackoverflow.com/questions/60747062/ios-named-pipes-works-in-objc-does-not-work-in-swift-same-code

Testing

• Tested locally
• Added/updated tests
• Added/updated docs

Result

-> % ./bin/container run -it --rm --env-file <(printf "VAR1=value1\nVAR2=value2\n") ubuntu:latest
Warning! Running debug build. Performance may be degraded.
root@725da974-da43-47cd-b86e-745265317ed9:/# echo $VAR2
value2

[Bortnyak]

@dcantah Thank you for the review. I've added the requested changes. Please take a look.

[dcantah]

This LGTM, but I'd like an integration test for this (e.g. try and spawn a container doing what's in the issue and verifying it works). Can be added to TestCLIRunCommand.swift

[dcantah]

You'll need to make fmt and re-push. Also it's complaining that the commits aren't signed (ssh, or gpg will work)

[Bortnyak]

You'll need to make fmt and re-push. Also it's complaining that the commits aren't signed (ssh, or gpg will work)

Done

[dcantah]

@Bortnyak Still says some of these don't have signatures. Can you try squashing your commits down to one, signing that, and then pushing?

[dcantah]

@Bortnyak I think the rebase was screwed up a bit, there's 32 files changed. Unrelated, but it also seems whatever email you're using for the commits is not assigned to your github account.

[Bortnyak]

@dcantah I'm sorry, I messed things up while switching between laptops.
I've opened a new PR - #974, let's close this one.

[dcantah]

Fixed via #974

[Bortnyak]

Type of Change

• Bug fix
• New feature
• Breaking change
• Documentation update

Motivation and Context

This is a fix for issue#956

FileManager.default.contents(atPath:) returns nil for named pipes (FIFOs)
and process substitutions like /dev/fd/XX because:

1. It expects regular files with a known size
2. Named pipes are stream-based and block until data arrives

Solution

Use FileHandle(forReadingFrom:) instead, which:

• Properly handles blocking I/O
• Works with named pipes, process substitutions, and regular files (mentioned in the doc)

References

• https://stackoverflow.com/questions/60747062/ios-named-pipes-works-in-objc-does-not-work-in-swift-same-code

Testing

• Tested locally
• Added/updated tests
• Added/updated docs

Result

-> % ./bin/container run -it --rm --env-file <(printf "VAR1=value1\nVAR2=value2\n") ubuntu:latest
Warning! Running debug build. Performance may be degraded.
root@725da974-da43-47cd-b86e-745265317ed9:/# echo $VAR2
value2

[Bortnyak]

@dcantah Thank you for the review. I've added the requested changes. Please take a look.

[dcantah]

This LGTM, but I'd like an integration test for this (e.g. try and spawn a container doing what's in the issue and verifying it works). Can be added to TestCLIRunCommand.swift

[dcantah]

You'll need to make fmt and re-push. Also it's complaining that the commits aren't signed (ssh, or gpg will work)

[Bortnyak]

You'll need to make fmt and re-push. Also it's complaining that the commits aren't signed (ssh, or gpg will work)

Done

[dcantah]

@Bortnyak Still says some of these don't have signatures. Can you try squashing your commits down to one, signing that, and then pushing?

[dcantah]

@Bortnyak I think the rebase was screwed up a bit, there's 32 files changed. Unrelated, but it also seems whatever email you're using for the commits is not assigned to your github account.

[Bortnyak]

@dcantah I'm sorry, I messed things up while switching between laptops.
I've opened a new PR - #974, let's close this one.

[dcantah]

Fixed via #974

[Bortnyak]

Type of Change

• Bug fix
• New feature
• Breaking change
• Documentation update

Motivation and Context

This is a fix for issue#956

FileManager.default.contents(atPath:) returns nil for named pipes (FIFOs)
and process substitutions like /dev/fd/XX because:

1. It expects regular files with a known size
2. Named pipes are stream-based and block until data arrives

Solution

Use FileHandle(forReadingFrom:) instead, which:

• Properly handles blocking I/O
• Works with named pipes, process substitutions, and regular files (mentioned in the doc)

References

• https://stackoverflow.com/questions/60747062/ios-named-pipes-works-in-objc-does-not-work-in-swift-same-code

Testing

• Tested locally
• Added/updated tests
• Added/updated docs

Result

-> % ./bin/container run -it --rm --env-file <(printf "VAR1=value1\nVAR2=value2\n") ubuntu:latest
Warning! Running debug build. Performance may be degraded.
root@725da974-da43-47cd-b86e-745265317ed9:/# echo $VAR2
value2

[Bortnyak]

@dcantah Thank you for the review. I've added the requested changes. Please take a look.

[dcantah]

This LGTM, but I'd like an integration test for this (e.g. try and spawn a container doing what's in the issue and verifying it works). Can be added to TestCLIRunCommand.swift

[dcantah]

You'll need to make fmt and re-push. Also it's complaining that the commits aren't signed (ssh, or gpg will work)

[Bortnyak]

You'll need to make fmt and re-push. Also it's complaining that the commits aren't signed (ssh, or gpg will work)

Done

[dcantah]

@Bortnyak Still says some of these don't have signatures. Can you try squashing your commits down to one, signing that, and then pushing?

[dcantah]

@Bortnyak I think the rebase was screwed up a bit, there's 32 files changed. Unrelated, but it also seems whatever email you're using for the commits is not assigned to your github account.

[Bortnyak]

@dcantah I'm sorry, I messed things up while switching between laptops.
I've opened a new PR - #974, let's close this one.

[dcantah]

Fixed via #974

[Bortnyak]

Type of Change

• Bug fix
• New feature
• Breaking change
• Documentation update

Motivation and Context

This is a fix for issue#956

FileManager.default.contents(atPath:) returns nil for named pipes (FIFOs)
and process substitutions like /dev/fd/XX because:

1. It expects regular files with a known size
2. Named pipes are stream-based and block until data arrives

Solution

Use FileHandle(forReadingFrom:) instead, which:

• Properly handles blocking I/O
• Works with named pipes, process substitutions, and regular files (mentioned in the doc)

References

• https://stackoverflow.com/questions/60747062/ios-named-pipes-works-in-objc-does-not-work-in-swift-same-code

Testing

• Tested locally
• Added/updated tests
• Added/updated docs

Result

-> % ./bin/container run -it --rm --env-file <(printf "VAR1=value1\nVAR2=value2\n") ubuntu:latest
Warning! Running debug build. Performance may be degraded.
root@725da974-da43-47cd-b86e-745265317ed9:/# echo $VAR2
value2

[Bortnyak]

@dcantah Thank you for the review. I've added the requested changes. Please take a look.

[dcantah]

This LGTM, but I'd like an integration test for this (e.g. try and spawn a container doing what's in the issue and verifying it works). Can be added to TestCLIRunCommand.swift

[dcantah]

You'll need to make fmt and re-push. Also it's complaining that the commits aren't signed (ssh, or gpg will work)

[Bortnyak]

You'll need to make fmt and re-push. Also it's complaining that the commits aren't signed (ssh, or gpg will work)

Done

[dcantah]

@Bortnyak Still says some of these don't have signatures. Can you try squashing your commits down to one, signing that, and then pushing?

[dcantah]

@Bortnyak I think the rebase was screwed up a bit, there's 32 files changed. Unrelated, but it also seems whatever email you're using for the commits is not assigned to your github account.

[Bortnyak]

@dcantah I'm sorry, I messed things up while switching between laptops.
I've opened a new PR - #974, let's close this one.

[dcantah]

Fixed via #974