Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 310757 - Latest mcelog recommends running as daemon
Summary: Latest mcelog recommends running as daemon
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal with 1 vote (vote)
Assignee: No maintainer - Look at https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers if you want to take care of it
URL:
Whiteboard:
Keywords:
Depends on: 399241
Blocks:
  Show dependency tree
 
Reported: 2010-03-22 17:20 UTC by Paul B. Henson
Modified: 2012-10-24 16:20 UTC (History)
3 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Updated ebuild installing init script (mcelog-1.0_pre3.ebuild,1.04 KB, text/plain)
2010-03-22 17:20 UTC, Paul B. Henson
Details
Example init script for mcelog (mcelog.init,377 bytes, text/plain)
2010-03-22 17:21 UTC, Paul B. Henson
Details
patch against mcelog-1.0_pre3.ebuild to handle cron use flag (mcelog-1.0_pre3.ebuild.diff,791 bytes, patch)
2010-08-23 09:18 UTC, Bruno Redondi
Details | Diff
${FILESDIR}/mcelog.init (mcelog.init,394 bytes, text/plain)
2010-10-19 21:23 UTC, Bruno Redondi
Details
A slightly more flexible mcelog init file. (mcelog.init,446 bytes, text/plain)
2010-11-16 16:07 UTC, David Pyke
Details
updated to be more flexble and include localmount (mcelog.init,465 bytes, text/plain)
2010-11-16 16:10 UTC, David Pyke
Details
new snapshot ebuild (mcelog-1.0_pre4_pre20110805.ebuild,1.13 KB, text/plain)
2012-01-18 02:55 UTC, Ben Kohler
Details

Note You need to log in before you can comment on or make changes to this bug.
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.