iTrooz / efivar-rs

EFI boot manager tool written in Rust
MIT License
22 stars 7 forks source link

Looking for contributors ? #41

Closed iTrooz closed 1 year ago

iTrooz commented 1 year ago

Hey, if I ever get time to do this, would you accept contributions to the efiboot tool ? In particular, I'd like to add a way to delete and add new UEFI boot entries. Maybe even get on feature parity with efiboomgr ?

alixinne commented 1 year ago

Hello, thanks for your interest in the project! The initial goal was indeed to get it to feature parity with efibootmgr, but I don't have enough spare time to work on this anymore. I can't really add new features but PRs are welcome and I'll make time to review them.

iTrooz commented 1 year ago

@vtavernier Hi, I see you are reviewing my PRs right now, so I wanted to take this opportunity to ask you something.

I'd like to fork this repository to continue working on the project from there

My reasons are:

Forking the project itself is not a big problem, but I'd like to have access to the https://crates.io/efivar crate to be able to update it myself (not really efiboot, I plan to change its name to efivarcli since its not limited to boot variables stuff). Plus, I kinda wanted an excuse to talk to you about this, since I think it would be an asshole move to just fork the project and move on without talking to you about it

So.. would you be okay with my forking the project, and would you be okay with giving me access to publish the efivar crate ?

alixinne commented 1 year ago

Hello @iTrooz

Thanks for being upfront about this. I've been busy recently (and away for the past two weeks), hence the delay in reviewing PRs. I haven't caught up yet but looking at the recent code you pushed, I think it makes sense you could become the maintainer for this project.

Since I'm available to figure this out, here's my proposal:

I think the transfer option is preferable to a fork, since it'll automatically redirect users to the new home for the project, as well as keep all issues and open pull requests. What do you think?

iTrooz commented 1 year ago

I think that would be great to keep issues and PRs, yeah !

I add you as owner of efiboot and efivar crates on crates.io (there's no point in me keeping access to only one of those)

Just to be sure I understand, do you want me to remove your access from the crates once I am an owner ?

alixinne commented 1 year ago

I add you as owner of efiboot and efivar crates on crates.io (there's no point in me keeping access to only one of those)

Just to be sure I understand, do you want me to remove your access from the crates once I am an owner ?

Yes, but if you plan to stop maintaining the crates at some point in the future don't hesitate to reach out to me if you don't find another maintainer to replace you then.

iTrooz commented 1 year ago

Okay !

alixinne commented 1 year ago

I've sent you the invites for crates.io on both crates, but on GitHub you'll need to rename your copy of efiboot-rs to something else so I can initiate the transfer

iTrooz commented 1 year ago

I've just accepted the crates.io invitations, and I've renamed my fork

alixinne commented 1 year ago

Ok so it turns out you can't transfer a repository to an organization that contains a fork of the repository itself (iTrooz already has a repository in the vtavernier/efiboot-rs network).

I think you'd have to delete your fork, then we do the transfer and you can push your (local) branches to the new repository since you'll become an admin of the repository and re-open the PRs (they'll close automatically when deleting the fork I assume). Would that be okay with you?

iTrooz commented 1 year ago

Yep ! I just did it, and it indeed closed the PRs

alixinne commented 1 year ago

You should have received the transfer request now!

iTrooz commented 1 year ago

I have the repository !

Well.. thanks for everything, don't hesitate to come by in the future ^^

alixinne commented 1 year ago

Sure thing, thanks for taking on the maintenance!