Bug 215596 - sysutils/mcelog: Updated to 146
Summary: sysutils/mcelog: Updated to 146
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Nikolai Lifanov
URL: https://github.com/andikleen/mcelog/c...
Keywords: easy, patch-ready
Depends on:
Blocks:
 
Reported: 2016-12-27 02:00 UTC by Ultima
Modified: 2016-12-29 19:00 UTC (History)
2 users (show)

See Also:


Attachments
mcelog.diff (1.29 KB, patch)
2016-12-27 02:00 UTC, Ultima
Ultima1252: maintainer-approval+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ultima 2016-12-27 02:00:24 UTC
Created attachment 178307 [details]
mcelog.diff

This is an update to mcelog to the most current version, 146.

* Updated to 146
* Added shebangfix

 - Fix malloc buffer overrun for unordered CPUs in sysfs 
 - Fix overriding of DEBUG for make test
 - Change all test scripts to /bin/bash to handle Debian
 - Run all tests in make test

portlint -AC:
WARN: Makefile: for new port, make $FreeBSD$ tag in comment section empty, to make SVN happy.
WARN: /usr/local/poudriere/ports/test/sysutils/mcelog/pkg-message: possible use of absolute pathname "/path/to/log". (false positive)
0 fatal errors and 2 warnings found.

poudriere bulk -tC:
12amd64: success https://poudriere.ultimasbox.com/data/12amd64-test/2016-12-26_20h51m19s/logs/mcelog-146.log
12i386: success https://poudriere.ultimasbox.com/data/12i386-test/2016-12-26_20h51m20s/logs/mcelog-146.log
110amd64: success https://poudriere.ultimasbox.com/data/110amd64-test/2016-12-26_20h51m17s/logs/mcelog-146.log
110i386: success https://poudriere.ultimasbox.com/data/110i386-test/2016-12-26_20h51m18s/logs/mcelog-146.log
101amd64: success https://poudriere.ultimasbox.com/data/101amd64-test/2016-12-26_20h51m15s/logs/mcelog-146.log
101i386: success https://poudriere.ultimasbox.com/data/101i386-test/2016-12-26_20h51m16s/logs/mcelog-146.log
93amd64: success https://poudriere.ultimasbox.com/build.html?mastername=93amd64-test&build=2016-12-26_20h51m21s
93i386: success https://poudriere.ultimasbox.com/build.html?mastername=93i386-test&build=2016-12-26_20h51m22s
Comment 1 Nikolai Lifanov freebsd_committer freebsd_triage 2016-12-29 14:24:59 UTC
I got some feedback and

TEST_DEPENDS=	bin/bash:shells/bash

should be

TEST_DEPENDS=	bash:shells/bash

because bash is a command.

Do you mind if I commit this with this one change?
Comment 2 Ultima 2016-12-29 15:31:35 UTC
(In reply to Nikolai Lifanov from comment #1)
ah yes, good catch. Go for it!
Comment 3 commit-hook freebsd_committer freebsd_triage 2016-12-29 18:55:48 UTC
A commit references this bug:

Author: lifanov
Date: Thu Dec 29 18:55:25 UTC 2016
New revision: 429939
URL: https://svnweb.freebsd.org/changeset/ports/429939

Log:
  sysutils/mcelog: Updated to 146

  * Updated to 146
  * Added shebangfix

   - Fix malloc buffer overrun for unordered CPUs in sysfs
   - Fix overriding of DEBUG for make test
   - Change all test scripts to /bin/bash to handle Debian
   - Run all tests in make test

  PR:		215596
  Submitted by:	Ultima1252@gmail.com (maintainer)
  Reviewed by:	matthew, mat
  Approved by:	matthew (mentor)
  Differential Revision:	https://reviews.freebsd.org/D8957

Changes:
  head/sysutils/mcelog/Makefile
  head/sysutils/mcelog/distinfo
Comment 4 Nikolai Lifanov freebsd_committer freebsd_triage 2016-12-29 19:00:21 UTC
Committed, thanks!