Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 310757

Summary: Latest mcelog recommends running as daemon
Product: Gentoo Linux Reporter: Paul B. Henson <henson>
Component: New packagesAssignee: No maintainer - Look at https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers if you want to take care of it <maintainer-needed>
Status: RESOLVED FIXED    
Severity: normal CC: bkohler, gwyn.connor, joshua.rich
Priority: High    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on: 399241    
Bug Blocks:    
Attachments: Updated ebuild installing init script
Example init script for mcelog
patch against mcelog-1.0_pre3.ebuild to handle cron use flag
${FILESDIR}/mcelog.init
A slightly more flexible mcelog init file.
updated to be more flexble and include localmount
new snapshot ebuild

Description Paul B. Henson 2010-03-22 17:20:08 UTC
The latest mcelog (currently 1.0_pre3 in portage) recommends running as a daemon instead of via cron. The ebuild should include an init script instead of the current cron and logrotate files, or perhaps have a use flag allowing a choice between them. I'm attaching an example updated ebuild and init script.
Comment 1 Paul B. Henson 2010-03-22 17:20:49 UTC
Created attachment 224731 [details]
Updated ebuild installing init script
Comment 2 Paul B. Henson 2010-03-22 17:21:16 UTC
Created attachment 224733 [details]
Example init script for mcelog
Comment 3 Bruno Redondi 2010-08-23 09:18:17 UTC
Created attachment 244175 [details, diff]
patch against mcelog-1.0_pre3.ebuild to handle cron use flag
Comment 4 Bruno Redondi 2010-08-23 09:21:21 UTC
mcelog is needed also on x86 systems, not only on amd64. imho this bug must be reassigned.
Comment 5 Bruno Redondi 2010-10-19 21:23:12 UTC
Created attachment 251289 [details]
${FILESDIR}/mcelog.init

added dependency on localmount.
if /usr is on a separate filesystem, with baselayout2 and rc_parallel="YES" it needs localmount to have /usr mounted.
Comment 6 David Pyke 2010-11-16 16:07:51 UTC
Created attachment 254513 [details]
A slightly more flexible mcelog init file.
Comment 7 David Pyke 2010-11-16 16:10:24 UTC
Created attachment 254515 [details]
updated to be more flexble and include localmount

didn't read to the end before submitting.
Comment 8 Paul B. Henson 2010-11-22 22:22:28 UTC
Any thoughts from a dev on getting the in-portage version to include the init script?
Comment 9 Markos Chandras (RETIRED) gentoo-dev 2010-11-22 22:54:24 UTC
Yes patch looks good. I will try to fix this bug soonish
Comment 10 Bruno Redondi 2011-06-29 09:11:33 UTC
any news on this?
Comment 11 Ben Kohler gentoo-dev 2012-01-18 02:55:06 UTC
Created attachment 299195 [details]
new snapshot ebuild

Attached is a new ebuild based on 1.0_pre3 from portage.  Since upstream doesn't seem to be releasing any new tarballs, this is based on a specific git revision (latest as of today), it pulls a tarball straight from git.kernel.org's gitweb.  Includes mcelog.init from above, but I am not sure that script is totally right yet.

Added kernel config check, dropped virtual/cron dep as the system service is the preferred run method now.  The cron.daily file is still installed but not active by default now.

Builds and runs fine here.  I know the versioning is a bit funny but upstream has released pre3 and may release pre4 so I was advised to set a dated version between those releases.  Open to more suggestions.
Comment 12 Ben Kohler gentoo-dev 2012-01-18 02:57:17 UTC
Just realized this wasn't a version bump request bug.  Will create new report.
Comment 13 Agostino Sarubbo gentoo-dev 2012-08-19 12:41:53 UTC
This package is now maintainer-needed. If you want proxy-maintain it, poke the relative herd.