Skip to content
This repository has been archived by the owner on Jan 13, 2024. It is now read-only.

Add missing stat.isSocket #720

Merged
merged 1 commit into from
Mar 2, 2021
Merged

Add missing stat.isSocket #720

merged 1 commit into from
Mar 2, 2021

Conversation

zvin
Copy link
Contributor

@zvin zvin commented Jun 17, 2019

No description provided.

Copy link

@pdcastro pdcastro left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@sommd
Copy link

sommd commented Jul 5, 2019

Any reason this hasn't been merged yet? This fixes a problem I'm having with tar-fs, which is preventing me from packaging my app.

@pdcastro
Copy link

pdcastro commented Jul 5, 2019

@zvin and I are external contributors without permission to merge this.
@igorklopov might be able to...

@pdcastro
Copy link

pdcastro commented Jul 5, 2019

By the way, I can also confirm that this PR fixes a problem we were having with the balena-cli. We've been successfully applying a patch over pkg 4.4.0 while this isn't merged:
https://github.com/balena-io/balena-cli/blob/v11.4.4/patches/pkg%2B4.4.0.patch

@sommd
Copy link

sommd commented Jul 5, 2019

It might also be worth adding the missing isBlockDevice and isCharacterDevice. I'm guessing they should always return false.

Copy link
Member

@leerob leerob left a comment

Choose a reason for hiding this comment

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

Thank you so much! Sorry for the slow response here.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants