-
Notifications
You must be signed in to change notification settings - Fork 92
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
Wrappers for pread & pwrite. #105
Comments
Hello. The coincidence of module names is too bad, though... |
Good to know, thanks for the pointer. |
If you really want to this moved into |
I'm unlikely to open a PR; at a certain point I got frustrated with the existing set of packages and wrote https://hackage.haskell.org/package/unix-simple. Feel free to close. |
Yeah, I think 4 years isn't a reasonable time frame to respond to feature requests. |
To crystal be clear, I don't think I am owed a response at all, much less in a specific time frame. The package I wrote was as much addressing general design issues I saw as it was dealing with particular missing functions; my message yesterday was merely meant to communicate that I'd moved on, though looking back I see that without tone of voice to aid it could be interpreted as somewhat passive-agressive, and if it was taken that way I apologize. |
Well, my point is that we possibly lost a contributor, because we didn't manage to respond in time and provide both an opinion and guidance about a possible implementation. I think that needs to improve for boot libraries. |
@hasufell it's not Viktor's fault that previous maintainers were unresponsive for years. |
I wasn't trying to imply that. Just saying that from a contributors point of view this is depressing and boot libraries should do better. |
I must sheepishly admit to also have been cycle-starved recently, and would ideally have piped up somewhat sooner. Mea culpa. That said, my intent is to make amends shortly. In light of that, let's return to this PR. Is there still a desire to see pwrite and pread implemented? My FreeBSD
So |
I think it would certainly still make sense, at least for pwrite --
though as I noted I have sortof moved on for my own projects.
Thinking about it I'm not sure how pread should actually work, since it
normally takes a series of buffers to read into, so a high level
translation that returned [ByteString] or such wouldn't really make
sense, since there's no clear way to know how to size the buffers. It
only really makes sense if the caller is providing the buffers.
Quoting Viktor Dukhovni (2022-04-15 01:05:30)
… I must sheepishly admit to also have been cycle-starved recently, and
would ideally have piped up somewhat sooner. Mea culpa. That said, my
intent is to make amends shortly. In light of that, let's return to
this PR. Is there still a desire to see pwrite and pread implemented?
My FreeBSD pwrite(2) manpage states:
STANDARDS
The write() system call is expected to conform to IEEE Std 1003.1-1990
(�POSIX.1�). The writev() and pwrite() system calls are expected to
conform to X/Open Portability Guide Issue� 4, Version� 2 (�XPG4.2�).
So pwrite is POSIX for a sufficiently recent version of POSIX. So
perhaps the case for inclusion is reasonable?
--
Reply to this email directly, [1]view it on GitHub, or [2]unsubscribe.
You are receiving this because you authored the thread. Message ID:
***@***.***>
Verweise
1. #105 (comment)
2. https://github.com/notifications/unsubscribe-auth/AAGXYPWPBCMRAZZIU3EVCS3VFD2JVANCNFSM4ELIILVQ
|
Perhaps you're thinking of Some of this was discussed a while back in the context of the |
Quoting Viktor Dukhovni (2022-04-15 15:46:51)
although there are of course also pwritev and preadv
Ah, you're right I was thinking of preadv; sorry, not paying enough
attention apparently. pread should be straightforward.
|
Do we have more insights now? |
Yes, in With |
Like the title says, there don't seem to be any wrappers for pread & pwrite -- these would be useful.
The text was updated successfully, but these errors were encountered: