Bug 207638

Summary: sysutils/minimunin: cpu configuration
Product: Ports & Packages Reporter: Xavier Garcia <xavi.garcia>
Component: Individual Port(s)Assignee: freebsd-ports-bugs (Nobody) <ports-bugs>
Status: Closed Overcome By Events    
Severity: Affects Only Me CC: erdgeist, lme
Priority: --- Keywords: needs-qa, patch
Version: LatestFlags: erdgeist: maintainer-feedback+
Hardware: amd64   
OS: Any   
Attachments:
Description Flags
Patch that adds the missing white space rakuco: maintainer-approval? (erdgeist)

Description Xavier Garcia 2016-03-02 09:57:41 UTC
Created attachment 167629 [details]
Patch that adds the missing white space

There is a typo in the bash expression that extracts the clockrate from sysconfig because there is a white space missing.

Tested on FreeBSD 10 but not on FreeBSD 9.


As a result, the Munin master cannot evaluate the CDEF expression and fails to configure the CPU graphs.


Steps to reproduce the bug:

Notice the white space before 127

# echo "config cpu" | minimunin | grep cdef
system.cdef system, 127,/,100,*
interrupt.cdef interrupt, 127,/,100,*
user.cdef user, 127,/,100,*
nice.cdef nice, 127,/,100,*
idle.cdef idle, 127,/,100,*


Attached patch with the fix
Comment 1 erdgeist 2016-03-04 13:35:22 UTC
Commited in

https://erdgeist.org/gitweb/minimunin/commit/?id=941288a85098c489a268d40adddd4751545d057e

Will roll new port version soon.

Thanks

 erdgeist
Comment 2 erdgeist 2016-03-17 23:34:21 UTC
New port version submitted in bug #208105.