From owner-svn-ports-head@freebsd.org Sat May 21 11:27:28 2016 Return-Path: Delivered-To: svn-ports-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id C1EC3B442DF; Sat, 21 May 2016 11:27:28 +0000 (UTC) (envelope-from danfe@freebsd.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:1900:2254:206c::16:87]) by mx1.freebsd.org (Postfix) with ESMTP id B1DEB1796; Sat, 21 May 2016 11:27:28 +0000 (UTC) (envelope-from danfe@freebsd.org) Received: by freefall.freebsd.org (Postfix, from userid 1033) id AF7351CDC; Sat, 21 May 2016 11:27:28 +0000 (UTC) Date: Sat, 21 May 2016 11:27:28 +0000 From: Alexey Dokuchaev To: Ed Maste Cc: Baptiste Daroussin , ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: Re: svn commit: r415078 - in head: . Mk Message-ID: <20160521112728.GA624@FreeBSD.org> References: <201605121820.u4CIKROJ004026@repo.freebsd.org> <20160513160151.GA30219@FreeBSD.org> <20160513182837.GF49383@ivaldir.etoilebsd.net> <20160513201919.GA48945@FreeBSD.org> <20160519122306.GA24015@FreeBSD.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20160519122306.GA24015@FreeBSD.org> User-Agent: Mutt/1.6.0 (2016-04-01) X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.22 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, 21 May 2016 11:27:28 -0000 On Thu, May 19, 2016 at 12:23:06PM +0000, Alexey Dokuchaev wrote: > ... > I'm still not convinced though, sorry. Ports tree can be obtained by > a number of means, but this new ugly TIMESTAMP thingy is added for a > very specific usecase, and there should be no problem to require that > for that particular usecase, exported ports tree must have its files' > mtimes correctly set. (If svn/git/hg are not setting right mtimes on > export, they should be fixed.) It looks more like quick'n'dirty hack > rather than thoroughly thought-out solution. Given lack of replies, I guess I'd have to elaborate a bit on problems with TIMESTAMP and why I'm against it. 1. It does not line up with distinfo format. This file contains certain properties for each distfile a port requires, namely SIZE() and SHA256(), which are used to verify their integrity. Notice that distinfo may list several distfiles, and both SIZE() and SHA256() are essentially functions of a particular distfile. Introducing TIMESTAMP breaks this logic, as it's not attached to anything. Which distfile does it refer to? If it supposed to be bumped every time distinfo changes, why not simply use max(m in mtime(distfiles)) if there is (or rather, you *believe* there is) a problem with mtime(distinfo)? 2. It is not needed even if ports repo is obtained as tarball: svn/git/hg all correctly set mtimes on files ("svn export", "hg/git archive"). If users are tarring their checked out copy they're doing it wrong and should be instructed how to export repositories correctly. Putting TIMESTAMP might be easier than writing proper documentation, but that's not an excuse good enough for FreeBSD. Please reconsider. ./danfe