From owner-svn-src-vendor@freebsd.org Sun May 29 19:22:26 2016 Return-Path: Delivered-To: svn-src-vendor@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 3BA31B53611; Sun, 29 May 2016 19:22:26 +0000 (UTC) (envelope-from cy.schubert@komquats.com) Received: from smtp-out-so.shaw.ca (smtp-out-so.shaw.ca [64.59.136.138]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "Client", Issuer "CA" (not verified)) by mx1.freebsd.org (Postfix) with ESMTPS id E27B21B52; Sun, 29 May 2016 19:22:25 +0000 (UTC) (envelope-from cy.schubert@komquats.com) Received: from spqr.komquats.com ([96.50.22.10]) by shaw.ca with SMTP id 76HxbbV2JBLFm76Hybqj1U; Sun, 29 May 2016 13:22:19 -0600 X-Authority-Analysis: v=2.2 cv=W5d6blek c=1 sm=1 tr=0 a=jvE2nwUzI0ECrNeyr98KWA==:117 a=jvE2nwUzI0ECrNeyr98KWA==:17 a=L9H7d07YOLsA:10 a=9cW_t1CCXrUA:10 a=s5jvgZ67dGcA:10 a=yrkiwgmsf1kA:10 a=6I5d2MoRAAAA:8 a=BWvPGDcYAAAA:8 a=YxBL1-UpAAAA:8 a=QvAgiI5a5Fp75Pt56YEA:9 a=IjZwj45LgO3ly-622nXo:22 a=pxhY87DP9d2VeQe4joPk:22 a=Ia-lj3WSrqcvXOmTRaiG:22 Received: from slippy.cwsent.com (slippy [10.1.1.91]) by spqr.komquats.com (Postfix) with ESMTPS id 9959513751; Sun, 29 May 2016 12:22:17 -0700 (PDT) Received: from slippy (localhost [127.0.0.1]) by slippy.cwsent.com (8.15.2/8.15.2) with ESMTP id u4TJMHUZ069145; Sun, 29 May 2016 12:22:17 -0700 (PDT) (envelope-from Cy.Schubert@cschubert.com) Message-Id: <201605291922.u4TJMHUZ069145@slippy.cwsent.com> X-Mailer: exmh version 2.8.0 04/21/2012 with nmh-1.6 Reply-to: Cy Schubert From: Cy Schubert X-os: FreeBSD X-Sender: cy@cwsent.com X-URL: http://www.cschubert.com/ To: Pedro Giffuni cc: Cy Schubert , src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-vendor@freebsd.org Subject: Re: svn commit: r300961 - vendor/one-true-awk/dist In-Reply-To: Message from Pedro Giffuni of "Sun, 29 May 2016 14:08:59 -0500." <574B3E4B.2020501@FreeBSD.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Date: Sun, 29 May 2016 12:22:16 -0700 X-CMAE-Envelope: MS4wfLXPICTXZfzp8DMk99hrDZkPZ2O0bul+yoRV2QOdgBMEqy+KosBgBL7FpHfBJX3dCpfLoQoC4rIKKCex4qDzqrDQWwNsnRbnbziG8kfLsf3FHSEDlV4L 4ahT7OKGpaVrqBEy8w7FuqL3AINlq8MssIeYVnUqJIVRnp8+2w3ZDpi3rmZfR9IeIWJNCofeznf8W2VTnlNztkXx2fDFFDnOlkG19DLY+Sw8UTwTpGN1TLRn SuhnIc4GjPFrRxVQK9fhwjDtC73L6QuHGcSddP2DkXEZLCJzdGAV4DBJYC26vuIm X-BeenThere: svn-src-vendor@freebsd.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: SVN commit messages for the vendor work area tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 29 May 2016 19:22:26 -0000 In message <574B3E4B.2020501@FreeBSD.org>, Pedro Giffuni writes: > On 29/05/2016 13:17, Cy Schubert wrote: > > In message <574B2EAC.3010908@FreeBSD.org>, Pedro Giffuni writes: > >> > >> > >> On 29/05/2016 12:37, Cy Schubert wrote: > >>> In message <201605291618.u4TGItNJ024583@repo.freebsd.org>, "Pedro F. > >>> Giffuni" w > >>> rites: > >>>> Author: pfg > >>>> Date: Sun May 29 16:18:55 2016 > >>>> New Revision: 300961 > >>>> URL: https://svnweb.freebsd.org/changeset/base/300961 > >>>> > >>>> Log: > >>>> one-true-awk: replace 0 with NULL for pointers > >>>> > >>>> Also remove a redundant semicolon. > >>>> Submitted upstream already. > >>>> > >>>> Modified: > >>>> vendor/one-true-awk/dist/b.c > >>>> vendor/one-true-awk/dist/lex.c > >>>> vendor/one-true-awk/dist/maketab.c > >>>> vendor/one-true-awk/dist/parse.c > >>>> vendor/one-true-awk/dist/run.c > >>>> vendor/one-true-awk/dist/tran.c > >>>> > >>> Was this commit and r300962 obtained from the upline or vendor or were > >>> these commits local to FreeBSD only? > >>> > >>> > >> There is no public awk public repository AFAICT, but bwk acknowledged > >> the submission. > >> > >> The change to openresolv was merged to the public repository. > > As they've acknowledged the submissions, can you please tag the new > > versions of awk and openresolve with the correct upstream version numbers, > > please? > > > That's an impossible request as there are no "correct upstream version > numbers". > > In the case of openresolv, I included the repository revision, but > checksums have > no chronological sense and should be avoided for tags. > > In this case using tags for anything other than official releases would > be a mess. > As stated in our subversion primer (5.4.4): > > "Vendor patches should be committed to the vendor branch, and merged > from there to head. If the patch addresses an issue in a new release > that is currently being imported, it /must not/ be committed along with > the new release: the release must be imported and tagged first, then the > patch can be applied and committed. There is no need to re-tag the > vendor sources after committing the patch." > That doesn't address local patches. These were local patches that were upstreamed. I rea this as, if one got a patch from an upstream VCS repo but these were our local patches. At the very least the commit logs should reference the upstream commit log entry (which the openresolve one did but the awk commit did not). I think that a clear audit trail back to the vendor VCS, bug#, etc., be documented in the commit log. -- Cheers, Cy Schubert or FreeBSD UNIX: Web: http://www.FreeBSD.org The need of the many outweighs the greed of the few.