Bug 122637 - Please add ~alpha keyword for app-admin/conky-1.4.1
Bug#: 122637 Product:  Gentoo Linux Version: 2005.1 Platform: Alpha
OS/Version: Linux Status: RESOLVED Severity: enhancement Priority: P2
Resolution: FIXED Assigned To: alpha@gentoo.org Reported By: linuxfreakus@gmail.com
Component: Applications
URL: 
Summary: Please add ~alpha keyword for app-admin/conky-1.4.1
Keywords:  
Status Whiteboard: 
Opened: 2006-02-12 19:12 0000
Description:   Opened: 2006-02-12 19:12 0000
Went to get torsmo and noticed it has been removed, so I thought I'd try conky.
As far as I can tell conky 1.3.5 seems to work great on my alpha (ev56). All I
did was edit the keywords on the ebuild, it compiled, installed and works
perfectly, didn't even have to do any work.  I would love it if someone could
just add the ~alpha keyword to the ebuild so it won't get clobbered on my end
every time I run a sync.

------- Comment #1 From Daniel Black 2006-02-18 15:28:47 0000 -------
This is supported by me and Brenden (upstream maintainer).

------- Comment #2 From Mike Hlavac 2006-02-25 11:39:45 0000 -------
I've confirmed that app-admin/conky-1.4.0 works as well, however the Frequency
in MHz and Frequency in GHz both display as 0.

------- Comment #3 From Thomas Cort (RETIRED) 2006-03-05 16:08:25 0000 -------
Created an attachment (id=81467) [details]
Corrects CPU Frequency for Alpha

(In reply to comment #2)
> I've confirmed that app-admin/conky-1.4.0 works as well, however the Frequency
> in MHz and Frequency in GHz both display as 0.

I tested conky-1.4.0-r1 and ran into the same problem as Mike, the "Frequency
in MHz" and "Frequency in GHz" both display as 0. The problem is CPU frequency
in /proc/cpuinfo is different on alpha. For example, my 500MHz Alpha has 
"cycle frequency [Hz]    : 499695364 est."
while a 500MHz x86 system would have something like
"cpu MHz         : 499.695"

This patch adds support for the cycle frequency used by /proc/cpuinfo on alpha.
I've tested it on alpha, and it works as expected. However, I haven't tested
this patch on non-alpha systems, so it should be tested on other arches before
it gets added.

------- Comment #4 From Brenden Matthews 2006-03-05 16:31:49 0000 -------
(In reply to comment #3)
> Created an attachment (id=81467) [edit] [details]
> Corrects CPU Frequency for Alpha
> 
> (In reply to comment #2)
> > I've confirmed that app-admin/conky-1.4.0 works as well, however the Frequency
> > in MHz and Frequency in GHz both display as 0.
> 
> I tested conky-1.4.0-r1 and ran into the same problem as Mike, the "Frequency
> in MHz" and "Frequency in GHz" both display as 0. The problem is CPU frequency
> in /proc/cpuinfo is different on alpha. For example, my 500MHz Alpha has 
> "cycle frequency [Hz]    : 499695364 est."
> while a 500MHz x86 system would have something like
> "cpu MHz         : 499.695"
> 
> This patch adds support for the cycle frequency used by /proc/cpuinfo on alpha.
> I've tested it on alpha, and it works as expected. However, I haven't tested
> this patch on non-alpha systems, so it should be tested on other arches before
> it gets added.
> 

Thanks a plenty.  Committed to CVS.

------- Comment #5 From Eric Nichols 2006-04-04 08:09:02 0000 -------
How do we get the keyword ~alpha added then?  This bug has been here for a
while and I've since updated to the latest version, it still works fine.  Is
there a reason the keyword still doesn't show up when I do a sync? Do I need to
post this someplace else?

------- Comment #6 From Thomas Cort (RETIRED) 2006-04-04 08:32:23 0000 -------
(In reply to comment #5)
> How do we get the keyword ~alpha added then?  This bug has been here for a
> while and I've since updated to the latest version, it still works fine.  Is
> there a reason the keyword still doesn't show up when I do a sync? Do I need
> to post this someplace else?

conky 1.4.1 includes the patch attached to this bug (Attachment #81467 [details])[1]. The
patch fixes CPU frequency detection on Alpha. Once conky 1.4.1 hits portage[2],
I'll test it, add the ~alpha keyword if there are no problems, and close this
bug.

In Comment #4, the phrase "Committed to CVS" meant that the patch was added to
the upstream cvs, not portage.

[1] http://conky.sourceforge.net/changelog.html
[2] http://bugs.gentoo.org/126008

------- Comment #7 From Thomas Cort (RETIRED) 2006-04-11 16:11:14 0000 -------
conky 1.4.1 is in portage now, but it doesn't compile for me, see Bug #129639.
I'll test and keyword conky once that bug is fixed. Thanks for being patient!

------- Comment #8 From Thomas Cort (RETIRED) 2006-04-12 09:45:11 0000 -------
Added ~alpha keyword.

Thanks to everyone who tested and helped get this program working on alpha.