You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I unexported RenamedFrom to renamedFrom for now, and the tests aren't run on platforms where this isn't supported yet (kq, fen). This way new features like this can be developed and merged in to main without holding up everything, and PRs can be a bit smaller in scope (I did some work on kqueue, which required rewriting significant bits of it).
The goal would be to get the rename tests enabled on illumos and have them passing correctly. Additionally it would be nice, if possible, to remove any other illumos specific quirks in the test suite.
The text was updated successfully, but these errors were encountered:
There's actually a few other illumos issues as well:
inotify: implement WithNoFollow() #631: option to disable following links. I tried replacing the Stat() with Lstat(), but that didn't really seem to work. Tests enabled with supportsNofollow()
But really, there's no hurries with any of this, many of these have been feature requests that have been around for years, some over a decade. People have died of old age waiting for this – literally. It can wait a few months more.
I suspect one reason these features have never been done in spite of popular demand is that it required implementing at least 3 (now 4) different systems all at once; it's a lot to keep in your head, and a lot of work to squeeze in.
Just let me know when/if you look at any of this, to prevent double work.
Follows from #628
Assigning to myself for now. If I don't make any progress on this in 2 weeks, we should attach the help wanted label and unassign it from me.
Breadcrumb for how to pick up on this :
The goal would be to get the rename tests enabled on illumos and have them passing correctly. Additionally it would be nice, if possible, to remove any other illumos specific quirks in the test suite.
The text was updated successfully, but these errors were encountered: