-
Notifications
You must be signed in to change notification settings - Fork 73
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
Push for v4.4/v5.0 before Ownership Transfer or Maintenence #40
Comments
AsyncWindowsSerialStream needs some cleanup, most importantly to address this comment Working towards a release of 5.4.0-beta1 to crates.io |
I wonder if there might be any interest from the tokio maintainers themselves (@Darksonn @carllerche ?) to take this project under the umbrella of the https://github.com/tokio-rs org. That would make it independent from the GitHub account of a single person, and probably easier to reassign maintainership later. After all, this is the de-facto official tokio crate for dealing with serial ports. |
You are welcome to add Tokio as an owner to function as a backup. You can do that by using |
This might be the way to go. @Darksonn, just to confirm, you're talking about ownership on crates.io and not github itself correct? I only see ownership transfers on github and for organizations I'd need creation permissions for the tokio-rs org which I obviously don't and shouldn't have. It doesn't quite solve the repo maintenance issue but it would at least keep the crates.io space from fragmenting |
Yes, I was talking about ownership on crates.io. Though, I assume you would also add whoever will take over maintenance duties as an owner. When it comes to location of the repo on github, we prefer to only have repositories in the tokio-rs org that we are committed to maintaining ourselves, and I don't personally have the time to take on tokio-serial. |
Yep, that's perfectly reasonable. I'm more concerned about keeping crates.io fragmentation to a minimum |
@berkowski |
@anton-dutov @ColinFinck @emgre 5.4.0-beta1 has landed on crates.io for testing. Merging #39 and #41 with I'm at the "seems to work for me" stage but I'm not happy about the lack of tests. I'm not sure there's really a way to do anything using a CI service, but at least I could create some manual tests like In the meantime I will start re-evaluating outstanding issues here and in |
@berkowski O windows com0com, but the installation task will be non-trivial |
Looks good! Will do some more testing next week, and I suggest some minor improvements in #42. But other than that, I have no objections regarding a release. |
Good to have something on crates.io again that seems to help people. I've been slowly bashing my way through setting up com0com on appveyor for |
|
@berkowski Both my projects linux/windows works fine after updated to 5.4.0-beta1, no issues to report |
bumped to |
bumped to |
If no one else is willing to step up then I volunteer. I can't promise to commit a huge amount of time, but I do have several crates depending on this one, so there's that .... Someone from the Tokio org would obviously be much better, please consider me a last resort. Also, thanks a lot for stepping up and putting time into this important crate even though you no longer personally care about it. |
I have a few crates depending on tokio-serial as well. As a contributor, I will also do my best to keep it aligned with latest tokio releases, but I can't promise more than that. I guess it's the best to put it under the umbrella of the tokio GitHub and crates.io organizations and give admin access to a few people, e.g. @estokes, me, and possibly others that volunteer. Changes can then be mutually reviewed via PRs without depending on a single person alone. |
Thank-you @estokes, @ColinFinck. Would you be willing to take on It doesn't seem like tokio-rs is interested in taking ownership of the actual repos at this point. They are willing to monitor the packages on crates.io to prevent fracturing the user side of things for cargo though. How does this sound?
This seems like the easiest path forward but let me know if you have reservations about any portion. Hopefully the actual effort in maintaining these crates will be minimal moving forward now that tokio is stable. It's not that I don't care about these crates anymore, more that I just don't use them at all for any projects in over a year. I learned a lot from this experience, both good and bad, and at the very least will try to be more up front with publicly released code that seems to gain some use by others. |
Yes I would also be willing to look after mio-serial. This sounds like it's
worth a try.
…On Wed, Jul 28, 2021 at 7:37 AM Zac Berkowitz ***@***.***> wrote:
Thank-you @estokes <https://github.com/estokes>, @ColinFinck
<https://github.com/ColinFinck>. Would you be willing to take on
mio-serial as well? In my mind they go hand in hand.
It doesn't seem like tokio-rs <https://github.com/tokio-rs/> is
interested in taking ownership of the actual repos at this point. They are
willing to monitor the packages on crates.io to prevent fracturing the
user side of things for cargo though. How does this sound?
- Add estokes and ColinFinck as collaborators to mio-serial, and
tokio-serial. That provides the ability to commit and merge PR's in my
absence
- Add tokio-rs, estokes, and ColinFinck as owners on crates.io. That
provides the ability to publish new versions
- Keep the repos under my ownership. Keeps all existing links to these
repos intact.
This seems like the easiest path forward but let me know if you have
reservations about any portion.
Hopefully the actual effort in maintaining these crates will be minimal
moving forward now that tokio is stable. It's not that I don't care about
these crates anymore, more that I just don't use them at all for any
projects in over a year. I learned a lot from this experience, both good
and bad, and at the very least will try to be more up front with publicly
released code that seems to gain some use by others.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#40 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABWJHLBLIXN7GUFOUUPUODLTZ7TY5ANCNFSM5AKCJJUQ>
.
|
Sounds great! 👍 |
So how are we going to proceed here? |
I figured I would wait on the 5.4.0 release, which I'll plan on doing this weekend assuming there's no issues that crop up this week. As I see it the steps left are:
|
Whatever suits. My crates.io login is linked to github, so it's just estokes.
|
Same here. This is my crates.io page: https://crates.io/users/ColinFinck |
released @ColinFinck, @estokes: Ive added you both as collaborators for the @Darksonn: I'm having trouble adding
Ever see this before? |
hmm, so far I only have a crates.io invitation for mio-serial |
Might have just done it on mio-serial, resent both for tokio-serial |
Thanks a lot for handling this! I accepted the invitations for mio-serial and tokio-serial on crates.io and GitHub. |
Yep, that worked, thanks! |
Folks, if you are the new maintainers, care to update the README with the status quo? |
It's always the little things ... Done, thanks for reporting. |
My motivation to keep tokio-serial up to date with the async ecosystem has long since waned and it shouldn't be the limiting factor for other people. I'd like to get this crate usable with tokio 1.X, then transfer of ownership to someone more active in the space. Or failing such, explicit notice that the package will no longer be actively developed.
github:tokio-rs:core
as owner on crates.io once stable package is releasedThe text was updated successfully, but these errors were encountered: