From owner-svn-ports-head@FreeBSD.ORG Sat May 25 16:43:41 2013 Return-Path: Delivered-To: svn-ports-head@freebsd.org Received: from mx1.freebsd.org (mx1.FreeBSD.org [8.8.178.115]) by hub.freebsd.org (Postfix) with ESMTP id 12A17D01; Sat, 25 May 2013 16:43:41 +0000 (UTC) (envelope-from utisoft@gmail.com) Received: from mail-ie0-x22d.google.com (mail-ie0-x22d.google.com [IPv6:2607:f8b0:4001:c03::22d]) by mx1.freebsd.org (Postfix) with ESMTP id B530F97D; Sat, 25 May 2013 16:43:40 +0000 (UTC) Received: by mail-ie0-f173.google.com with SMTP id k5so15189827iea.32 for ; Sat, 25 May 2013 09:43:40 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:sender:in-reply-to:references:from:date :x-google-sender-auth:message-id:subject:to:cc:content-type; bh=gzXW9ZeMxd+bx6pY5zFWnF2XKT59VO1feWOYZQgSJAs=; b=JoqSmK0QP0BNvVDcX1m9yDX0jdePydrKiOv1yHwyqRXLZM12Bs6QM+7WPLVIjFJk7F G6s2/GNlATCx+0UNwXtijmRLB5/oJDghDeqiU8WJtmU6JhgAVWNJ4ItJePfMgDRG8nXu 4ZPYRSZIZW1B03enB+JoglAgXCqva/LB9II5+cD3Anov1HrL6hmgYP2atpe9S9HKlEMG LuY32YpFYZMRIVGLoIhBKbikjASmpSws7ImPa0rVeRUbhGG2rMFPVhDqM1yWbE45gvB1 XyLw3tb2vnxsVEOLpDKdeXpKu2KTVlvSno2UTvTxQIspppCB0gWF2haJaz4jmKJ2+ZVb 0YQA== X-Received: by 10.50.153.113 with SMTP id vf17mr1828862igb.101.1369500220413; Sat, 25 May 2013 09:43:40 -0700 (PDT) MIME-Version: 1.0 Sender: utisoft@gmail.com Received: by 10.64.23.243 with HTTP; Sat, 25 May 2013 09:43:09 -0700 (PDT) In-Reply-To: References: <201305232205.r4NM5MSa002349@svn.freebsd.org> <20130524033728.GE48975@FreeBSD.org> <20130525024425.GC89166@FreeBSD.org> From: Chris Rees Date: Sat, 25 May 2013 17:43:09 +0100 X-Google-Sender-Auth: 1LU-XAtVDxvw0tmjJQDehXgHMIU Message-ID: Subject: Re: svn commit: r318913 - in head/devel/libcfu: . files To: Alexey Dokuchaev Content-Type: text/plain; charset=ISO-8859-1 Cc: "svn-ports-head@freebsd.org" , svn-ports-all@freebsd.org, Eitan Adler , "ports-committers@freebsd.org" X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: SVN commit messages for the ports tree for head List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 25 May 2013 16:43:41 -0000 On 25 May 2013 10:01, Chris Rees wrote: > On 25 May 2013 03:44, Alexey Dokuchaev wrote: >> On Fri, May 24, 2013 at 07:14:09PM +0200, Eitan Adler wrote: >>> On 24 May 2013 05:37, Alexey Dokuchaev wrote: >>> > On Thu, May 23, 2013 at 10:05:22PM +0000, Chris Rees wrote: >>> >> New Revision: 318913 >>> >> URL: http://svnweb.freebsd.org/changeset/ports/318913 >>> >> >>> >> Log: >>> >> New versions of clang complain about assigning variables to themselves. >>> >> >>> >> Let's just remove -Werror and ignore the warnings. >>> > >>> > Hmm, wouldn't it be better to fix these bugs (judging from commit log it >>> > should not be too hard) and send the patch upstream? >>> >>> I agree fixing these bugs is a good thing. I don't believe that this >>> is a function of the ports team though. IMHO disabling warnings is >>> the simplest, least intrusive patch that works. >> >> Good maintainer typically would work with upstream to make sure their >> software builds and runs on FreeBSD with little possible special tuning >> required. Not just it is "being nice", it also helps us to create and >> maintain image of an OS that is worth writing software for. :) >> >> While I agree that investing time in a large complex patch when simple >> workaround is available is a bit too much (unless port maintainer is also >> on the upstream devteam), fixing *simple* things such as compiler warnings >> is better than throwing -Werror. > > Yes, however this software hasn't been updated since 2005. > > I'll consider talking to upstream, but my priority at the time was to > just get the stuff building and working-- it was a Thursday night and > I had a load of Real Life work to do when I got the failure > notification. While I was in the process of forking libcfu, I found someone else had it on github. I've fixed the bugs and passed them to them, and updated the port to the new site at the same time. Thanks for the suggestion, Chris