Summary: | sysutils/mcelog: Updated to 148 | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | Ports & Packages | Reporter: | Ultima <Ultima1252> | ||||||
Component: | Individual Port(s) | Assignee: | Nikolai Lifanov <lifanov> | ||||||
Status: | Closed FIXED | ||||||||
Severity: | Affects Only Me | CC: | lifanov, pi | ||||||
Priority: | --- | Keywords: | easy, patch-ready | ||||||
Version: | Latest | ||||||||
Hardware: | Any | ||||||||
OS: | Any | ||||||||
URL: | https://github.com/andikleen/mcelog | ||||||||
Attachments: |
|
Description
Ultima
2017-02-04 18:30:37 UTC
Hi! I don't think the git version thing is desirable. We don't want to produce a different package depending on what the user has installed when building a port. This needs to be addressed by either depending on git or removing the section. This also highlights that the top of the file always sets prefix to /usr/local, while PREFIX is a user-defined variable that can be set to something else. Perhaps both this and the CFLAGS components of the patch could be replaced by adding "MAKE_ARGS+= CFLAGS=${CFLAGS} prefix=${PREFIX}" to the FreeBSD port Makefile instead? I propose this patch instead: Created attachment 179677 [details]
proposed mcelog patch
o respect PREFIX set in make.conf
o produce the same package regardless of whether git is installed
Comment on attachment 179677 [details]
proposed mcelog patch
Looks good to me.
Comment on attachment 179607 [details] mcelog.diff >Index: sysutils/mcelog/Makefile >=================================================================== >--- sysutils/mcelog/Makefile (revision 433327) >+++ sysutils/mcelog/Makefile (working copy) >@@ -2,7 +2,7 @@ > # $FreeBSD$ > > PORTNAME= mcelog >-PORTVERSION= 147 >+PORTVERSION= 148 > DISTVERSIONPREFIX= v > CATEGORIES= sysutils > >Index: sysutils/mcelog/distinfo >=================================================================== >--- sysutils/mcelog/distinfo (revision 433327) >+++ sysutils/mcelog/distinfo (working copy) >@@ -1,3 +1,3 @@ >-TIMESTAMP = 1484491926 >-SHA256 (andikleen-mcelog-v147_GH0.tar.gz) = f18c5c12a8c8addf56f66f0046a8f0c7ba262f48f95bf57abed8430df0eeebe6 >-SIZE (andikleen-mcelog-v147_GH0.tar.gz) = 296947 >+TIMESTAMP = 1486231306 >+SHA256 (andikleen-mcelog-v148_GH0.tar.gz) = 3de7312ee13b0968ae8e1aa0a77acaaee89a0ed7de45f7ce557939723df3dc04 >+SIZE (andikleen-mcelog-v148_GH0.tar.gz) = 296927 >Index: sysutils/mcelog/files/patch-Makefile >=================================================================== >--- sysutils/mcelog/files/patch-Makefile (revision 433327) >+++ sysutils/mcelog/files/patch-Makefile (working copy) >@@ -1,4 +1,4 @@ >---- Makefile.orig 2016-10-18 22:32:19 UTC >+--- Makefile.orig 2017-02-04 00:51:04 UTC > +++ Makefile > @@ -1,5 +1,4 @@ > -CFLAGS := -g -Os >@@ -41,15 +41,6 @@ > > # dbquery intentionally not installed by default > install: mcelog mcelog.conf mcelog.conf.5 mcelog.triggers.5 >-@@ -82,7 +88,7 @@ depend: .depend >- >- version.tmp: FORCE >- ( echo -n "char version[] = \"" ; \ >-- if type -p git >/dev/null; then \ >-+ if command -v git >/dev/null; then \ >- if [ -d .git ] ; then \ >- git describe --tags HEAD | tr -d '\n'; \ >- else \ > @@ -98,8 +104,6 @@ version.c: version.tmp > .depend: ${SRC} > ${CC} -MM -I. ${SRC} > .depend.X && mv .depend.X .depend Trying to add a +, but it won't let me =/ Hi! Don't worry about it! A commit references this bug: Author: lifanov Date: Mon Feb 6 18:49:27 UTC 2017 New revision: 433508 URL: https://svnweb.freebsd.org/changeset/ports/433508 Log: update sysutils/mcelog to 148 - Intel Xeons from Ivy Bridge onwards support a processor identification number. - Fix warning with gcc 6.x - replace asprintf with xasprintf as per memutil.c - Return -ENOMEM for vasprintf in sysfs_write PR: 216789 Submitted by: ultima1252@gmail.com (maintainer) Reviewed by: matthew Approved by: matthew (mentor) Differential Revision: https://reviews.freebsd.org/D9454 Changes: head/sysutils/mcelog/Makefile head/sysutils/mcelog/distinfo head/sysutils/mcelog/files/patch-Makefile Committed, thanks! |