From owner-freebsd-ports@FreeBSD.ORG Thu Jul 14 19:34:20 2011 Return-Path: Delivered-To: freebsd-ports@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id D9FE9106564A for ; Thu, 14 Jul 2011 19:34:20 +0000 (UTC) (envelope-from lists@eitanadler.com) Received: from mail-vw0-f54.google.com (mail-vw0-f54.google.com [209.85.212.54]) by mx1.freebsd.org (Postfix) with ESMTP id 93C4C8FC12 for ; Thu, 14 Jul 2011 19:34:20 +0000 (UTC) Received: by vws18 with SMTP id 18so602139vws.13 for ; Thu, 14 Jul 2011 12:34:19 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=eitanadler.com; s=0xdeadbeef; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc:content-type; bh=G5gfDYN7mhfIhXSqECz2//zQXqNbzigCbFdi9L4lzoU=; b=Ne6awjjkXRm7rQh6aZNFS4LJ5WR1BxXFp3tnmdeYJz+QWwpLW/Mc8ioArCQu7uRtrY FTHb2atr0yac5hGRrW5bTKrmKkHavqhwoRZ9ObvFvmi49o0q0oPsoAs6rPR7VwI7WxIX rSSwTsiqWW7Bt7Ir75uf3w82OYtmLKLMMCUxs= Received: by 10.52.31.1 with SMTP id w1mr2797476vdh.371.1310670175177; Thu, 14 Jul 2011 12:02:55 -0700 (PDT) MIME-Version: 1.0 Received: by 10.52.165.74 with HTTP; Thu, 14 Jul 2011 12:02:25 -0700 (PDT) In-Reply-To: <64bc4d1f59e39f71f77ced1aed64e734@etoilebsd.net> References: <4E0C5B7A.5060102@FreeBSD.org> <4E109521.10209@FreeBSD.org> <64bc4d1f59e39f71f77ced1aed64e734@etoilebsd.net> From: Eitan Adler Date: Thu, 14 Jul 2011 15:02:25 -0400 Message-ID: To: Baptiste Daroussin , Chris Rees Content-Type: text/plain; charset=UTF-8 Cc: freebsd-ports@freebsd.org Subject: Re: Call for testers -- CONF_FILES variable X-BeenThere: freebsd-ports@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Porting software to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 14 Jul 2011 19:34:20 -0000 > The reason I choose pkgconf (we can change that name) is that it concerns > only configuration files that the maintainers DO want. > > I want to make sure that maintainers are looking at the samples the proprose > to provide a usable sample, not the default one from the distfile (the > default one can still be provided as an example.) So, this suffix is only for configuration files that port maintainers write and included sample files from upstream would not have this suffix? Why would the maintainer be writing sample conf files? It is not the maintainer's job to write documentation for the upstream project. The only case I could see this becoming an issue is if the default configuration file ignores hier(1) and a REINPLACE is needed. I do _not_ want to see sample configuration files being written for ports unless a considerable amount of rework is needed to make the application run on FreeBSD. > I wanted that pkgng and the ports in general can manage default usable > configuration files, and to distinguish them from the samples. Thanks crees@ > has done the job I wanted to do myself so that and he has done it right. I am confused. I thought Chris's option was for the upstream sample configuration files.My understanding is that it replaces the logic of "only delete the real config file if it does not differ from the sample file". Why then does it matter who wrote the sample since the logic works the same way? Either we will need multiple copies of this macro, one for "official" files and the other for "package" files or the logic will still have to be replicated per port for non-included samples. IMHO the suffix (and type of sample file) should not be touched by the macro. Perhaps I misunderstand what will be new in pkgng or what this patch provides? -- Eitan Adler