Arch Linux

Please read this before reporting a bug:
https://wiki.archlinux.org/title/Bug_reporting_guidelines

Do NOT report bugs when a package is just outdated, or it is in the AUR. Use the 'flag out of date' link on the package page, or the Mailing List.

REPEAT: Do NOT report bugs for outdated packages!
Tasklist

FS#23224 - [htop] --sort-key doesn't work

Attached to Project: Arch Linux
Opened by Roman Kosenko (kite) - Thursday, 10 March 2011, 14:45 GMT
Last edited by Angel Velasquez (angvp) - Wednesday, 18 May 2011, 04:44 GMT
Task Type Bug Report
Category Packages: Extra
Status Closed
Assigned To Angel Velasquez (angvp)
Architecture All
Severity Medium
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:
In the version 0.8.3-1 there is a good feature: parameter --sort-key or -s.
In the version 0.9-1 it becomes broken - htop prints help instead.
Suggested patch is attached.

Steps to reproduce:
$ htop --sort-key PERCENT_MEM
This task depends upon

Closed by  Angel Velasquez (angvp)
Wednesday, 18 May 2011, 04:44 GMT
Reason for closing:  Implemented
Additional comments about closing:  Thanks for the patch Roman and sorry for the inconveniences applied on htop-0.9-2
Comment by Greg (dolby) - Thursday, 10 March 2011, 15:44 GMT
Please report this upstream and post the link here.
Comment by Angel Velasquez (angvp) - Wednesday, 11 May 2011, 20:33 GMT
ping @kite

Did you reported this? it was accepted?
Comment by Roman Kosenko (kite) - Thursday, 12 May 2011, 17:34 GMT
Angel Velasquez, are you joking?
Nothing will come of reporting this issue, moreover - it's already reported several times a long time ago and even with suggested patches.
Comment by Angel Velasquez (angvp) - Thursday, 12 May 2011, 18:16 GMT
Ok, would you mind to provide a link where you suggested this patch, and since you suggested it, can I ask you if it was accepted, could you understand that I won't apply any patches not accepted by upstream, right?
Comment by Karol Błażewicz (karol) - Thursday, 12 May 2011, 18:27 GMT
Another bug report that deals with patches & up[stream: https://bugs.archlinux.org/task/23324
Are we dealing with this on a case-by-case basis?
Comment by Roman Kosenko (kite) - Thursday, 12 May 2011, 21:12 GMT
Angel Velasquez, I didn't report upstream - I don't want to waste my time - they will not read. You can simply use search and found almost the same pathes, for example:
http://sourceforge.net/tracker/?func=detail&aid=3152252&group_id=108839&atid=651635

You can open this patch and realize what this stupid bug is. This patch isn't rejected - it isn't accepted because developers по-русски похуй.
Comment by Angel Velasquez (angvp) - Thursday, 12 May 2011, 22:15 GMT
Hey Roman Kosenko,

We are nice people here, please calm down a little bit, or low a little bit your tone.

I remember to have tested this patch the same day that I saw for first time on the bugtracker. I think is silly and yes, it's usable, and certainly fix the problem, but I thought that is more productive to have it upstream instead having us (devs of a linux distro) patching their software anytime they have something broken.

I will apply this, since I know it fix a problem, I've reviewed the code (there are very few lines) and is a secure IMO, hoping that the htop devs will apply it (they didn't denied yet) on the next release .. thanks for your contribution.

Oh, please be kind next time on bugreports there is no need to be harsh.
Comment by Roman Kosenko (kite) - Friday, 13 May 2011, 14:42 GMT
> I thought that is more productive to have it upstream instead having us (devs of a linux distro) patching their software anytime they have something broken.
I think that self-respectful distributive should use only quality software. If it is broken - fix them, use old stable version or don't use at all.
Comment by Angel Velasquez (angvp) - Tuesday, 17 May 2011, 15:27 GMT
Привет Роман, в мире софта очень сложно зделать так, чтобы все работало без проблем и обычно, возвращаться к предыдущей версии, из-за чего-то очень простого не имеет смысла, я применил патч для того, чтобы все было наиболее функциональным и актуализированым, спасибо за понимание

Loading...