Skip site navigation (1)Skip section navigation (2)
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>