Macchina-CLI / macchina

A system information frontend with an emphasis on performance.
https://crates.io/crates/macchina
MIT License
1.48k stars 51 forks source link

Packaging for Cargo is temporarily discontinued; we'd like your opinion #333

Closed grtcdr closed 2 weeks ago

grtcdr commented 3 weeks ago

I recently made the decision to discontinue packaging macchina on crates.io, this change can be traced back to this commit.

That change was my attempt at providing a solution to a recurring problem we've been having concerning two packages we share a dependency with, ansi-to-tui and color-to-tui (macchina and these two packages all share a dependency on ratatui).

We have been having monthly reports of build issues (#320 #326 #332) for people installing the package using cargo and despite a number of attempts to pin down the version numbers of ratatui, ansi-to-tui and color-to-tui, the issue would always come back.

I'm posting this issue to gather feedback on this decision and to arrive at a better solution than I could come up with and ultimately to find consensus.

If you have any ideas as to what we can do to avoid future build errors whilst still maintaining a presence on crates.io, please let us know.

Note that @0323pin has reached out to me (in a private email) about how the changes in the aforementioned commit have affected him as a package maintainer. If you too are maintaining macchina for users of your distribution, we value your opinion and would love to hear your feedback as well.

grtcdr commented 3 weeks ago

@123marvin123 and @uttarayan21 I'd love to hear your thoughts on this one.

grtcdr commented 3 weeks ago

@0323pin I've just merged #334, the included changes are a far better solution than vendoring, the next release will not include the current submodules. I apologize for all the trouble my "fix" put you through.

0323pin commented 3 weeks ago

That's good. But, it was rather straightforward and I've learnt something new.