From owner-freebsd-ports-bugs@FreeBSD.ORG Wed Nov 5 07:16:33 2014 Return-Path: Delivered-To: freebsd-ports-bugs@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 46216DDA for ; Wed, 5 Nov 2014 07:16:33 +0000 (UTC) Received: from kenobi.freebsd.org (kenobi.freebsd.org [IPv6:2001:1900:2254:206a::16:76]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 2D69EB9C for ; Wed, 5 Nov 2014 07:16:33 +0000 (UTC) Received: from bugs.freebsd.org ([127.0.1.118]) by kenobi.freebsd.org (8.14.9/8.14.9) with ESMTP id sA57GXTR091651 for ; Wed, 5 Nov 2014 07:16:33 GMT (envelope-from bugzilla-noreply@freebsd.org) From: bugzilla-noreply@freebsd.org To: freebsd-ports-bugs@FreeBSD.org Subject: [Bug 194825] [PATCH] Make sysutils/mcelog build on DragonFly Date: Wed, 05 Nov 2014 07:16:33 +0000 X-Bugzilla-Reason: AssignedTo X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: Ports Tree X-Bugzilla-Component: Individual Port(s) X-Bugzilla-Version: Latest X-Bugzilla-Keywords: X-Bugzilla-Severity: Affects Many People X-Bugzilla-Who: marino@FreeBSD.org X-Bugzilla-Status: Needs Triage X-Bugzilla-Priority: --- X-Bugzilla-Assigned-To: freebsd-ports-bugs@FreeBSD.org X-Bugzilla-Target-Milestone: --- X-Bugzilla-Flags: maintainer-feedback+ X-Bugzilla-Changed-Fields: Message-ID: In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit X-Bugzilla-URL: https://bugs.freebsd.org/bugzilla/ Auto-Submitted: auto-generated MIME-Version: 1.0 X-BeenThere: freebsd-ports-bugs@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: Ports bug reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 05 Nov 2014 07:16:33 -0000 https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=194825 --- Comment #3 from John Marino --- (In reply to Jeremy Chadwick from comment #2) > I approve of this patch (updating Makefile and files/patch-Makefile. > > One thing I want to make clear here, however: the CFLAGS definition (of -Os > -g) in the software's Makefile is somewhat justified. If you read the top > of that Makefile, they explain why: it has to do with compiler warning flags > catching certain scenarios only when -Os is in use (compared to just -O2 or > -O or stock defaults). That's something that needs to be kept in mind going > forward. Wouldn't that be essentially a developer flag then? In any case, the blocking of CFLAGS from ports is considered an error. There are three ways to resolve the error. 1) Remove the pre-definition (as ftigeot did) 2) Change the makefile to accept the addition of new CFLAGS (not possible here because -g can't be switched off and -O would conflict) 3) Remove the definition from vendor makefile but put them back in ports makefile (where they can be changed or overridden easily) I think the approach ftigeot is using is okay. As ports users we're not that interested in compilers warning about actual code, those are for the developers and the code is considered release quality now. -- You are receiving this mail because: You are the assignee for the bug.