Date: Fri, 17 Aug 2018 12:35:25 -0400 From: Mark Johnston <markj@freebsd.org> To: Conrad Meyer <cem@freebsd.org> Cc: Warner Losh <imp@bsdimp.com>, src-committers <src-committers@freebsd.org>, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r337936 - in head/sys/contrib/libsodium: . builds builds/msvc builds/msvc/build builds/msvc/properties builds/msvc/vs2010 builds/msvc/vs2010/libsodium builds/msvc/vs2012 builds/msvc/vs2... Message-ID: <20180817163525.GB13168@raichu> In-Reply-To: <CAG6CVpUW7VkTvNNWe3n%2Bqyw3rgu2Tgipp323M7N7M7mJMQdXfA@mail.gmail.com> References: <cem@FreeBSD.org> <201808170023.w7H0Np0H027296@repo.freebsd.org> <201808170503.w7H53GR9023607@slippy.cwsent.com> <CAG6CVpXHWN7%2BNqoVKxMuvrKsbhO2pC%2BqbuYEDnNp8H-k5ubk6w@mail.gmail.com> <CANCZdfoOvTAM-u2Nj4cVBbPT9xhKBhZkEdm2MCv9bVnmuZXT%2Bw@mail.gmail.com> <CAG6CVpUW7VkTvNNWe3n%2Bqyw3rgu2Tgipp323M7N7M7mJMQdXfA@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Fri, Aug 17, 2018 at 09:17:58AM -0700, Conrad Meyer wrote: > On Fri, Aug 17, 2018 at 9:12 AM, Warner Losh <imp@bsdimp.com> wrote: > > > > On Fri, Aug 17, 2018, 10:05 AM Conrad Meyer <cem@freebsd.org> wrote: > >> We don't intend[sic] on tailoring it at all. > > > > We don't document such an exception... history tells me this belief may be > > optimistic.. > > I think it is a generally understood convention re: contrib code. I > called out intent in the commit message for the directory, and anyway > it is generally frowned upon in the project to modify contrib code for > non-abandoned upstreams. E.g., there was quite a bit of resistance > modifying top(1) until it was deemed dead upstream and moved out of > contrib. Rather, I'd say it shouldn't be done gratuitously: every commit to contrib code adds to our cumulative technical debt until it is reconciled with upstream. Sometimes, however, that's the right tradeoff to make (or it's simply necessary for some reason). In that case having a vendor branch is important, and one should be created preemptively even if there is no current intention to modify the imported code.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20180817163525.GB13168>