Date: Wed, 26 Aug 2020 10:33:09 -0700 From: John Baldwin <jhb@FreeBSD.org> To: Matthew Macy <mat.macy@gmail.com>, Toomas Soome <tsoome@freebsd.org> Cc: src-committers <src-committers@freebsd.org>, svn-src-all <svn-src-all@freebsd.org>, svn-src-head <svn-src-head@freebsd.org> Subject: Re: svn commit: r364806 - in head/sys/contrib/openzfs: include/os/freebsd/spl/sys module/os/freebsd/spl Message-ID: <d4b0c0a0-88dd-7a88-2899-4781aa7223d2@FreeBSD.org> In-Reply-To: <CAPrugNoaOLF0V5h8N_A_2BZ9sGAuhAkVi8-9of3RqqiSQMmdmg@mail.gmail.com> References: <202008260729.07Q7THlo023745@repo.freebsd.org> <CAPrugNoaOLF0V5h8N_A_2BZ9sGAuhAkVi8-9of3RqqiSQMmdmg@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On 8/26/20 10:15 AM, Matthew Macy wrote: > Do not commit directly to sys/contrib. PR, vendor branch update, then merge. We do live in a world where source control is able to merge changes, and it should be ok in that world to commit things directly while waiting for the PR to be merged and letting the VCS figure it out on the next merge. We haven't had a policy of never comitting directly to contrib/ since the CVS days. I do agree that changes should be pushed upstream to keep local diffs as small as possible, but reasonable judgement should be applied rather than a hard rule banning all commits. > On Wed, Aug 26, 2020 at 12:29 AM Toomas Soome <tsoome@freebsd.org> wrote: >> >> Author: tsoome >> Date: Wed Aug 26 07:29:17 2020 >> New Revision: 364806 >> URL: https://svnweb.freebsd.org/changeset/base/364806 >> >> Log: >> remove pragma ident lines >> >> The #pragma ident is historical relict and not needed any more, this >> pragma is actually unknown for common compilers and is only causing >> trouble. -- John Baldwin
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?d4b0c0a0-88dd-7a88-2899-4781aa7223d2>