archlinux / alpm.rs

Rust bindings for libalpm
GNU General Public License v3.0
112 stars 21 forks source link

Upstream some `Alpm` methods #30

Open fosskers opened 2 years ago

fosskers commented 2 years ago

This PR upstreams some local functionality I've had for a while, but it's better suited to live here than in Aura.

Morganamilo commented 2 years ago

Right. I don't really know how this ended up like this. I think I have have just been experimenting and accidentally committed the file I don't know.

fosskers commented 2 years ago

Looks like my email message didn't get delivered to the PR for some reason. Oh well. Anyway I was on vacation for a few days; fixing this now.

fosskers commented 2 years ago

Should be good to go now.

fosskers commented 1 year ago

Is this okay to merge?

fosskers commented 1 year ago

Any further thoughts?

Morganamilo commented 1 year ago

Sorry bit busy. Also you removed the alpmext trait (even though it was dead code anyway). I think the trait should remain and these methods be on it. Especially if one day I wish to add these functions back.

fosskers commented 1 year ago

Roger, no problem. Busy here too, settling into a new job in a new country.

fosskers commented 1 year ago

Hey Morgan, just out of town for a few days, I’ll fix this up once I’m back.

iPhoneから送信

2022/08/20 18:34、Lulu @.***>のメール:

 Right. I don't really know how this ended up like this. I think I have have just been experimenting and accidentally committed the file I don't know.

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread.

fosskers commented 1 year ago

Finally updated!