Skip to content
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

ytop: remove package #38644

Merged
merged 1 commit into from Aug 21, 2022
Merged

ytop: remove package #38644

merged 1 commit into from Aug 21, 2022

Conversation

hervyqa
Copy link
Contributor

@hervyqa hervyqa commented Aug 13, 2022

fix: #37240

Testing the changes

  • I tested the changes in this PR: NO

@classabbyamp
Copy link
Member

does it still build and run? If it still works, I don't see a reason to remove it at the moment, it's only two years past when the dev abandoned it.

@classabbyamp classabbyamp added the remove Package removal label Aug 13, 2022
@hervyqa
Copy link
Contributor Author

hervyqa commented Aug 13, 2022

@classabbyamp that's right, ytop is still working properly. but in the void there is already btm with a similar program. and both are built with cargo.

@paper42
Copy link
Member

paper42 commented Aug 13, 2022

I also don't see a reason to remove it. It builds, works and a lot of people are probably still using it without knowing it's abandoned and that btm exists.

@paper42 paper42 closed this Aug 13, 2022
@paper42
Copy link
Member

paper42 commented Aug 20, 2022

@hervyqa I just noticed ytop is not compiling anymore, upstream is dead and we are the only distribution that still packages it, let's reopen this PR.

@paper42 paper42 reopened this Aug 20, 2022
@leahneukirchen leahneukirchen merged commit 61a39c4 into void-linux:master Aug 21, 2022
@hervyqa hervyqa deleted the ytop branch October 2, 2022 02:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
remove Package removal
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ytop: no longer maintained
4 participants