Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 519494 - app-admin/mcelog-103 version bump
Summary: app-admin/mcelog-103 version bump
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal enhancement (vote)
Assignee: Julian Ospald
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-08-09 12:17 UTC by Lars Wendler (Polynomial-C) (RETIRED)
Modified: 2014-09-09 01:59 UTC (History)
2 users (show)

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


Attachments
mcelog.init (mcelog.init,361 bytes, text/plain)
2014-08-09 12:17 UTC, Lars Wendler (Polynomial-C) (RETIRED)
Details
~/mcelog-init-simplify.patch (mcelog-init-simplify.patch,983 bytes, patch)
2014-09-08 21:40 UTC, Ben Kohler
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Lars Wendler (Polynomial-C) (RETIRED) gentoo-dev 2014-08-09 12:17:22 UTC
Created attachment 382582 [details]
mcelog.init

mcelog-102 is out since July 4th.

I took the liberty to add a rewritten init script which makes use of default start/stop functions from openrc (which should save us up some bytes in size).
Comment 1 Julian Ospald 2014-08-09 12:54:47 UTC
(In reply to Lars Wendler (Polynomial-C) from comment #0)
> Created attachment 382582 [details]
> mcelog.init
> 
> mcelog-102 is out since July 4th.
> 
> I took the liberty to add a rewritten init script which makes use of default
> start/stop functions from openrc (which should save us up some bytes in
> size).

If you use it, just add yourself as maintainer. I am merely proxying for bkohler.
Comment 2 Lars Wendler (Polynomial-C) (RETIRED) gentoo-dev 2014-08-09 13:19:10 UTC
Unfortunately I cannot really use it. 
Funny story, each time I want to check my MCE logs I wonder why I don't have the package installed. After installation the first run remembers me why:

  # mcelog 
  mcelog: AMD Processor family 16: Please load edac_mce_amd module.
  : Success
  CPU is unsupported

But I always forget about this. Today was at least the fith time I installed mcelog just to get rememberd why I don't need it on my systems... :-/
Comment 3 Ben Kohler gentoo-dev 2014-09-08 21:38:09 UTC
103 was just released, it's a trivial ebuild bump, but I went ahead and also tested with Poly-C's new init script (filename bumped to -r1), works well.

--- /usr/portage/app-admin/mcelog/mcelog-100.ebuild	2014-06-15 19:01:02.000000000 -0500
+++ mcelog-103.ebuild	2014-09-08 16:34:19.163578484 -0500
@@ -38,7 +38,7 @@
 	insinto /etc/logrotate.d/
 	newins ${PN}.logrotate ${PN}
 
-	newinitd "${FILESDIR}"/${PN}.init ${PN}
+	newinitd "${FILESDIR}"/${PN}.init-r1 ${PN}
 	systemd_dounit "${FILESDIR}"/${PN}.service
 
 	insinto /etc/${PN}
Comment 4 Ben Kohler gentoo-dev 2014-09-08 21:40:25 UTC
Created attachment 384420 [details, diff]
~/mcelog-init-simplify.patch

To be thorough, here are Poly-C's init script changes in diff form.
Comment 5 Ben Kohler gentoo-dev 2014-09-08 21:43:30 UTC
(In reply to Ben Kohler from comment #3)
> 103 was just released, it's a trivial ebuild bump, but I went ahead and also
> tested with Poly-C's new init script (filename bumped to -r1), works well.
> 
> --- /usr/portage/app-admin/mcelog/mcelog-100.ebuild	2014-06-15
> 19:01:02.000000000 -0500
> +++ mcelog-103.ebuild	2014-09-08 16:34:19.163578484 -0500
> @@ -38,7 +38,7 @@
>  	insinto /etc/logrotate.d/
>  	newins ${PN}.logrotate ${PN}
>  
> -	newinitd "${FILESDIR}"/${PN}.init ${PN}
> +	newinitd "${FILESDIR}"/${PN}.init-r1 ${PN}
>  	systemd_dounit "${FILESDIR}"/${PN}.service
>  
>  	insinto /etc/${PN}

Not that I would expect you to forget this, hasufell, but I should have reset keywords to ~arch for the above patch, forgot 100 was stable already.

Thanks!
Comment 6 Julian Ospald 2014-09-09 01:59:21 UTC
+*mcelog-103 (09 Sep 2014)
+
+  09 Sep 2014; Julian Ospald <hasufell@gentoo.org> +mcelog-103.ebuild,
+  +files/mcelog.init-r1:
+  version bump by Ben Kohler and Lars Wendler wrt #519494