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

[DRAFT][SKETCH] Ideas for stat.h interface #56

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

rauhul
Copy link
Contributor

@rauhul rauhul commented Jul 9, 2021

  • Adds wrapper functions for all posix functions in darwin's stat.h header.
  • Non-posix stat.h functions (<fn>_np) are out of scope of this sketch.
  • fcntrl functions are flags are out of scope of this sketch.

Needs work:

  • Function names
  • API for umask
  • Documentation
  • Tests

@rauhul rauhul requested review from milseman and lorentey July 9, 2021 01:16
@rauhul rauhul added the enhancement New feature or request label Jul 9, 2021
@milseman
Copy link
Contributor

milseman commented Oct 3, 2021

I want to make sure we can make (even if slow) progress on this. @rauhul, have you thought any more about this?

@rauhul
Copy link
Contributor Author

rauhul commented Oct 4, 2021

@milseman I think we need a good story about fcntl flags and if want to move the non-permissions bits of FilePermissions out into a different type. In the sketch as is, these are called FileModeOther as a placeholder. I think two other open areas are TimeSpecification and function naming.

@milseman
Copy link
Contributor

Regarding Timespec, it might be interesting to see how we fit in with the outcome of https://forums.swift.org/t/se-0329-clock-instant-date-and-duration/53309

@rauhul
Copy link
Contributor Author

rauhul commented Mar 2, 2024

@glessard would you be interested in reviving this PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants